[Pottus]

I was taking a look into it and it looks like it is coded pretty good here is one thing I picked up on reading through.

Код:

	new index = (gMenu[menuid][M_LISTITEMS] + 1);
    if(index > MAX_MENU_ITEMS) return -1;
    index -= 1;

Why not just do it like this.

Код:

	new index = (gMenu[menuid][M_LISTITEMS]);
    if(index >= MAX_MENU_ITEMS) return -1;

The ordering in OnPlayerUpdate() could be done better for sure.

pawn Код:

new keys, updown, leftright;
    GetPlayerKeys(playerid, keys, updown, leftright);

    if(updown == KEY_DOWN)
    {
        if(GetPlayerMenuID(playerid) != INVALID_MENU_ID)
        {

You are already making function calls when you don't have to, this would be a better logical order.

Код:

    if(GetPlayerMenuID(playerid) != INVALID_MENU_ID)
	{
		new keys, updown, leftright;
		GetPlayerKeys(playerid, keys, updown, leftright);

		if(updown == KEY_DOWN)
		{

Another thing to keep in mind is you are using == to check for keys which will work but you can not be pressing any other keys! Usually keys are checked like below. I won't say this method as long as it was your intention for it to be that way.

Код:

if(updown & KEY_DOWN)

What I really need to do myself is polish off my GUI system that is used in Texture Studio the idea is to actually build an GUI editor using it's own GUI includes this way the system is developed on top of it's self for proof of concept.
Here - https://github.com/Pottus/Texture-St...ts/tstudio/gui