[wEEk]

Hello, someone could help me because when they commit suicide either slay, kill and lasermine gain experience?

PHP Code:

public fw_PlayerKilled(victim, attacker, shouldgib)
{ 
    if (get_user_weapon(attacker) == CSW_HEGRENADE)
    {
        PlayerXP[attacker] += get_pcvar_num(XP_Hs)
        client_print(attacker, print_chat,"You won  [1 Frags]")
    
        if(get_pcvar_num(SaveXP) == 1) 
        { 
                SaveData(attacker)
        }
    }
    
    if (get_user_weapon(attacker) == CSW_KNIFE)
    {
        PlayerXP[attacker] += get_pcvar_num(XP_Knife)
        client_print(attacker, print_chat, "You won [1 Frags]")
    
        if(get_pcvar_num(SaveXP) == 1) 
        { 
                SaveData(attacker) 
        }
    }
    else
    {
        PlayerXP[attacker] += get_pcvar_num(XP_Kill) 
        client_print(attacker, print_chat, "You won [1 Frags]")
        if(get_pcvar_num(SaveXP) == 1) 
        { 
                SaveData(attacker) 
        }
    }

    while(PlayerXP[attacker] >= LEVELS[PlayerLevel[attacker]]) 
    { 
        PlayerLevel[attacker]++
        client_print(attacker, print_chat, "You win a level: %d", PlayerLevel[attacker])

        if(get_pcvar_num(SaveXP) == 1) 
        { 
            SaveData(attacker) 
        }
    } 
    SaveData(attacker)
} 


[Blizzard_87]

PHP Code:

public fw_PlayerKilled(victim, attacker, shouldgib) 
{  

    if(victim == attacker)
        return PLUGIN_HANDLED;

    if (get_user_weapon(attacker) == CSW_HEGRENADE) 
    { 
        PlayerXP[attacker] += get_pcvar_num(XP_Hs) 
        client_print(attacker, print_chat,"You won  [1 Frags]") 
     
        if(get_pcvar_num(SaveXP) == 1)  
        {  
                SaveData(attacker) 
        } 
    } 
     
    if (get_user_weapon(attacker) == CSW_KNIFE) 
    { 
        PlayerXP[attacker] += get_pcvar_num(XP_Knife) 
        client_print(attacker, print_chat, "You won [1 Frags]") 
     
        if(get_pcvar_num(SaveXP) == 1)  
        {  
                SaveData(attacker)  
        } 
    } 
    else 
    { 
        PlayerXP[attacker] += get_pcvar_num(XP_Kill)  
        client_print(attacker, print_chat, "You won [1 Frags]") 
        if(get_pcvar_num(SaveXP) == 1)  
        {  
                SaveData(attacker)  
        } 
    } 

    while(PlayerXP[attacker] >= LEVELS[PlayerLevel[attacker]])  
    {  
        PlayerLevel[attacker]++ 
        client_print(attacker, print_chat, "You win a level: %d", PlayerLevel[attacker]) 

        if(get_pcvar_num(SaveXP) == 1)  
        {  
            SaveData(attacker)  
        } 
    }  
    SaveData(attacker) 
} 


[wEEk]

Hi, thanks I tried it but instead of that gain experience gives me two bone frags, I duplicated the experience

[EpicMonkey]

PHP Code:

if(victim == attacker)
        return PLUGIN_HANDLED; 


to

PHP Code:

if(attacker == 0 || attacker == victim)
{
        return PLUGIN_HANDLED;
} 


[didoWEE]

Originally Posted by EpicMonkey PHP Code: if(attacker == 0 || attacker == victim)         return PLUGIN_HANDLED;

-->

PHP Code:

if(!attacker || attacker == victim)
        return PLUGIN_HANDLED; 


[hornet]

@wEEK & @Blizzard_87 you cannot use get_user_weapon() with grenades in this case ... that function returns the weapon that the player is holding ... meaning that if the player was killed by a grenade, the attacker would not still be holding one.
You should use DeathMsg instead, and read data 4, or use client_death() with CSX module which gives you the CSW_* constant.
The rest of your code could also be simplified.

Originally Posted by didoWEE --> PHP Code: if(!attacker || attacker == victim)         return PLUGIN_HANDLED;

No difference.

[Blizzard_87]

Originally Posted by hornet @wEEK & @Blizzard_87 you cannot use get_user_weapon() with grenades in this case ... that function returns the weapon that the player is holding ... meaning that if the player was killed by a grenade, the attacker would not still be holding one. You should use DeathMsg instead, and read data 4, or use client_death() with CSX module which gives you the CSW_* constant. The rest of your code could also be simplified. No difference.

i was only going off what code he had.... without showing anymore i didnt know what function he was using...

i only added the if(victim == attacker) check.. rest of code was his.... i know the get_user_weapon wouldnt work but that was not the question he asked.