[thecount]

So I have about 4 or 5 timers in my plugin, and only 1 or 2 work. I've gone over to see if I've made them in different ways but they seem to be same syntax. Plugin compiles and everything, but when in-game nothing happens and it says something about an invalid timer handle.

Example of Timer:

PHP Code:

new Handle:h_kickall = INVALID_HANDLE;
h_kickall = CreateTimer(12.0, Timer_KickAll);

public Action:Timer_KickAll(Handle:timer){
    new String:daTeam[25];
    if(winnerByte == CS_TEAM_T){
        Format(daTeam, sizeof(daTeam), "Terrorists");
    }else{
        if(winnerByte == CS_TEAM_CT){
            Format(daTeam, sizeof(daTeam), "Counter-Terrorists");
        }else{
            Format(daTeam, sizeof(daTeam), "Nobody");
        }
    }
    for(new i=1;i<11;i++){
        KickClient(i, "Match over: %s won", daTeam);
    }
    ServerCommand("quit");
    CloseHandle(h_kickall);
    return Plugin_Continue;
} 


Thanks if you can help.

[xf117]

Never close timer's handle inside itself.

PHP Code:

public Action:Timer_KickAll(Handle:timer){
    new String:daTeam[25];
    if(winnerByte == CS_TEAM_T){
        Format(daTeam, sizeof(daTeam), "Terrorists");
    }else{
        if(winnerByte == CS_TEAM_CT){
            Format(daTeam, sizeof(daTeam), "Counter-Terrorists");
        }else{
            Format(daTeam, sizeof(daTeam), "Nobody");
        }
    }
    for(new i=1;i<11;i++){
        KickClient(i, "Match over: %s won", daTeam);
    }
    ServerCommand("quit");
    h_kickall = INVALID_HANDLE;
    return Plugin_Handled;
} 


[thecount]

Originally Posted by xf117 Never close timer's handle inside itself.

Ok I changed that but still doesn't call the timer.

[sheo]

Originally Posted by thecount PHP Code: for(new i=1;i<11;i++){         KickClient(i, "Match over: %s won", daTeam);     }

You need some extra checks for valid players here (is in game, is not a bot...)

[thecount]

Originally Posted by sheo You need some extra checks for valid players here (is in game, is not a bot...)

Still doesn't work after adding that.

[sheo]

Post your error than

[Bacardi]

Are you using global Handle at all... ?
You could just createtimer without it.

PHP Code:



that_callback_which_team_won()
{
    ...

    CreateTimer(12.0, timer_kickall, team);
    ...
}

public Action:timer_kickall(Handle:timer, any:team)
{
    for(new i = 1; i <= MaxClients; i++)
    {
        if(IsClientConnected(i) && !IsFakeClient(i))
        {
            KickClient(i, " \n Match over: %s won", team == 2 ? "Terrorists":(team == 3 ? "Counter-Terrorists":"Nobody") );
        }
    }

    ServerCommand("quit");
} 


[thecount]

Originally Posted by Bacardi Are you using global Handle at all... ? You could just createtimer without it.

Ok I got rid of the global handle and changed the timer to your sample code, but still doesn't call the action when the timer ends.

Error:

[SM] Invalid timer handle 820078 (error 3) during timer end, displayed function is timer callback, not the stack trace

[Bacardi]

show your code, all of it

[Mitchell]

I've never found a use to actually save a timer's handle, as you could just check a boolean before any of the functions in side the timer fire, and just return it to stop. (it'll free up the handle too)
Maybe i'm missing something though. Also there is the repeat flag, and no map change flag, that also helps.