[sereky]

PHP Code:

new client[MAXPLAYERS + 1];

public OnPluginStart()
{
    HookEvent("weapon_fire", OnWeaponFire);
    HookEvent("round_end", OnRoundEnd);
}

public Action:OnWeaponFire(Handle:event, const String:name[], bool:dontBroadcast)
{
    client[?] = GetClientOfUserId(GetEventInt(event, "userid"));
}

public Action:OnRoundEnd(Handle:event, const String:name[], bool:dontBroadcast)
{
*(how to use here the client of OnWeaponFire event?)
} 


The new client; would save only the last client.

I tried to save each client separately with new client[MAXPLAYERS + 1];, but I don't know how to use it in anohter event.

[Jоnny]

new client;

public OnPluginStart()
{
    HookEvent("weapon_fire", OnWeaponFire);
    HookEvent("round_end", OnRoundEnd);
}

public Action:OnWeaponFire(Handle:event, const String:name[], bool:dontBroadcast)
{
    client = GetClientOfUserId(GetEventInt(event, "userid"));
}

public Action:OnRoundEnd(Handle:event, const String:name[], bool:dontBroadcast)
{
    PrintToChat(client, "Hello, World!");
}  

[sereky]

Originally Posted by Jоnny ...

Sorry, I forgot something. I've updated the first post.

[Gate]

You better dont use client as a global Int... You better call it SavedClient to avoid problems with local ints...

[Bacardi]

There is no sense to save player indexs...
you have purpose to use event weapon_fire for some reason ?

