[man_s_our]

since I tried this plugin with respawn related plugins and founded that the players whom killed by the bomb doesn't respawn I coded this respawner to solve the problem.

[HamletEagle]

This code has more problems than lines of code.

[man_s_our]

Originally Posted by HamletEagle This code has more problems than lines of code.

like what?

[HamletEagle]

Originally Posted by man_s_our like what?

1.You are using permanent tasks to respawn the player where you could simply hook Ham_Killed.
2.The way you get all players(loop with get_maxplayers) is wrong. In most cases you should use get_players.
3.Declaring variables in a loop.
4.If I remember right using spawn() on players has issues, but I don't feel like checking right now. Anyway, you should use Ham for this.
5.Creating different tasks with the same id.
6.Why are you using 2 tasks at 0.5 and at 0.7 seconds to respawn the player?
7.Why include cstrike if you are not going to use it?

[D3XT3R]

Originally Posted by HamletEagle 1.You are using permanent tasks to respawn the player where you could simply hook Ham_Killed. 2.The way you get all players(loop with get_maxplayers) is wrong. In most cases you should use get_players. 3.Declaring variables in a loop. 4.If I remember right using spawn() on players has issues, but I don't feel like checking right now. Anyway, you should use Ham for this. 5.Creating different tasks with the same id. 6.Why are you using 2 tasks at 0.5 and at 0.7 seconds to respawn the player? 7.Why include cstrike if you are not going to use it?

Why you told him all this if you can tell him as dexter show you in example
Also him code is stolen from the code that i showed you and i think this idea is already unnaproved D:

[OciXCrom]

Originally Posted by D3XT3R Why you told him all this if you can tell him as dexter show you in example

Why are you talking to yourself in third person?

The code is terrible. If you want to fix a broken plugin, you usually do it by making a better version, not this.

[man_s_our]

Originally Posted by HamletEagle 1.You are using permanent tasks to respawn the player where you could simply hook Ham_Killed. 2.The way you get all players(loop with get_maxplayers) is wrong. In most cases you should use get_players. 3.Declaring variables in a loop. 4.If I remember right using spawn() on players has issues, but I don't feel like checking right now. Anyway, you should use Ham for this. 5.Creating different tasks with the same id. 6.Why are you using 2 tasks at 0.5 and at 0.7 seconds to respawn the player? 7.Why include cstrike if you are not going to use it?

either I didn't see Ham_killed tutorial or it was used in other respawn plugins and didn't work with the c4.
this is the first time I know about get_players(). after all I'm amature in amxx coding.
I declared the index in the loop starting only. if you mean somewhere else then I forgetted about.
when I searched for the set_task tutorials I didn't find the identification for the id but I found that setting the same id for two tasks (at least if they aren't different) doesn't effect.

[D3XT3R]

Originally Posted by man_s_our since I tried this plugin with respawn related plugins and founded that the players whom killed by the bomb doesn't respawn I coded this respawner to solve the problem.

https://forums.alliedmods.net/showthread.php?p=8906

[D3XT3R]

can be done with better ways

#include < amxmodx > 
#include < amxmisc > 
#include < hamsandwich > 

public plugin_init( ) { 
    RegisterHam( Ham_Killed, "player", "Ham_PlayerDeath", 1 ) 
} 

public Ham_PlayerDeath( victim, attacker, shouldgib ) { 
    set_task( 1.0, "respawn", victim ) 
}

public respawn( victim ) { 
    if( is_user_alive( victim ) ) 
        return 
    
    ExecuteHamB( Ham_CS_RoundRespawn, victim ) 
}