[maqi]

maybe in Ham_Killed pre, everything else is called after the user is already dead.

[^SmileY]

Originally Posted by CrAzY MaN Okay, will mixup that. But isn't there a way of doing this with DeathMsg or client_death?

Yes maybe DeathMsg since client_death is only used to count stats in csx module.

[HamletEagle]

I would go with Ham_Killed registered as pre. Btw, hamsandwich is a very useful module, trying to avoid it is like shooting yourself in the foot and trying to walk.

[CrAzY MaN]

#include <amxmodx>
#include <cstrike>
#include <fun>
#include <hamsandwich>
#include <fakemeta>
#include <engine>
#if AMXX_VERSION_NUM < 183
    #include <dhudmessage>
#endif

#define PLUGIN "Test:Remember Weapons on Respawn"
#define VERSION "1.0"
#define AUTHOR "CrAzY MaN"

new Array:g_weaponname;
new iWeapons

public plugin_init()
{
    register_plugin(PLUGIN, VERSION, AUTHOR)
    
    RegisterHam(Ham_Killed, "player", "Ham_Killed_Pre", 0)
    //RegisterHam(Ham_Killed, "player", "Ham_Killed_Post", 1)
}

public Ham_Killed_Pre(id)
{
    g_weaponname = ArrayCreate(32);
    get_weapons(id)
}

//public Ham_Killed_Post(id) // or 
public client_death(killer, victim)
{      
    set_task(3.0, "respawn_player", victim)
    
    set_dhudmessage(0, 255, 0, -1.0, 0.77, 0, 6.0, 12.0)
    show_dhudmessage(victim, "You will respawn in 3 seconds.")
}

public get_weapons(id)
{
    new weapons[32], weapon_name[32];
    get_user_weapons(id, weapons, iWeapons)
    
    for (new i=0; i < iWeapons; i++)
    {      
        get_weaponname(weapons[i], weapon_name, charsmax(weapon_name))
        
        ArrayPushString(g_weaponname, weapon_name)
        
        client_print(id, print_chat, "Old Weapons : %s", weapon_name)
    }
}

public give_weapons(id)
{
    strip_user_weapons(id)
    new weapon_name[32];
    
    for(new i=0; i < iWeapons; i++ )
    {
        ArrayGetString(g_weaponname, i, weapon_name, charsmax(weapon_name))
        
        give_item(id, weapon_name)
        //ExecuteHam(Ham_AddPlayerItem, weapon_name[i], id)
        
        client_print( id, print_chat, "New Weapons : %s", weapon_name )
    }   
}

//----------------------------------------------------------------//
/*-------------------------RESPAWN PLAYER-------------------------*/
//----------------------------------------------------------------//
public respawn_player(id)
{
    if (!is_user_alive(id) && (CS_TEAM_T <= cs_get_user_team(id) <= CS_TEAM_CT))
    {
        ExecuteHamB(Ham_CS_RoundRespawn, id)
    }
    
    give_weapons(id)
                
    ArrayClear(g_weaponname);
}

What am i doing wrong here?

[maqi]

Nvm i was on the phone. What part is not working ?

[PurposeLessx]

Why do you use client_death and Ham_Killed together? Go with Ham_Killed and respawn player?
And what kind of code is it ? Why do you use this?

(CS_TEAM_T <= cs_get_user_team(id) <= CS_TEAM_CT)

[maqi]

What?

[maqi]

Make sure that num has an initial value of 0 or the native will not work correctly.

[shauli]

Originally Posted by CrAzY MaN What am i doing wrong here?

You've created g_weaponname in Ham_Killed_Pre but never destroyed it, so you're basically recreating it over and over again.

Actually I don't see the point in using dyanmic arrays here, just use a regular one. Also it's better to just save the weapon index and use get_weaponname in respawn_player() instead of saving the strings.

BTW, is the current code working? If not, try to tell us when or where you think there's a problem, and show us the results from the client_print().

[CrAzY MaN]

Ham_Killed as post works same as client_death.
So there's no difference.

I'm clearing the array(IDK maybe you haven't seen) by "ArrayClear(g_weaponname).
Should I use ArrayDestroy?

Actually, it works perfectly.
Getting perfect "wepon names" on client print.

It just shows error of "ArrayGetString".
Will attach full error soon.