Variables in timer functions

[captain1342]

Hey,

I am working on a new addon and for that I need to start a timer for each player, but how can I get the player for wich the timer was started, inside the function? I wanted to use the timer name as storage but that doesen't work and I have no more ideas how I could do it.

This is what I tryed:

Code: Lua

1. local num = table.insert( AntiHighPing.HighPingPlayers, v )
2. timer.Create( (v:SteamID64().."_"..num.."_high_pinger"), AntiHighPing.Config.WaitSecBeforeKick, 1, AntiHighPing.CheckPlayer() )

[iViscosity]

Well if you already have "v" as your variable for player (in a player. GetAll() loop I'm assuming), you could just pass v to your CheckPlayer() function.

[captain1342]

You mean I can just do CheckPlayer(v) in the function?
but when I start another timer with an other V does it still give the old V or the new V ?

Like
Player = Player1
timer.Create( "meh"..num, AntiHighPing.Config.WaitSecBeforeKick, 1, AntiHighPing.CheckPlayer(Player))
Player = Player2
timer.Create( "meh"..num, AntiHighPing.Config.WaitSecBeforeKick, 1, AntiHighPing.CheckPlayer(Player))

timer("Meh1") = Player2
timer("Meh2") = Player2

[iViscosity]

Sorry, wasn't really clear when I made that. I was assuming you had a player.GetAll() loop, which is where you got "v" from. This is what I thought:

Code: Lua

1. for _, v in pairs( player.GetAll() ) do
2.     -- Some code if you had before
3.     local num = table.insert( AntiHighPing.HighPingPlayers, v )
4.     timer.Create( v:SteamID64() .. "_" .. num .. "_high_pinger",  AntiHighPing.Config.WaitSecBeforeKick, 1, function() AntiHighPing.CheckPlayer( v ) end )
5. end
6.  

P.s. sorry I'm at work right now but I'll try to give a more in-depth answer when I get home.

[captain1342]

Okay I get what you are doing but the function runns at the end of the timer, so when I start another 2 timers wouldn't that overwrite the v variable?
I mean how does it get an temp var?

[JamminR]

I'm reasonably sure that it remembers because it's filled in when it runs.
Have you tried it? Do you get a null error?
If it doesn't work, try a simple bracket precedence.
See if using parenthesis around the var v makes it 'fill' before being run and 'remembered'
AntiHighPing.CheckPlayer( ( v ) )
Or, you just inserted it into a table...
Does anything else modify that table?
If not, you could, in theory, use;
AntiHighPing.CheckPlayer( AntiHighPing.HighPingPlayers.num )
If you can get v to work, it would be more efficient however. Tables are almost always more costly in the long run.

FYI - There's a discussion on this at Facepunch. My Google-Fu found it.
What you're trying to do, that iViscosity stated, should work, because v is set outside the timer create.
https://facepunch.com/showthread.php?t=1548817&p=51677181

[captain1342]

Hadn't time to try it yet, I will test it tomorrow, thanks guys.