[sHoC]

Originally Posted by mug1wara You could just remove the index of the global variable if you want all players kills to be shown. For hud: PHP Code: #include <sourcemod> int g_iKills[MAXPLAYERS + 1]; public void OnPluginStart() {     HookEvent("player_death", Event_Death);          HookEvent("round_end", Event_End); } public void OnGameFrame() {     SetHudTextParams(0.5, 0.5, 1.0, 1, 1, 255, 1, 0);          for (int i = 1; i <= MaxClients; i++)     {         if (IsClientInGame(i))         {             int iCount = GetAlivePlayersCount();                          ShowHudText(i, 1, "Remaining players: %s Kills: %s", iCount, g_iKills[i]);         }     } } public Action Event_Death(Event hEvent, const char[] sName, bool bDontBroadcast) {     int iAttacker = hEvent.GetInt("attacker");          g_iKills[iAttacker]++; } public Action Event_End(Event hEvent, const char[] sName, bool bDontBroadcast) {     int iClient = GetClientOfUserId(hEvent.GetInt("userid"));          g_iKills[iClient] = 0; } stock int GetAlivePlayersCount() {     int iCount;          for (int i = 1; i <= MaxClients; i++)     {         if (IsClientInGame(i) && IsPlayerAlive(i))             iCount++;     }          return iCount; }

this version dosent work, remaining players and kills dosent show nothing. if I switch %s with %d for both, there will be showing the number of players alive but when I make a kill still 0 kills

[KlausLaw]

PHP Code:

#include <sourcemod>
#include <sdktools>
#include <cstrike>
#include <sdkhooks>


int g_Kills[MAXPLAYERS + 1];

public void OnPluginStart()
{
    HookEvent("round_end", Event_RoundEnd);
    HookEvent("player_death", Event_PlayerDeath);
    CreateTimer(1.0, Timer_HudMsg, _, TIMER_REPEAT);
}


public Action Event_PlayerDeath(Handle event, const char[] name, bool dontBroadcast)
{
    int attacker = GetClientOfUserId(GetEventInt(event, "attacker"));
    g_Kills[attacker]++;
    return Plugin_Continue;
}

public Action Event_RoundEnd(Event event, const char[] name, bool dontBroadcast)
{
    for (int i = 1; i <= MaxClients; i++)
    {
        g_Kills[i] = 0;
    }
    return Plugin_Continue;
}

public Action Timer_HudMsg(Handle timer)
{
    static int iAlivePlayers;
    iAlivePlayers = 0;
    for (int i = 1; i <= MaxClients; i++)
    {
        if (!IsClientInGame(i) || !IsPlayerAlive(i))continue;
        iAlivePlayers++;
    }
    SetHudTextParams(0.4, 0.2, 1.0, 2, 2, 252, 1, 0);
    for (int i = 1; i <= MaxClients; i++)
    {
        if (!IsClientInGame(i))continue;
        ShowHudText(i, 1, "Remaining players: %d\nKills: %d", iAlivePlayers, g_Kills[i]);
    }
} 


Enjoy

[KlausLaw]

And @mug1wara
1.You can't get a client from round end.
2.You are wasting so many resources, you are doing a loop inside a loop on every frame.
You don't have to get the amount of alive players on every client in the loop, you can just get it outside the loop, and you don't have to do it on every frame, it's just a waste.

[sHoC]

Originally Posted by KlausLaw PHP Code: #include <sourcemod> #include <sdktools> #include <cstrike> #include <sdkhooks> int g_Kills[MAXPLAYERS + 1]; public void OnPluginStart() {     HookEvent("round_end", Event_RoundEnd);     HookEvent("player_death", Event_PlayerDeath);     CreateTimer(1.0, Timer_HudMsg, _, TIMER_REPEAT); } public Action Event_PlayerDeath(Handle event, const char[] name, bool dontBroadcast) {     int attacker = GetClientOfUserId(GetEventInt(event, "attacker"));     g_Kills[attacker]++;     return Plugin_Continue; } public Action Event_RoundEnd(Event event, const char[] name, bool dontBroadcast) {     for (int i = 1; i <= MaxClients; i++)     {         g_Kills[i] = 0;     }     return Plugin_Continue; } public Action Timer_HudMsg(Handle timer) {     static int iAlivePlayers;     iAlivePlayers = 0;     for (int i = 1; i <= MaxClients; i++)     {         if (!IsClientInGame(i) || !IsPlayerAlive(i))continue;         iAlivePlayers++;     }     SetHudTextParams(0.4, 0.2, 1.0, 2, 2, 252, 1, 0);     for (int i = 1; i <= MaxClients; i++)     {         if (!IsClientInGame(i))continue;         ShowHudText(i, 1, "Remaining players: %d\nKills: %d", iAlivePlayers, g_Kills[i]);     } }  Enjoy

I tested it, when I make a kill its still 0 the number of kills

[mug1wara]

Me doing a loop inside a loop, so do you. ^^ I'd still use OnGameFrame, works fine for me. Also didn't know you couldn't get client index from round_end? Better check on wiki again thanks.
@shoc

Use this, tested and working.

PHP Code:

#include <sourcemod>

int g_iKills[MAXPLAYERS + 1];

public void OnPluginStart()
{
    HookEvent("player_death", Event_Death);
    
    HookEvent("round_end", Event_End);
}

public void OnGameFrame()
{
    SetHudTextParams(-1.0, 0.4, 0.1, 1, 255, 1, 0);
    
    for (int i = 1; i <= MaxClients; i++)
    {
        if (IsClientInGame(i))
        {
            int iCount = GetAlivePlayersCount();
            
            ShowHudText(i, 1, "Remaining players: %i | Kills: %i", iCount, g_iKills[i]);
        }
    }
}

public Action Event_Death(Event hEvent, const char[] sName, bool bDontBroadcast)
{
    int iAttacker = GetClientOfUserId(hEvent.GetInt("attacker"));
    
    g_iKills[iAttacker]++;
}

public Action Event_End(Event hEvent, const char[] sName, bool bDontBroadcast)
{
    for (int i = 1; i <= MaxClients; i++)
        g_iKills[i] = 0;
}

stock int GetAlivePlayersCount()
{
    int iCount; 
    
    for (int i = 1; i <= MaxClients; i++) 
    { 
        if (IsClientInGame(i) && IsPlayerAlive(i)) 
            iCount++; 
    } 
    
    return iCount; 
} 


[KlausLaw]

Maybe you did something wrong, just download the plugin.
Maybe you tested it with 1 player? because if you kill him, it will trigger round end, and then the kills will get reset.
I can make it to reset on round start if you'd like.

and mug1wara, i'm not doing an inside loop.
You are doing a loop to get the alive players amount inside the loop to show the hud.
What i'm doing, is to get the alive players amount, and then i'm doing a loop to show the hud, HUGE DIFFERENCE.

[mug1wara]

ok fine ):<