[beastedout]

What am I doing wrong ;_;? No Compilation error! Server Console error (in title) whenever round starting..

PHP Code:

#include <sourcemod>


public void OnPluginStart()
{
    HookEvent("round_start", OnRoundStart);
}
public void OnRoundStart(Handle event, char[] name, bool dbc)
{
    new client = GetClientOfUserId(GetEventInt(event,"userid"));
    if (GetClientTeam(client) == 2) 
    {
        PrintToChat(client,"Hello Terrorists!");
    }
} 


[Ilusion9]

round_start doesn't have a variable called userid.

PHP Code:

#include <sourcemod>


public void OnPluginStart()
{
    HookEvent("round_start", OnRoundStart);
}
public void OnRoundStart(Handle event, char[] name, bool dbc)
{
    for(int i = 1; i <= MaxClients; i++)
    {
        if(IsClientInGame(i))
        {
            if (GetClientTeam(i) == 2) 
            {
                PrintToChat(i,"Hello Terrorists!");
            }
        }
    }
} 


[beastedout]

Originally Posted by Ilusion9 round_start doesn't have a variable called userid. PHP Code: #include <sourcemod> public void OnPluginStart() {     HookEvent("round_start", OnRoundStart); } public void OnRoundStart(Handle event, char[] name, bool dbc) {     for(int i = 1; i <= MaxClients; i++)     {         if(IsClientInGame(i))         {             if (GetClientTeam(i) == 2)              {                 PrintToChat(i,"Hello Terrorists!");             }         }     } }

OO, thanks man!

[Neuro Toxin]

CS_TEAM_T should be used instead of 2 for readability

[mug1wara]

Originally Posted by Neuro Toxin CS_TEAM_T should be used instead of 2 for readability

What?!

[Neuro Toxin]

GetClientTeam(i) == CS_TEAM_T

[mug1wara]

Originally Posted by Neuro Toxin GetClientTeam(i) == CS_TEAM_T

I totally understood that.

But to be totally honest, making a comment that says "Use CS_TEAM_T instead of 2" is just ridiculously dumb.

[DarkDeviL]

Originally Posted by mug1wara But to be totally honest, making a comment that says "Use CS_TEAM_T instead of 2" is just ridiculously dumb.

Constructive criticism is not "ridiculously dumb".

Frowning upon constructive criticism is "ridiculously dumb" though.

@Neuro Toxin is right, use == CT_TEAM_CT, etc. for readability. The team numbers are not the same in-game, and as such, things could confuse people with == 1, == 2, .. etc. in the code. CT_TEAM_CT would be easy to understand by reading the code.

[Powerlord]

Originally Posted by Neuro Toxin CS_TEAM_T should be used instead of 2 for readability

That adds a dependency on the cstrike module, though.

Right now it works on any game, although it'd be weird to refer to the Survivors (L4D/L4D2), RED team (TF2), Combine (HL2:DM), or Allies (DoD:S) as "Terrorists".

[mug1wara]

Originally Posted by Powerlord That adds a dependency on the cstrike module, though. Right now it works on any game, although it'd be weird to refer to the Survivors (L4D/L4D2), RED team (TF2), Combine (HL2M), or Allies (DoD:S) as "Terrorists".

That's what I thought!