battle309 Posted September 24, 2012 Posted September 24, 2012 i want to get $100 every 5 mins it saying lots of error dont know why function giveMoney(source) givePlayerMoney ( thePlayer, 1000 ) setTimer ( giveMoney, 5000, 0 ) local money = getPlayerMoney(source) if (money > 6000) then resetTimer ( giveMoney ) end end addEventHandler ( "onPlayerSpawn", getRootElement(), giveMoney )
DiSaMe Posted September 24, 2012 Posted September 24, 2012 Because you make the giveMoney function create an infinite timer which calls giveMoney itself. That means, every time a new timer is created. If only want one repeating cycle, either create the timer in another function and attach that function to "onPlayerSpawn" event or simple change the third setTimer argument to 1.
Anderl Posted September 24, 2012 Posted September 24, 2012 Also, first parameter of resetTimer is the timer, not the function attached to the timer.
Baseplate Posted September 24, 2012 Posted September 24, 2012 function kay (source) givePlayerMoney (source, 100) setTimer(kay, 300000, 1) end addEventHandler ("onPlayerSpawn", root, kay) Not sure/untested but I think that's what u want i want to get $100 every 5 mins
Anderl Posted September 24, 2012 Posted September 24, 2012 function kay (source) givePlayerMoney (source, 100) setTimer(kay, 300000, 1) end addEventHandler ("onPlayerSpawn", root, kay) Not sure/untested but I think that's what u want i want to get $100 every 5 mins There's no source parameter.
Baseplate Posted September 24, 2012 Posted September 24, 2012 Not sure/untested Meh, I forgot anyways.
battle309 Posted September 24, 2012 Author Posted September 24, 2012 function kay (source) givePlayerMoney (source, 100) setTimer(kay, 300000, 1) end addEventHandler ("onPlayerSpawn", root, kay) got errors line 2
Baseplate Posted September 24, 2012 Posted September 24, 2012 Okay lemme check function kay (thePlayer) givePlayerMoney (thePlayer, 100) setTimer(kay, 300000, 1) end addEventHandler ("onPlayerSpawn", root, kay) Try this, untested
Anderl Posted September 24, 2012 Posted September 24, 2012 Okay lemme check function kay (thePlayer) givePlayerMoney (thePlayer, 100) setTimer(kay, 300000, 1) end addEventHandler ("onPlayerSpawn", root, kay) Try this, untested Again, there's no player parameter. It's source and it's not sent in parameters of function. function kay (source) givePlayerMoney (source, 100) setTimer(kay, 300000, 1) end addEventHandler ("onPlayerSpawn", root, kay) got errors line 2 If you would have read what I said, you would not get them but you seem like you just want to copy code.
Castillo Posted September 24, 2012 Posted September 24, 2012 If I'm right, it's inefficient to use one timer for each player, and it's more efficient to loop all players with one timer. setTimer ( function ( ) for _, player in ipairs ( getElementsByType ( "player" ) ) do givePlayerMoney ( player, 100 ) end end ,300000, 0 )
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now