[Halt]

Would this be correct?

PHP Code:

    for    (new i = 1; i <= MaxClients; i++)
        if    (IsClientInGame(i) && bIsPlayerAlive[i] && bIsPlayerZombie[i])
        {
            SetEntityRenderColor(i, 255, 255, 255, 255);
            TF2_ChangeClientTeam(i, TFTeam_Blue);
            TF2_RegeneratePlayer(i);
            CreateTimer(0.3, RespawnPlayer, i);
        }
        else if (IsClientInGame(i) && bIsPlayerAlive[i] && !bIsPlayerZombie[i])
            CreateTimer(0.3, RespawnPlayer, i);
        CreateTimer(0.3, ResetRound); 


or this

PHP Code:

    for    (new i = 1; i <= MaxClients; i++)
        {
        if    (IsClientInGame(i) && bIsPlayerAlive[i] && bIsPlayerZombie[i])
        {
            SetEntityRenderColor(i, 255, 255, 255, 255);
            TF2_ChangeClientTeam(i, TFTeam_Blue);
            TF2_RegeneratePlayer(i);
            CreateTimer(0.3, RespawnPlayer, i);
        }
        else if (IsClientInGame(i) && bIsPlayerAlive[i] && !bIsPlayerZombie[i])
            CreateTimer(0.3, RespawnPlayer, i);
        }
        CreateTimer(0.3, ResetRound); 


[fiction]

I believe it's just preference, so you can use whichever looks better to you. The SourceMod contributing guidelines says to always use braces for block statements, so if you want to follow that standard you can. https://wiki.alliedmods.net/Contribu...d#Coding_Style

[Neuro Toxin]

Tbh those both look horrible.

public void SomeFuntion()
{
   for (int client=1; client <= MaxClients; client++)
   {
      if (!IsClientInGame(client))
         continue;

      // code goes here
   }
}

[Bacardi]

You do multiple timers (ResetRound) for purpose ? Spawning and ResetRound ?

PHP Code:

...
{
     for (new i = 1; i <= MaxClients; i++)
     {
        if(!IsClientInGame(i) || !IsPlayerAlive(i)) continue;

        if(bIsPlayerZombie[i])
        {
            SetEntityRenderColor(i, 255, 255, 255, 255);
            TF2_ChangeClientTeam(i, TFTeam_Blue);
            TF2_RegeneratePlayer(i);
            CreateTimer(0.3, RespawnPlayer, i);
        }
        else
        {
            CreateTimer(0.3, RespawnPlayer, i);
        }
     }
     
     CreateTimer(0.3, ResetRound); 
 } 


*edit
not sure does your variable bIsPlayerAlive[i] match with native IsPlayerAlive...

[Neuro Toxin]

That awful whitespace you place between "if"/"for" and "(" hurts my brain.

Bad:

for    (int client; ...)

Good:

for (int client; ...)