[H.RED.ZONE]

How can i set on menu that after option is used he will remember and use that command every round.It something like choosing a class?

[ot_207]

Another variable? And you don't open the menu again

[H.RED.ZONE]

Yep until you open the menu and choose option again class will be the same as you choose it first time.

[hleV]

new Class = -1;

public OnMenu(client, menu, item)
{
    if (item != MENU_EXIT)
        Class = item;
    
    menu_destroy(menu);
    
    return PLUGIN_HANDLED;
}

Simple as that.

[ot_207]

Originally Posted by H.RED.ZONE Yep until you open the menu and choose option again class will be the same as you choose it first time.

I told you that you don't show it to the player again.

[H.RED.ZONE]

Tired can't do it can you show me?

[ot_207]

I will not look at such a code.
Please create bigger functions with more functionality. Use switch case. You are basically repeating a code that could have been written just once.

Ex:

PHP Code:

        case 3: 
        {
            if(zp_get_user_zombie(id) || zp_get_user_survivor(id) || zp_get_user_nemesis(id)) return PLUGIN_HANDLED;
            //g_player_class[id] = 3
            got_class[id] = true
            class_3(id)
        }
        case 4: 
        {
            if(zp_get_user_zombie(id) || zp_get_user_survivor(id) || zp_get_user_nemesis(id)) return PLUGIN_HANDLED;
            //g_player_class[id] = 4
            got_class[id] = true
            class_4(id)
    
        }
        case 5: 
        {
            if(zp_get_user_zombie(id) || zp_get_user_survivor(id) || zp_get_user_nemesis(id)) return PLUGIN_HANDLED;
            //g_player_class[id] = 5
            got_class[id] = true
            class_5(id)
        }
        case 6: 
        {
            if(zp_get_user_zombie(id) || zp_get_user_survivor(id) || zp_get_user_nemesis(id)) return PLUGIN_HANDLED;
            //g_player_class[id] = 6
            got_class[id] = true
            class_6(id)
        }
        case 7: 
        {
            if(zp_get_user_zombie(id) || zp_get_user_survivor(id) || zp_get_user_nemesis(id)) return PLUGIN_HANDLED;
            //g_player_class[id] = 7
            got_class[id] = true
            class_7(id)
        }
        case 8: 
        {
            if(zp_get_user_zombie(id) || zp_get_user_survivor(id) || zp_get_user_nemesis(id)) return PLUGIN_HANDLED;
            //g_player_class[id] = 8
            got_class[id] = true
            class_8(id)
        }
        case 9: 
        {
            if(zp_get_user_zombie(id) || zp_get_user_survivor(id) || zp_get_user_nemesis(id)) return PLUGIN_HANDLED;
            //g_player_class[id] = 9
            got_class[id] = true
            class_9(id) 


TO:

PHP Code:

if(zp_get_user_zombie(id) || zp_get_user_survivor(id) || zp_get_user_nemesis(id)) return PLUGIN_HANDLED;
g_player_class[id] = key
got_class[id] = true
class(id, key) //class(id, class_number) 


Create only one class function that give the properties to the user using case, switch so that the code will be easier to debug.

[H.RED.ZONE]

Originally Posted by ot_207 I will not look at such a code. Please create bigger functions with more functionality. Use switch case. You are basically repeating a code that could have been written just once.

Now i get why i couldn't do it. Thanks i need to rewrite it full (:

[ot_207]

Originally Posted by H.RED.ZONE Now i get why i couldn't do it. Thanks i need to rewrite it full (:

Ok, after you will rewrite it, if you still have problems, please post the code here so we can help.