[flamingkirby]

Hello everyone. I am puzzled as to why this is not working. I saw other people use this exact same code but when i die, it says i am still alive.

public OnPluginStart()
{
	HookEvent("player_death", EVENT_playerDeath);
}

public EVENT_playerDeath(Handle:event, const String:name[], bool:dontBroadcast)
{
	new client = GetClientOfUserId(GetEventInt(event, "userid"));
	
	if(getAliveClients() == 0) PrintToChatAll("All clients are dead!");
}

public getAliveClients()
{
	new count = 0;
	
	for(new i = 1; i <= MaxClients; i++)
	{
		if(IsFakeClient(i)) continue; //Ignores bots
		new name[32];
		GetClientName(i, name, 32);
		if(IsPlayerAlive(i) && IsPlayerInGame(i)) count++; PrintToChatAll("%s is alive", name);
	}
	
	return count;
}

The only thing that i may see wrong with this is the event, it is set at default, post which should happen after the event is fired so i should be declared dead in game but for some weird reason it says i am still alive. Can someone please see what is wrong with this code. Thanks.

[Sillium]

The PrintToChatAll is always executed because it is not part of the if clause:

if(IsPlayerAlive(i) && IsPlayerInGame(i)) count++; PrintToChatAll("%s is alive", name);

should probably be:

if(IsPlayerAlive(i) && IsPlayerInGame(i))
{
   count++;
   PrintToChatAll("%s is alive", name);
}

[flamingkirby]

Ye, just for debugging to see what it is printing. I think the problem was that i did not have the hook event in pre as when i changed it to pre, it seemed to work but just need a bigger crowd to test it fully.

[GsiX]

Do as Silium suggest.. otherwise you going no where..
Its so obvious you dead or alive, you will be declared alive.

[Versace14]

I am extremely happy to read about moltivite.. I need to read about this completely..
I'll share this info to my friends too..

[NIGathan]

If you're going to be doing something with the alive clients, you could do this:

PHP Code:

public getAliveClients(String:&targets[])
{
    decl String:target_name[MAX_TARGET_LENGTH];
    decl target_count;
    decl bool:tn_is_ml;
    if ((target_count = ProcessTargetString(
            "@all",
            0,
            targets,
            MAXPLAYERS,
            COMMAND_FILTER_ALIVE|COMMAND_FILTER_NO_BOTS,
            target_name,
            sizeof(target_name),
            tn_is_ml)) <= 0)
                return 0;
    return target_count;
} 


Haven't tested, and also have never used a reference variable in PAWN, so not sure if the format is correct.. But you give it a string array with size of MAXPLAYERS and it will be filled with the alive non-bot players, and the return value will be the number of matching clients.

[TnTSCS]

No need to hook the event in pre... the action of player death has already occurred, but the event is just the notification.

PHP Code:

public OnPluginStart()
{
    HookEvent("player_death", EVENT_playerDeath, EventHookMode_PostNoCopy);
}

public EVENT_playerDeath(Handle:event, const String:name[], bool:dontBroadcast)
{
    if (getAliveClients() == 0)
    {
        PrintToChatAll("All clients are dead!");
    }
}

getAliveClients()
{
    new count = 0;
    
    for (new i = 1; i <= MaxClients; i++)
    {
        if (!IsClientInGame(i) || IsFakeClient(i) || !IsPlayerAlive(i))
        {
            continue; //Check if client (i) is in game and Ignores bots and dead players
        }
        
        count++; 
        PrintToChatAll("%N is alive", i);
    }
    
    return count;
} 


And you don't need to declare a string and use get the player's name, just use the format %N with the client ID and it will give you the player's name...

Also, you need to always check if IsClientInGame whenever you're looping through all client IDs if you're going to work with client IDs otherwise, some of the calls/checks you make will error out on invalid clients.

[11530]

Not sure why new client = ... is even needed, unless you left some code out. If you haven't then, as TnTSCS suggested, you can use EventHookMode_PostNoCopy.

[TnTSCS]

True - sample code updated

[GsiX]

Otherwise, nearest local hospital provide "death declaration certificate".