[Podarok]

So I have got a menu and prices for it and in handler i remove amout of money required to buy item, but console is showing me an error : Index is out of bounds, how can I cope with it ?

public shopmenu_handler( id, menu, item )  { 
    if( item == MENU_EXIT )  { 
        menu_destroy( menu ); 
        return PLUGIN_HANDLED; 
    } 
    new money = cs_get_user_money(id)
    cs_set_user_money(id, money-str_to_num(g_Cost[item]))            // <----- This Line

    static const g_Cost[][] = {
    { "10000" },    
    { "13000" },    
    { "16000" },    
    { "7500" },     
    { "7000" },     
    { "12000" },    
    { "7500" },     
    { "15000" },    
    { "16000" } 
}

[~Ice*shOt]

how much items are?

[YamiKaitou]

What is the value of item? IIRC, the New Menu style does not use a Zero based item scale, so you would need to cope with that.

[Backstabnoob]

Yami is right, use str_to_num( g_Cost[ item - 1 ] ).

[Podarok]

Originally Posted by YamiKaitou What is the value of item? IIRC, the New Menu style does not use a Zero based item scale, so you would need to cope with that.

If I press 0 - aka Exit, it shows me this error...

Originally Posted by Backstabnoob Yami is right, use str_to_num( g_Cost[ item - 1 ] ).

Also this is not right.

[guipatinador]

Originally Posted by Backstabnoob Yami is right, use str_to_num( g_Cost[ item - 1 ] ).

No.
If I press key one, item = 0.
g_Cost[ 0 ] = 10000

[Podarok]

Originally Posted by guipatinador No. If I press key one, item = 0. g_Cost[ 0 ] = 10000

Yes thats what I am talking about. This error occurs when u press 0. Aka Exit!

[YamiKaitou]

Originally Posted by Podarok Yes thats what I am talking about. This error occurs when u press 0. Aka Exit!

No it doesn't. You have caught the MENU_EXIT. If you believe it is the other way, then DEBUG it

[Podarok]

Originally Posted by YamiKaitou No it doesn't. You have caught the MENU_EXIT. If you believe it is the other way, then DEBUG it

I opened 2 windows, CS and My Remote Connection to the server i pressed every key on the menu and only when I pressed 0 aka Exit this error occured.. I was also wondered why because as u said i have caught the MENU_EXIT ? Maybe I should do it like this : if(item == -1)... ?

[YamiKaitou]

This is the only way to know what the true value of item is

public shopmenu_handler( id, menu, item )  { 
server_print("item == %d", item);
    if( item == MENU_EXIT )  { 
        menu_destroy( menu ); 
        return PLUGIN_HANDLED; 
    } 
server_print("item was not MENU_EXIT");
    new money = cs_get_user_money(id)
    cs_set_user_money(id, money-str_to_num(g_Cost[item]))            // <----- This Line