...here example (maybe you didn't mean this), if you want store "shots" what players have done.
On round end it will print in server console stored shot by index and will print to that player chat who is game.

Spoiler

PHP Code:

new player_shots[MAXPLAYERS+1] // array size 66

public OnPluginStart()
{
    HookEvent("weapon_fire", weapon_fire);
    HookEvent("round_end", round_end);
}

public weapon_fire(Handle:event, const String:name[], bool:dontBroadcast) // every time when shot
{
    new a = GetClientOfUserId(GetEventInt(event, "userid")); // Get player index from event #userid

    player_shots[a] += 1; // Add +1 in player array
}

public round_end(Handle:event, const String:name[], bool:dontBroadcast) // When round end
{
    // Do something to players on round end ?

    for(new i = 1; i <= MaxClients; i++) // Do loop start from 1 to server maxplayers
    {
        PrintToServer("player_shots[index %i] = shots %i", i, player_shots[i]); // This will print to server console, shots stored to each array

        if(IsClientInGame(i)) // This player is in game
        {
            PrintToChat(i, "player_shots[your index %i] = Shots fired %i", i, player_shots[i]); // I can print to him in chat shots what he have shoot.
        }
    }
}

public OnClientConnected(client)
{
    // Player has joined server
    // Reset shots from that array index
    player_shots[client] = 0;
} 


uuhh... those old days.
Simple code what start count headshot [Help]

[sereky]

Originally Posted by Bacardi There is no sense to save player indexs... you have purpose to use event weapon_fire for some reason ? ...

My source code is only an example.

This is another example from here:
http://forums.alliedmods.net/showthread.php?t=175819

PHP Code:

#include <sourcemod>
#include <sdktools>

new clip[MAXPLAYERS + 1];
new client[MAXPLAYERS + 1];

public OnPluginStart()
{
    HookEvent("weapon_fire", OnWeaponFire);
    HookEvent("bot_player_replace", bot_player_replace);
}

public Action:OnWeaponFire(Handle:event, const String:name[], bool:dontBroadcast)
{
    client[?] = GetClientOfUserId(GetEventInt(event, "userid"));
    new victim;

    new slot = GetPlayerWeaponSlot(client, 0);
    decl String:weaponname[32];
    decl String:gun[64];
    GetClientWeapon(client, weaponname, sizeof weaponname);
    GetEdictClassname(slot, gun, sizeof(gun));
    if (StrEqual(gun, weaponname, false))
    {
        victim = GetPlayerWeaponSlot(client, 1);

        if (victim != -1)
        {
            clip[client] = GetEntProp(GetPlayerWeaponSlot(client, 1), Prop_Send, "m_iClip1")
            remove(client, 1);
            CreateTimer(5.0, givepistol, client);
            return;
        }
        return;
    }
}

public bot_player_replace(Handle:Spawn_Event, const String:Spawn_Name[], bool:Spawn_Broadcast)
{
    bot = GetClientOfUserId(GetEventInt(Spawn_Event, "bot"));
    player = GetClientOfUserId(GetEventInt(Spawn_Event, "player"));

    if (client[?] == bot)
    {
        giveweapon(player, "weapon_pistol");
        SetEntProp(GetPlayerWeaponSlot(player, 1), Prop_Send, "m_iClip1", clip[client]);
    }
}

public Action:givepistol(Handle:timer, any:client)
{
    giveweapon(client, "weapon_pistol");
    SetEntProp(GetPlayerWeaponSlot(client, 1), Prop_Send, "m_iClip1", clip[client]);
}

giveweapon(client, String:class[40])
{
    new ent = CreateEntityByName(class);

    if (ent != -1)
    {
        DispatchSpawn(ent);
        EquipPlayerWeapon(client, ent);
    }
}

remove(client, slot)
{
    new ent = GetPlayerWeaponSlot(client, slot);

    if (ent != -1)
    {
        RemovePlayerItem(client, ent);
        RemoveEdict(ent);
    }
} 


I can save the clip of each client by clip[client] = ... , but I want to save each client. I don't know how to use client[?]. What sould be in the brackets?

You can read the description of this plugin in the link above. The new problem is that if a player take control a bot who's pistol removed temporarily and the timer isn't ended, this fix would give it back immediately to the player.

if (client[?] == bot) could compare them. This is why I want to save each client.

[sereky]

What is the solution of this problem?

[Leonardo]

new bool:bMemory[ MAXPLAYERS+1 ] = { false, ... };

public Action:OnWeaponFire( Handle:hEvent, const String:strEventName[], bool:bDontBroadcast )
{
    new iClient = GetClientOfUserId( GetEventInt( hEvent, "userid" ) );
    
    if( !IsClientInGame( iClient ) )
        return Plugin_Continue;
        
    if( !IsFakeClient( iClient ) )
        return Plugin_Continue;
    
    bMemory[ iClient ] = true;
    
    return Plugin_Continue;
}

public Action:OnPlayerSpawn( Handle:hEvent, const String:strEventName[], bool:bDontBroadcast )
{
    new iClient = GetClientOfUserId( GetEventInt( hEvent, "userid" ) );
    
    if( !IsClientInGame( iClient ) )
        return Plugin_Continue;
    
    if( bMemory[ iClient ] )
    {
        // Do something;
        bMemory[ iClient ] = false;
    }
    
    return Plugin_Continue;
}

[TnTSCS]

I think we'd have a better way to help if you explained what you needed and what you ultimately wanted to do with that client variable... you can use Trie to store a bunch of information (but use userId instead of clientId)

[sereky]

Here is another example:

PHP Code:

#include <sourcemod>
#include <sdktools>

#pragma semicolon 1
new count[MAXPLAYERS+1];
new n_client;

public OnPluginStart()
{
    HookEvent("heal_begin", heal_begin);
    HookEvent("player_incapacitated", player_incapacitated);
}

public Action:player_incapacitated(Handle:event, const String:name[], bool:dontBroadcast)
{
    new client = n_client;
    count[client]++;
}

public Action:heal_begin(Handle:event, const String:name[], bool:dontBroadcast)
{
    new client;
    client = GetClientOfUserId(GetEventInt(event, "userid"));
    if (client == n_client)

    if (count[client]>1)
    {
        count[client]=0;
        ForcePlayerSuicide(client);
    }
} 


If a client incapacitated 2 times, then he begin healing, kill him. How to do this by storing clients?