[BOYSplayCS]

Admin Slaughter Reward v1.0
BOYSplayCS

Description:
When you kill an admin with a knife you are rewarded extra money, depending on a cvar. Also, the admin you kill must have the ADMIN_KICK flag or there will be no effect.


Cvars:
amx_adminkill_reward "300" - Sets amount of extra money you earn.


Modules:
Cstrike
Fun


Notice:
I have no idea if this has already been made, I was just having fun with this. Please post your feedback! I tried doing this plugin as efficient and easy as possible. If you are going to offer a code optimization, please exclude hamsandwich. I don't like working with that module.

[alan_el_more]

good plugin and idea

[Starsailor]

why have you included fun module ?

[BOYSplayCS]

I accidentally left that in there for another plugin I was creating.

[Anggara_nothing]

I have question.
What is different between ADMIN_KICK and ADMIN_BAN?
btw, good plugin.

[danielkza]

Curiosity, why do you hate ham?

[BOYSplayCS]

@Anggara: There isn't much of a difference, I believe they are the same flags. And thank you

@danielkza: Ham just doesn't seem that easy for me to work with, it uses functions that I don't understand and that have given me tons of issues.

[Exolent[jNr]]

Here we go.

1. You can rearrange your death function to only create variables when they are needed.

2. No need to have the weapon array that large when there are no other weapons that have "knife" in the name.

3. There is no need for the variable to hold the players money when it is only used once.

4. You should make sure that the money doesn't exceed the limits.

5. You didn't add a check for team killing.

public Event_Death()
{
    new Killer = read_data(1)
    new Victim = read_data(2)
    
    if ((Killer == Victim) || !(get_user_flags(Victim) & AdminKilled) || (get_user_team(Victim) == get_user_team(Killer)))
        return PLUGIN_CONTINUE
    
    new CheckWeapon[6];
    read_data(4, CheckWeapon, 5);
    
    if (equal(CheckWeapon, "knife"))
    {
        new KillReward = get_pcvar_num(Reward)
        
        cs_set_user_money(Killer, clamp(cs_get_user_money(Killer) + KillReward, 0, 16000))
        client_print(Killer, print_chat, "[AMXX] You have slaughtered an admin! You have been awarded an extra %s dollars!", KillReward)
    }
    return PLUGIN_HANDLED
}

[BOYSplayCS]

@Exolent: If I kept my weapon array at 32, would it make a difference? I'll change it, but I'm just wondering. Also, thank you for your optimization.

[danielkza]

Originally Posted by BOYSplayCS @Anggara: There isn't much of a difference, I believe they are the same flags. And thank you @danielkza: Ham just doesn't seem that easy for me to work with, it uses functions that I don't understand and that have given me tons of issues.

Did you try looking up hamconst.inc? It has descriptions for most of the functions.

And if it creates you problems there's no reason to use it, but I personally think it's very helpful, using 'proper' hooks instead of workarounds, allowing modification of parameters for hooks, etc.