Please help me.. whats wrong ?

[NeedSumHelp]

function MoneyForWinner() 
   local alivePlayers = getAlivePlayers() 
   if #alivePlayers == 1 then  
   local playername = getPlayerName( alivePlayers[1] ) 
   local serial = getPlayerSerial(alivePlayers[1]) 
   local Cash = executeSQLSelect( "GREENTHING", "Cash","serial = '" .. serial .. "'") 
    
   Cashs = tonumber(Cash[1]["Cash"]) + 150 
    
   setElementData(alivePlayers[1] ,"data.playername",playername,true) 
   setElementData(alivePlayers[1] ,"data.Cash",Cashs,true) 
         
   executeSQLUpdate ( "GREENTHING", "Cash = '"..Cashs.."'","serial = '" .. serial .. "'") 
   end  
end 
addEventHandler("onPlayerWasted", rootElement  , MoneyForWinner) 

it says bad argument @ getplayerserial and getplayername

This should give winner 150 rubla but w/e its not doing it and im sure everything is correct.. what i should do ?

[dzek (varez)]

im not sure if you should use 1 as table index.

try:

if #alivePlayers == 1 then 
for key,val in ipairs(alivePlayers) do 
  -- your stuff here 
  -- val is player element 
end 

[NeedSumHelp]

  
for key,val in ipairs(alivePlayers) do 
  

i changed to example:

local playername = getPlayerName( val[1] ) 

and no errors but still not working

[dzek (varez)]

show full code pls

[NeedSumHelp]

    function MoneyForWinner() 
       local alivePlayers = getAlivePlayers() 
       if #alivePlayers == 1 then 
for key,val in ipairs(alivePlayers) do 
      local playername = getPlayerName( val[1] ) 
      local serial = getPlayerSerial(val[1]) 
   local Cash = executeSQLSelect( "GREENTHING", "Cash","serial = '" .. serial .. "'") 
       
      Cashs = tonumber(Cash[1]["Cash"]) + 150 
        
     setElementData(val[1] ,"data.playername",playername,true) 
      setElementData(val[1] ,"data.Cash",Cashs,true) 
           
      executeSQLUpdate ( "GREENTHING", "Cash = '"..Cashs.."'","serial = '" .. serial .. "'") 
      end 
   end 
   addEventHandler("onPlayerWasted", rootElement  , MoneyForWinner) 

Or still error.. attempt to index local 'val' (a userdata value)

line

   local playername = getPlayerName( val[1] ) 

[dzek (varez)]

replace val[1] with only val

[NeedSumHelp]

ah right, tought it will change it for everyone but yeah u were right thanks for very very fast help.. never seen such speed in forums im serious ur the best moderator i ever seen

and i dint try to rise ur ego ^^ ok again, thank you

[dzek (varez)]

umm, well, thanks, being some kind of no-life makes my replies fast

hope i'll get new job soon..

whatever..

glad i helped