[bladeee]

#pragma semicolon 1
#include <sourcemod>

#define PLUGIN_VERSION "1.0"

int rdm;
int rdm2;

public Plugin myinfo = {
    name        = "",
    author        = "blade",
    description = "",
    version        = PLUGIN_VERSION,
    url            = ""
};
public OnPluginStart(){
	HookEvent("round_start", OnRoundStart);
}
public Action OnRoundStart(Handle:event, const String:name[], bool:dontBroadcast){
	rdm = GetRandomInt(1, 5);
	if (rdm == 1){
		rdm2 = GetRandomInt(1, 3);
		if (rdm2 == 1){ 
			SetEntityHealth(//need help with getting client index
		}
	}
}

I want to use SetEntityHealth in my OnRoundStart method, but it needs the client index and I'm not sure how to grab it.
PS general criticism of my code will help, new to SP.

[ddhoward]

Client indexes range from 1 - MaxPlayers, and each client's index reflects what server slot he's connected to.

So if you want the client connected to slot 1, then the index would be 1. 2 is 2, 3 is 3, etc.

You are also mixing aspects of the "old" and "new" Sourcepawn syntaxes. You should put "void" inbetween "public" and "OnPluginStart", and replace this:

PHP Code:

public Action OnRoundStart(Handle:event, const String:name[], bool:dontBroadcast){ 


with this:

PHP Code:

public Action OnRoundStart(Handle event, const char[] name, bool dontBroadcast){ 


[thecount]

Originally Posted by bladeee I want to use SetEntityHealth in my OnRoundStart method, but it needs the client index and I'm not sure how to grab it.

Well, which client do you want? I'm going to assume that you want all players alive at this time. As stated previously, the range of players is 1 - MaxClients. But be aware that a for loop using the total online clients will be misleading to the code, as the server may reach 8 players, but everyone else except the eighth person could leave. In that situation, the server would say that one player is online, but the slot would not be slot 1; it'd be 8.

Below is a for loop that checks for in-game, alive clients with the variable i.

PHP Code:

for(new i=1;i<=MaxClients;i++){
    if(IsClientInGame(i) && IsPlayerAlive(i)){
        SetEntityHealth(i, NUMBER);
    }
} 


[bladeee]

Originally Posted by thecount Well, which client do you want? I'm going to assume that you want all players alive at this time. As stated previously, the range of players is 1 - MaxClients. But be aware that a for loop using the total online clients will be misleading to the code, as the server may reach 8 players, but everyone else except the eighth person could leave. In that situation, the server would say that one player is online, but the slot would not be slot 1; it'd be 8. Below is a for loop that checks for in-game, alive clients with the variable i. PHP Code: for(new i=1;i<=MaxClients;i++){     if(IsClientInGame(i) && IsPlayerAlive(i)){         SetEntityHealth(i, NUMBER);     } }

Thanks, you make a good point!
+1