[Simple32]

Full Pack Grenades
Version 1.0.9
By Simple32

Description:
You receive grenades through Ham_Spawn each round. You can use Bit flags to put what grenades you want for e.g if you just want an HE Grenade you put the flag "a". There are three options of what grenades you want to put, they're "abc".

A = HE Grenade
B = Flashbang
C = Smoke Grenade

Cvars:
mode_grenades "abc" [Default]"

Credits:
Black Rose - Made Bit flags work and added extra codes.

Modules:
Hamsandwich
Fun

Changelog:
Version 1.0.3 - Initial Release.
Version 1.0.4 - Made the variable in one whole line.
Version 1.0.5 - Removed #defines and optimized the code.
Version 1.0.6 - Waste of if user is alive code and optimized it to once only.
Version 1.0.7 - Bit flags are added and cvar all in one.
Version 1.0.8 - Added client_print, to see what grenades you received.
Version 1.0.9 - Removed server_print from some places and added colorchat.

[wickedd]

PHP Code:

#include <hamsandwich>
#include <fun>

new gCvarHEGrenade, gCvarFlashbang, gCvarSmokeGrenade;

#define PLUGIN "Full Pack Grenades"
#define VERSION "1.0.4"
#define AUTHOR "Simple32" 

public plugin_init() {
    register_plugin( "Full Pack Grenades", "1.0.4", "Simple32" );
    
    gCvarHEGrenade = register_cvar( "mode_hegrenade", "1" );
    gCvarFlashbang = register_cvar( "mode_flashbang", "1" );
    gCvarSmokeGrenade = register_cvar( "mode_smokegrenade", "1" );
    
    RegisterHam( Ham_Spawn, "player", "fwdPlayerSpawn", 1 );
}

public fwdPlayerSpawn( id ) {
    if(get_pcvar_num( gCvarHEGrenade ))
    {
        if( is_user_alive( id ) ) {
            give_item(id, "weapon_hegrenade");
        }
        if(get_pcvar_num( gCvarFlashbang ))
        {
            if( is_user_alive( id ) ) {
                give_item(id, "weapon_flashbang");
            }
            if(get_pcvar_num( gCvarSmokeGrenade ))
            {
                if( is_user_alive( id ) ) {
                    give_item(id, "weapon_smokegrenade");
                }
            }
        }
    }
}
/* AMXX-Studio Notes - DO NOT MODIFY BELOW HERE
*{\\ rtf1\\ ansi\\ deff0{\\ fonttbl{\\ f0\\ fnil Tahoma;}}\n\\ viewkind4\\ uc1\\ pard\\ lang3081\\ f0\\ fs16 \n\\ par }
*/ 


Poorly coded and redundant.

[WildCard65]

From what I first saw:

PHP Code:

//Redacted Code
#define PLUGIN "Full Pack Grenades"
#define VERSION "1.0.4"
#define AUTHOR "Simple32" 

public plugin_init() {
    register_plugin( "Full Pack Grenades", "1.0.4", "Simple32" );
   //Redacted code
}
//Redacted Code 


Waste of #defines

PHP Code:

//Redacted Code
#define PLUGIN "Full Pack Grenades"
#define VERSION "1.0.4"
#define AUTHOR "Simple32" 

public plugin_init() {
    register_plugin( PLUGIN, VERSION, AUTHOR );
   //Redacted code
}
//Redacted Code 


[Simple32]

Wickedd, since you said it is poorly coded can you please give me suggestions I could fix it and make it look more efficient.

[Kia]

1. Your description is wrong. The plugin does not disable grenades, only gives them if CVar is 1.
2. You only need to check if the player is alive once.
3. Plugin is redundant.

[YamiKaitou]

4. Change the 3 cvars into 1 and use Bit flags to enable/disable each type

[Simple32]

Do you mean as in use flags for each grenades so you can disable and enable which ones you want?

[Blizzard_87]

Originally Posted by YamiKaitou 4. Change the 3 cvars into 1 and use Bit flags to enable/disable each type

Originally Posted by Simple32 Do you mean as in use flags for each grenades so you can disable and enable which ones you want?

No.

[Simple32]

Changelog:
Version 1.0.7 - Added Bit flags and cvar all in one.

[YamiKaitou]

Your description still incorrectly states what the plugin does