[Mini_Midget]

this was requested long ago but no one made it

eXtra-Kills
Author: Mini_Midget
Version: 1.3
Modules: Fun
Description:
For every headshot kill you get, your frags will increase by 2 (depending on cvar) but for every Team-Kill headshot, it will decrease by 2 (depending on cvar)

Commands:
amx_hs-kills "2"

Changelog:
Version 1.0
First release
Version 1.1
Less brackets and more readable (Greenberet)
Version 1.2
SweatyBanana helped
Small clean up
Version 1.3
Helped from GHW_Chronic
Made the cvar more better to understand
Deleted AMXX studio notes

[Deviance]

Dont see anything wrong whit this, gj

[Mini_Midget]

Originally Posted by Doombringer Dont see anything wrong whit this, gj

thxs man, first plugin too! (even though this took only 2hours, i was watching t.v at the same time...)

[Thriller]

Fun.^^

[k007]

not bad gj

[Greenberet]

i would change your deathmsg function to this

public deathmsg(id)
{
    new id = read_data(1)
    new victim = read_data(2)
    new headshot = read_data(3)
    
    new frags = get_user_frags(id)
    
    if (!is_user_alive(id) || !headshot )
        return PLUGIN_CONTINUE
    
    if (get_user_team(id) == get_user_team(victim))
        set_user_frags(id, frags - get_pcvar_num(hs_kills))
    else
        set_user_frags(id, frags + get_pcvar_num(hs_kills))
        
        
    return PLUGIN_CONTINUE
}

[SweatyBanana]

Why not:

PHP Code:

     if(!headshot)
        return PLUGIN_CONTINUE 


You cant get a headshot after ur dead can u?


Or do nades count as headshots?

[Mini_Midget]

not to sure if nade kills are headshot kills but pretty sure they aren't
anywho, i'm gonna upload the new version, nothing much changed but i'm just using Greenberet's method cause its a bit more easier to read has less brackets

EDIT
Version 1.1 up
any ideas would be nice

[SweatyBanana]

You should only call things once in the same function...

PHP Code:

public deathmsg(id)
{
    new id = read_data(1)
    new victim = read_data(2)
    new headshot = read_data(3)

    new frags = get_user_frags(id)
    new hsKills = get_pcvar_num(hs_kills)

    if (!is_user_alive(id) || !headshot )
        return PLUGIN_CONTINUE

    if (get_user_team(id) == get_user_team(victim))
        set_user_frags(id, frags - hsKills)
    else
        set_user_frags(id, frags + hsKills)
    return PLUGIN_CONTINUE
} 


[Mini_Midget]

Originally Posted by SweatyBanana You should only call things once in the same function... PHP Code: public deathmsg(id) {     new id = read_data(1)     new victim = read_data(2)     new headshot = read_data(3)     new frags = get_user_frags(id)     new hsKills = get_pcvar_num(hs_kills)     if (!is_user_alive(id) || !headshot )         return PLUGIN_CONTINUE     if (get_user_team(id) == get_user_team(victim))         set_user_frags(id, frags - hsKills)     else         set_user_frags(id, frags + hsKills)     return PLUGIN_CONTINUE }

hmm, didn't know that,
makes the script abit cleaner too i guess
thxs, uploaded new version