Грамотное использование switch и if

Когда-то давно я сделал систему инвентаря, но не очень оптимизированную - вся она строилась на if проверках
Вспомнив про switch и его превосходство над if, я решил переделать мою систему инвентаря под switch, старавшись грамотно его использовать
Нормально ли выглядит код? (P.S. я знаю, что переменную инвентаря можно сделать двухмерной и записывать в каждую меру (или как это назвать?)

new aaa[4];
aaa[0] = 1; // мера [0], значение в мере 0 - 1
aaa[1] = 2; // мера [1], значение 2
aaa[2] = 3; // etc
aaa[3] = 4;

значения слота, но как записывать такую конструкцию в бд? 3 ячейками на вид предмета и 3 ячейками на запас? сомневаюсь)

Сама система:

  Открыть/закрыть

case 12: // Инвентарь
        {
            if(response)
            {
                switch(listitem)
                {
                    case 0:
                    {
                        switch(player[playerid][pItem1])
                        {
                            case 0:
                            {
                                SPD(playerid, 9365, DSM, !"{00C0FF}Ошибка",
                                !"\
                                    {00C0FF}Данный слот пуст!", !"{00C0FF}Понятно", !"\
                                ");
                                return true;
                            }
                            default:
                            {
                                player[playerid][pItemSelect] = 1;
                                 static
                                    fmt_str[] = "{00C0FF}%s";
                                new
                                    string[sizeof(fmt_str)-2+20+8+1];
                                format(string, sizeof(string), fmt_str, invent_items[player[playerid][pItem1]]);
 
                                SPD(playerid, 13, DSL, string,
                                    !"\
                                        {00C0FF}Использовать\n{00C0FF}Выбросить", !"{00C0FF}Далее", !"{00C0FF}Назад\
                                    ");
                            }
                        }
                    }
                    case 1:
                    {
                        switch(player[playerid][pItem2])
                        {
                            case 0:
                            {
                                SPD(playerid, 9365, DSM, !"{00C0FF}Ошибка",
                                !"\
                                    {00C0FF}Данный слот пуст!", !"{00C0FF}Понятно", !"\
                                ");
                                return true;
                            }
                            default:
                            {
                                player[playerid][pItemSelect] = 2;
                                 static
                                    fmt_str[] = "{00C0FF}%s";
                                new
                                    string[sizeof(fmt_str)-2+20+8+1];
                                format(string, sizeof(string), fmt_str, invent_items[player[playerid][pItem2]]);
 
                                SPD(playerid, 13, DSL, string,
                                    !"\
                                        {00C0FF}Использовать\n{00C0FF}Выбросить", !"{00C0FF}Далее", !"{00C0FF}Назад\
                                    ");
                            }
                        }
                    }
                    case 2:
                    {
                        switch(player[playerid][pItem3])
                        {
                            case 0:
                            {
                                SPD(playerid, 9365, DSM, !"{00C0FF}Ошибка",
                                !"\
                                    {00C0FF}Данный слот пуст!", !"{00C0FF}Понятно", !"\
                                ");
                                return true;
                            }
                            default:
                            {
                                player[playerid][pItemSelect] = 3;
                                 static
                                    fmt_str[] = "{00C0FF}%s";
                                new
                                    string[sizeof(fmt_str)-2+20+8+1];
                                format(string, sizeof(string), fmt_str, invent_items[player[playerid][pItem3]]);
 
                                SPD(playerid, 13, DSL, string,
                                    !"\
                                        {00C0FF}Использовать\n{00C0FF}Выбросить", !"{00C0FF}Далее", !"{00C0FF}Назад\
                                    ");
                            }
                        }
                    }
                }
            }
        }
        case 13:
        {
            if(response)
            {
                switch(listitem)
                {
                    case 0:
                    {
                        switch(player[playerid][pItemSelect])
                        {
                            case 1:
                            {
                                switch(player[playerid][pItem1])
                                {
                                    case 1:
                                    {
                                        new Float:health_eat;
                                        GetPlayerHealth(playerid, health_eat);
                                        if(health_eat >= 100.0)
                                            SCM(playerid, COLOR_YELLOW, !"Вы сыты");
                                        else
                                           {
                                               SCM(playerid, COLOR_YELLOW, !"Вы съели бургер, ваше здоровье пополнилось на 20 единиц");
                                               SetPlayerHealth(playerid, health_eat+20.0);
                                               if(player[playerid][pItemAmmo1] >= 2)
                                               {
                                                   player[playerid][pItemAmmo1]--;
                                                SavePlayer(playerid, "ItemAmmo1", player[playerid][pItemAmmo1], "d");
                                               }
                                               else
                                               {
                                                   player[playerid][pItem1] = 0;
                                                player[playerid][pItemAmmo1] = 0;
                                                SavePlayer(playerid, "Item1", player[playerid][pItem1], "d");
                                                SavePlayer(playerid, "ItemAmmo1", player[playerid][pItemAmmo1], "d");
                                            }
                                            static
                                                fmt_str[] = "*%s съел бургер";
                                            new
                                                string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                            format(string, sizeof(string), fmt_str, GetName(playerid));
                                            ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                        }
                                        return true;
                                    }
                                    case 2:
                                    {
                                        new Float:health_drink;
                                        GetPlayerHealth(playerid, health_drink);
                                        if(health_drink >= 100.0)
                                            SCM(playerid, COLOR_YELLOW, !"Вы не имеете жажды");
                                        else
                                           {
                                            SCM(playerid, COLOR_YELLOW, !"Вы выпили Пепси-Колу, ваше здоровье пополнилось на 10 единиц");
                                            SetPlayerHealth(playerid, health_drink+10.0);
                                            if(player[playerid][pItemAmmo1] >= 2)
                                               {
                                                   player[playerid][pItemAmmo1]--;
                                                SavePlayer(playerid, "ItemAmmo1", player[playerid][pItemAmmo1], "d");
                                               }
                                               else
                                               {
                                                   player[playerid][pItem1] = 0;
                                                player[playerid][pItemAmmo1] = 0;
                                                SavePlayer(playerid, "Item1", player[playerid][pItem1], "d");
                                                SavePlayer(playerid, "ItemAmmo1", player[playerid][pItemAmmo1], "d");
                                            }
                                            static
                                                fmt_str[] = "*%s выпил Пепси-Колу";
                                            new
                                                string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                            format(string, sizeof(string), fmt_str, GetName(playerid));
                                            ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                        }
                                        return true;
                                    }
                                    case 7:
                                    {
                                        GivePlayerWeapon(playerid, 5, 1);
                                           player[playerid][pItem1] = 0;
                                         player[playerid][pItemAmmo1] = 0;
                                         SavePlayer(playerid, "Item1", player[playerid][pItem1], "d");
                                        SavePlayer(playerid, "ItemAmmo1", player[playerid][pItemAmmo1], "d");
                                        static
                                            fmt_str[] = "*%s достал из-за пазухи биту";
                                        new
                                            string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                        format(string, sizeof(string), fmt_str, GetName(playerid));
                                        ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                        return true;
                                    }
                                    default:
                                    {
                                           DialogTakePatrons(playerid);
                                        return true;
                                    }
                                }
                            }
                            case 2:
                            {
                                switch(player[playerid][pItem2])
                                {
                                    case 1:
                                    {
                                        new Float:health_eat;
                                        GetPlayerHealth(playerid, health_eat);
                                        if(health_eat >= 100.0)
                                            SCM(playerid, COLOR_YELLOW, !"Вы сыты");
                                        else
                                           {
                                               SCM(playerid, COLOR_YELLOW, !"Вы съели бургер, ваше здоровье пополнилось на 20 единиц");
                                               SetPlayerHealth(playerid, health_eat+20.0);
                                               if(player[playerid][pItemAmmo2] >= 2)
                                               {
                                                   player[playerid][pItemAmmo2]--;
                                                SavePlayer(playerid, "ItemAmmo2", player[playerid][pItemAmmo2], "d");
                                               }
                                               else
                                               {
                                                   player[playerid][pItem2] = 0;
                                                player[playerid][pItemAmmo2] = 0;
                                                SavePlayer(playerid, "Item2", player[playerid][pItem2], "d");
                                                SavePlayer(playerid, "ItemAmmo2", player[playerid][pItemAmmo2], "d");
                                            }
                                            static
                                                fmt_str[] = "*%s съел бургер";
                                            new
                                                string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                            format(string, sizeof(string), fmt_str, GetName(playerid));
                                            ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                        }
                                        return true;
                                    }
                                    case 2:
                                    {
                                        new Float:health_drink;
                                        GetPlayerHealth(playerid, health_drink);
                                        if(health_drink >= 100.0)
                                            SCM(playerid, COLOR_YELLOW, !"Вы не имеете жажды");
                                        else
                                           {
                                            SCM(playerid, COLOR_YELLOW, !"Вы выпили Пепси-Колу, ваше здоровье пополнилось на 10 единиц");
                                            SetPlayerHealth(playerid, health_drink+10.0);
                                            if(player[playerid][pItemAmmo2] >= 2)
                                               {
                                                   player[playerid][pItemAmmo2]--;
                                                SavePlayer(playerid, "ItemAmmo2", player[playerid][pItemAmmo2], "d");
                                               }
                                               else
                                               {
                                                   player[playerid][pItem2] = 0;
                                                player[playerid][pItemAmmo2] = 0;
                                                SavePlayer(playerid, "Item2", player[playerid][pItem2], "d");
                                                SavePlayer(playerid, "ItemAmmo2", player[playerid][pItemAmmo2], "d");
                                            }
                                            static
                                                fmt_str[] = "*%s выпил Пепси-Колу";
                                            new
                                                string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                            format(string, sizeof(string), fmt_str, GetName(playerid));
                                            ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                        }
                                        return true;
                                    }
                                    case 7:
                                    {
                                        GivePlayerWeapon(playerid, 5, 1);
                                           player[playerid][pItem2] = 0;
                                         player[playerid][pItemAmmo2] = 0;
                                         SavePlayer(playerid, "Item2", player[playerid][pItem2], "d");
                                        SavePlayer(playerid, "ItemAmmo2", player[playerid][pItemAmmo2], "d");
                                        static
                                            fmt_str[] = "*%s достал из-за пазухи биту";
                                        new
                                            string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                        format(string, sizeof(string), fmt_str, GetName(playerid));
                                        ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                        return true;
                                    }
                                    default:
                                    {
                                           DialogTakePatrons(playerid);
                                        return true;
                                    }
                                }
                            }
                            case 3:
                            {
                                switch(player[playerid][pItem3])
                                {
                                    case 1:
                                    {
                                        new Float:health_eat;
                                        GetPlayerHealth(playerid, health_eat);
                                        if(health_eat >= 100.0)
                                            SCM(playerid, COLOR_YELLOW, !"Вы сыты");
                                        else
                                           {
                                               SCM(playerid, COLOR_YELLOW, !"Вы съели бургер, ваше здоровье пополнилось на 20 единиц");
                                               SetPlayerHealth(playerid, health_eat+20.0);
                                               if(player[playerid][pItemAmmo3] >= 2)
                                               {
                                                   player[playerid][pItemAmmo3]--;
                                                SavePlayer(playerid, "ItemAmmo3", player[playerid][pItemAmmo3], "d");
                                               }
                                               else
                                               {
                                                   player[playerid][pItem3] = 0;
                                                player[playerid][pItemAmmo3] = 0;
                                                SavePlayer(playerid, "Item3", player[playerid][pItem3], "d");
                                                SavePlayer(playerid, "ItemAmmo3", player[playerid][pItemAmmo3], "d");
                                            }
                                            static
                                                fmt_str[] = "*%s съел бургер";
                                            new
                                                string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                            format(string, sizeof(string), fmt_str, GetName(playerid));
                                            ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                        }
                                        return true;
                                    }
                                    case 2:
                                    {
                                        new Float:health_drink;
                                        GetPlayerHealth(playerid, health_drink);
                                        if(health_drink >= 100.0)
                                            SCM(playerid, COLOR_YELLOW, !"Вы не имеете жажды");
                                        else
                                           {
                                            SCM(playerid, COLOR_YELLOW, !"Вы выпили Пепси-Колу, ваше здоровье пополнилось на 10 единиц");
                                            SetPlayerHealth(playerid, health_drink+10.0);
                                            if(player[playerid][pItemAmmo3] >= 2)
                                               {
                                                   player[playerid][pItemAmmo3]--;
                                                SavePlayer(playerid, "ItemAmmo3", player[playerid][pItemAmmo3], "d");
                                               }
                                               else
                                               {
                                                   player[playerid][pItem3] = 0;
                                                player[playerid][pItemAmmo3] = 0;
                                                SavePlayer(playerid, "Item3", player[playerid][pItem3], "d");
                                                SavePlayer(playerid, "ItemAmmo3", player[playerid][pItemAmmo3], "d");
                                            }
                                            static
                                                fmt_str[] = "*%s выпил Пепси-Колу";
                                            new
                                                string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                            format(string, sizeof(string), fmt_str, GetName(playerid));
                                            ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                        }
                                        return true;
                                    }
                                    case 7:
                                    {
                                        GivePlayerWeapon(playerid, 5, 1);
                                           player[playerid][pItem3] = 0;
                                         player[playerid][pItemAmmo3] = 0;
                                         SavePlayer(playerid, "Item3", player[playerid][pItem3], "d");
                                        SavePlayer(playerid, "ItemAmmo3", player[playerid][pItemAmmo3], "d");
                                        static
                                            fmt_str[] = "*%s достал из-за пазухи биту";
                                        new
                                            string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                        format(string, sizeof(string), fmt_str, GetName(playerid));
                                        ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                        return true;
                                    }
                                    default:
                                    {
                                        DialogTakePatrons(playerid);
                                        return true;
                                    }
                                }
                            }
                        }
                    }
                    case 1:
                    {
                        SPD(playerid, 14, DSM, !"{00C0FF}Внимание",
                        !"{00C0FF}Вы действительно хотите выбросить предмет/оружие из инвентаря? Оно будет потеряно навсегда!",
                        !"{00C0FF}Да", !"{00C0FF}Нет");
                        return true;
                    }
                }
            }
            else
            {
                callcmd::invent(playerid);
                return true;
            }
        }

UPD: Нашел кучу логических ошибок, видимо, когда я писал этот код, я что-то употреблял

Вот код:

  Открыть/закрыть

case 12: // Инвентарь
        {
            if(response)
            {
                switch(listitem)
                {
                    case 0:
                    {
                        switch(player[playerid][pItem1])
                        {
                            case 0:
                            {
                                SPD(playerid, 9365, DSM, !"{00C0FF}Ошибка",
                                !"\
                                    {00C0FF}Данный слот пуст!", !"{00C0FF}Понятно", !"\
                                ");
                                return true;
                            }
                            default:
                            {
                                player[playerid][pItemSelect] = 1;
                                 static
                                    fmt_str[] = "{00C0FF}%s";
                                new
                                    string[sizeof(fmt_str)-2+20+8+1];
                                format(string, sizeof(string), fmt_str, invent_items[player[playerid][pItem1]]);
 
                                SPD(playerid, 13, DSL, string,
                                    !"\
                                        {00C0FF}Использовать\n{00C0FF}Выбросить", !"{00C0FF}Далее", !"{00C0FF}Назад\
                                    ");
                            }
                        }
                    }
                    case 1:
                    {
                        switch(player[playerid][pItem2])
                        {
                            case 0:
                            {
                                SPD(playerid, 9365, DSM, !"{00C0FF}Ошибка",
                                !"\
                                    {00C0FF}Данный слот пуст!", !"{00C0FF}Понятно", !"\
                                ");
                                return true;
                            }
                            default:
                            {
                                player[playerid][pItemSelect] = 2;
                                 static
                                    fmt_str[] = "{00C0FF}%s";
                                new
                                    string[sizeof(fmt_str)-2+20+8+1];
                                format(string, sizeof(string), fmt_str, invent_items[player[playerid][pItem2]]);
 
                                SPD(playerid, 13, DSL, string,
                                    !"\
                                        {00C0FF}Использовать\n{00C0FF}Выбросить", !"{00C0FF}Далее", !"{00C0FF}Назад\
                                    ");
                            }
                        }
                    }
                    case 2:
                    {
                        switch(player[playerid][pItem3])
                        {
                            case 0:
                            {
                                SPD(playerid, 9365, DSM, !"{00C0FF}Ошибка",
                                !"\
                                    {00C0FF}Данный слот пуст!", !"{00C0FF}Понятно", !"\
                                ");
                                return true;
                            }
                            default:
                            {
                                player[playerid][pItemSelect] = 3;
                                 static
                                    fmt_str[] = "{00C0FF}%s";
                                new
                                    string[sizeof(fmt_str)-2+20+8+1];
                                format(string, sizeof(string), fmt_str, invent_items[player[playerid][pItem3]]);
 
                                SPD(playerid, 13, DSL, string,
                                    !"\
                                        {00C0FF}Использовать\n{00C0FF}Выбросить", !"{00C0FF}Далее", !"{00C0FF}Назад\
                                    ");
                            }
                        }
                    }
                }
            }
        }
        case 13:
        {
            if(response)
            {
                switch(listitem)
                {
                    case 0:
                    {
                        switch(player[playerid][pItemSelect])
                        {
                            case 1:
                            {
                                switch(player[playerid][pItem1])
                                {
                                    case 1:
                                    {
                                        new Float:health_eat;
                                        GetPlayerHealth(playerid, health_eat);
                                        if(health_eat >= 100.0)
                                            SCM(playerid, COLOR_YELLOW, !"Вы сыты");
                                        else
                                           {
                                               SCM(playerid, COLOR_YELLOW, !"Вы съели бургер, ваше здоровье пополнилось на 20 единиц");
                                               SetPlayerHealth(playerid, health_eat+20.0);
                                               if(player[playerid][pItemAmmo1] >= 2)
                                                   player[playerid][pItemAmmo1]--;
                                               else
                                               {
                                                   player[playerid][pItem1] = 0;
                                                player[playerid][pItemAmmo1] = 0;
                                                SavePlayer(playerid, "Item1", player[playerid][pItem1], "d");
                                            }
                                            SavePlayer(playerid, "ItemAmmo1", player[playerid][pItemAmmo1], "d");
                                            static
                                                fmt_str[] = "*%s съел бургер";
                                            new
                                                string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                            format(string, sizeof(string), fmt_str, GetName(playerid));
                                            ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                        }
                                        return true;
                                    }
                                    case 2:
                                    {
                                        new Float:health_drink;
                                        GetPlayerHealth(playerid, health_drink);
                                        if(health_drink >= 100.0)
                                            SCM(playerid, COLOR_YELLOW, !"Вы не имеете жажды");
                                        else
                                           {
                                            SCM(playerid, COLOR_YELLOW, !"Вы выпили Пепси-Колу, ваше здоровье пополнилось на 10 единиц");
                                            SetPlayerHealth(playerid, health_drink+10.0);
                                            if(player[playerid][pItemAmmo1] >= 2)
                                                   player[playerid][pItemAmmo1]--;
                                               else
                                               {
                                                   player[playerid][pItem1] = 0;
                                                player[playerid][pItemAmmo1] = 0;
                                                SavePlayer(playerid, "Item1", player[playerid][pItem1], "d");
                                            }
                                            SavePlayer(playerid, "ItemAmmo1", player[playerid][pItemAmmo1], "d");
                                            static
                                                fmt_str[] = "*%s выпил Пепси-Колу";
                                            new
                                                string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                            format(string, sizeof(string), fmt_str, GetName(playerid));
                                            ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                        }
                                        return true;
                                    }
                                    case 7:
                                    {
                                        GivePlayerWeapon(playerid, 5, 1);
                                        if(player[playerid][pItemAmmo1] >= 2)
                                            player[playerid][pItemAmmo1]--;
                                        else
                                        {
                                            player[playerid][pItem1] = 0;
                                             player[playerid][pItemAmmo1] = 0;
                                             SavePlayer(playerid, "Item1", player[playerid][pItem1], "d");
                                        }
                                        SavePlayer(playerid, "ItemAmmo1", player[playerid][pItemAmmo1], "d");
                                        static
                                            fmt_str[] = "*%s достал из-за пазухи биту";
                                        new
                                            string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                        format(string, sizeof(string), fmt_str, GetName(playerid));
                                        ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                        return true;
                                    }
                                    default:
                                    {
                                           DialogTakePatrons(playerid);
                                        return true;
                                    }
                                }
                            }
                            case 2:
                            {
                                switch(player[playerid][pItem2])
                                {
                                    case 1:
                                    {
                                        new Float:health_eat;
                                        GetPlayerHealth(playerid, health_eat);
                                        if(health_eat >= 100.0)
                                            SCM(playerid, COLOR_YELLOW, !"Вы сыты");
                                        else
                                           {
                                               SCM(playerid, COLOR_YELLOW, !"Вы съели бургер, ваше здоровье пополнилось на 20 единиц");
                                               SetPlayerHealth(playerid, health_eat+20.0);
                                               if(player[playerid][pItemAmmo2] >= 2)
                                                   player[playerid][pItemAmmo2]--;
                                               else
                                               {
                                                   player[playerid][pItem2] = 0;
                                                player[playerid][pItemAmmo2] = 0;
                                                SavePlayer(playerid, "Item2", player[playerid][pItem2], "d");
                                            }
                                            SavePlayer(playerid, "ItemAmmo2", player[playerid][pItemAmmo2], "d");
                                            static
                                                fmt_str[] = "*%s съел бургер";
                                            new
                                                string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                            format(string, sizeof(string), fmt_str, GetName(playerid));
                                            ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                        }
                                        return true;
                                    }
                                    case 2:
                                    {
                                        new Float:health_drink;
                                        GetPlayerHealth(playerid, health_drink);
                                        if(health_drink >= 100.0)
                                            SCM(playerid, COLOR_YELLOW, !"Вы не имеете жажды");
                                        else
                                           {
                                            SCM(playerid, COLOR_YELLOW, !"Вы выпили Пепси-Колу, ваше здоровье пополнилось на 10 единиц");
                                            SetPlayerHealth(playerid, health_drink+10.0);
                                            if(player[playerid][pItemAmmo2] >= 2)
                                                   player[playerid][pItemAmmo2]--;
                                               else
                                               {
                                                   player[playerid][pItem2] = 0;
                                                player[playerid][pItemAmmo2] = 0;
                                                SavePlayer(playerid, "Item2", player[playerid][pItem2], "d");
                                            }
                                            SavePlayer(playerid, "ItemAmmo2", player[playerid][pItemAmmo2], "d");
                                            static
                                                fmt_str[] = "*%s выпил Пепси-Колу";
                                            new
                                                string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                            format(string, sizeof(string), fmt_str, GetName(playerid));
                                            ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                        }
                                        return true;
                                    }
                                    case 7:
                                    {
                                        GivePlayerWeapon(playerid, 5, 1);
                                        if(player[playerid][pItemAmmo2] >= 2)
                                            player[playerid][pItemAmmo2]--;
                                        else
                                        {
                                            player[playerid][pItem2] = 0;
                                             player[playerid][pItemAmmo2] = 0;
                                             SavePlayer(playerid, "Item2", player[playerid][pItem2], "d");
                                        }
                                        SavePlayer(playerid, "ItemAmmo2", player[playerid][pItemAmmo2], "d");
                                        static
                                            fmt_str[] = "*%s достал из-за пазухи биту";
                                        new
                                            string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                        format(string, sizeof(string), fmt_str, GetName(playerid));
                                        ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                        return true;
                                    }
                                    default:
                                    {
                                           DialogTakePatrons(playerid);
                                        return true;
                                    }
                                }
                            }
                            case 3:
                            {
                                switch(player[playerid][pItem3])
                                {
                                    case 1:
                                    {
                                        new Float:health_eat;
                                        GetPlayerHealth(playerid, health_eat);
                                        if(health_eat >= 100.0)
                                            SCM(playerid, COLOR_YELLOW, !"Вы сыты");
                                        else
                                           {
                                               SCM(playerid, COLOR_YELLOW, !"Вы съели бургер, ваше здоровье пополнилось на 20 единиц");
                                               SetPlayerHealth(playerid, health_eat+20.0);
                                               if(player[playerid][pItemAmmo3] >= 2)
                                                   player[playerid][pItemAmmo3]--;
                                               else
                                               {
                                                   player[playerid][pItem3] = 0;
                                                player[playerid][pItemAmmo3] = 0;
                                                SavePlayer(playerid, "Item3", player[playerid][pItem3], "d");
                                            }
                                            SavePlayer(playerid, "ItemAmmo3", player[playerid][pItemAmmo3], "d");
                                            static
                                                fmt_str[] = "*%s съел бургер";
                                            new
                                                string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                            format(string, sizeof(string), fmt_str, GetName(playerid));
                                            ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                        }
                                        return true;
                                    }
                                    case 2:
                                    {
                                        new Float:health_drink;
                                        GetPlayerHealth(playerid, health_drink);
                                        if(health_drink >= 100.0)
                                            SCM(playerid, COLOR_YELLOW, !"Вы не имеете жажды");
                                        else
                                           {
                                            SCM(playerid, COLOR_YELLOW, !"Вы выпили Пепси-Колу, ваше здоровье пополнилось на 10 единиц");
                                            SetPlayerHealth(playerid, health_drink+10.0);
                                            if(player[playerid][pItemAmmo3] >= 2)
                                                   player[playerid][pItemAmmo3]--;
                                               else
                                               {
                                                   player[playerid][pItem3] = 0;
                                                player[playerid][pItemAmmo3] = 0;
                                                SavePlayer(playerid, "Item3", player[playerid][pItem3], "d");
                                            }
                                            SavePlayer(playerid, "ItemAmmo3", player[playerid][pItemAmmo3], "d");
                                            static
                                                fmt_str[] = "*%s выпил Пепси-Колу";
                                            new
                                                string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                            format(string, sizeof(string), fmt_str, GetName(playerid));
                                            ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                        }
                                        return true;
                                    }
                                    case 7:
                                    {
                                        GivePlayerWeapon(playerid, 5, 1);
                                        if(player[playerid][pItemAmmo3] >= 2)
                                            player[playerid][pItemAmmo3]--;
                                        else
                                        {
                                               player[playerid][pItem3] = 0;
                                             player[playerid][pItemAmmo3] = 0;
                                             SavePlayer(playerid, "Item3", player[playerid][pItem3], "d");
                                         }
                                        SavePlayer(playerid, "ItemAmmo3", player[playerid][pItemAmmo3], "d");
                                        static
                                            fmt_str[] = "*%s достал из-за пазухи биту";
                                        new
                                            string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                        format(string, sizeof(string), fmt_str, GetName(playerid));
                                        ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                        return true;
                                    }
                                    default:
                                    {
                                        DialogTakePatrons(playerid);
                                        return true;
                                    }
                                }
                            }
                        }
                    }
                    case 1:
                    {
                        SPD(playerid, 14, DSM, !"{00C0FF}Внимание",
                        !"{00C0FF}Вы действительно хотите выбросить предмет/оружие из инвентаря? Оно будет потеряно навсегда!",
                        !"{00C0FF}Да", !"{00C0FF}Нет");
                        return true;
                    }
                }
            }
            else
            {
                callcmd::invent(playerid);
                return true;
            }
        }
          case 14:
        {
            if(response)
            {
                switch(player[playerid][pItemSelect])
                {
                    case 1:
                    {
                        player[playerid][pItem1] = 0;
                        player[playerid][pItemAmmo1] = 0;
                        SavePlayer(playerid, "Item1", player[playerid][pItem1], "d");
                        SavePlayer(playerid, "ItemAmmo1", player[playerid][pItemAmmo1], "d");
                        callcmd::invent(playerid);
                        return true;
                    }
                    case 2:
                    {
                        player[playerid][pItem2] = 0;
                        player[playerid][pItemAmmo2] = 0;
                        SavePlayer(playerid, "Item2", player[playerid][pItem2], "d");
                        SavePlayer(playerid, "ItemAmmo2", player[playerid][pItemAmmo2], "d");
                        callcmd::invent(playerid);
                        return true;
                    }
                    case 3:
                    {
                        player[playerid][pItem3] = 0;
                        player[playerid][pItemAmmo3] = 0;
                        SavePlayer(playerid, "Item3", player[playerid][pItem3], "d");
                        SavePlayer(playerid, "ItemAmmo3", player[playerid][pItemAmmo3], "d");
                        callcmd::invent(playerid);
                        return true;
                    }
                    default:
                    {
                        SPD(playerid, 9284, DSM, !"{00C0FF}Ошибка", !"{00C0FF}Данный слот пуст!", !"{00C0FF}Понятно", !"");
                        callcmd::invent(playerid);
                        return true;
                    }
 
                }
                return true;
            }
        }
        case 15:
        {
            if(response)
            {
                new
                    len = strlen(inputtext),
                    val = strval(inputtext);
                if(!len)
                {
                    SCM(playerid, COLOR_GREY, !"Вы ничего не ввели!");
                    DialogTakePatrons(playerid);
                    return true;
                }
                switch(inputtext[len])
                {
                    case 'А'..'Я', 'а'..'я', 'A'..'Z', 'a'..'z':
                    {
                        SCM(playerid, COLOR_GREY, !"Введите количество патрон!");
                        DialogTakePatrons(playerid);
                        return true;
                    }
                }
                if(val <= 0)
                {
                    SCM(playerid, COLOR_GREY, !"Вы не можете зарядить меньше, чем 1 патрон!");
                    DialogTakePatrons(playerid);
                    return true;
                }
                switch(player[playerid][pItemSelect])
                {
                    case 1:
                    {
                        if(val > player[playerid][pItemAmmo1])
                        {
                            SCM(playerid, COLOR_GREY, !"Введите количество патрон, которое имеется у вас в инвентаре!");
                            DialogTakePatrons(playerid);
                            return true;
                        }
                        switch(player[playerid][pItem1])
                        {
                            case 3:
                            {
                                GivePlayerWeapon(playerid, WEAPON_DEAGLE, GetPlayerAmmo(playerid)+val);
                                if(val < player[playerid][pItemAmmo1])
                                {
                                    player[playerid][pItemAmmo1]-=val;
                                }
                                else
                                {
                                    player[playerid][pItemAmmo1] = 0;
                                    player[playerid][pItem1] = 0;
                                    SavePlayer(playerid, "Item1", player[playerid][pItem1], "d");
                                }
                                SavePlayer(playerid, "ItemAmmo1", player[playerid][pItemAmmo1], "d");
                                static
                                    fmt_str[] = "*%s достал из кобуры пистолет Desert Eagle";
                                new
                                    string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                format(string, sizeof(string), fmt_str, GetName(playerid));
                                ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                return true;
                            }
                            case 4:
                            {
                                GivePlayerWeapon(playerid, WEAPON_SHOTGUN, GetPlayerAmmo(playerid)+val);
                                if(val < player[playerid][pItemAmmo1])
                                {
                                    player[playerid][pItemAmmo1]-=val;
                                }
                                else
                                {
                                    player[playerid][pItemAmmo1] = 0;
                                    player[playerid][pItem1] = 0;
                                    SavePlayer(playerid, "Item1", player[playerid][pItem1], "d");
                                }
                                SavePlayer(playerid, "ItemAmmo1", player[playerid][pItemAmmo1], "d");
                                static
                                    fmt_str[] = "*%s достал из-за пазухи дробовик";
                                new
                                    string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                format(string, sizeof(string), fmt_str, GetName(playerid));
                                ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                return true;
                            }
                            case 5:
                            {
                                GivePlayerWeapon(playerid, WEAPON_M4, GetPlayerAmmo(playerid)+val);
                                if(val < player[playerid][pItemAmmo1])
                                {
                                    player[playerid][pItemAmmo1]-=val;
                                }
                                else
                                {
                                    player[playerid][pItemAmmo1] = 0;
                                    player[playerid][pItem1] = 0;
                                    SavePlayer(playerid, "Item1", player[playerid][pItem1], "d");
                                }
                                SavePlayer(playerid, "ItemAmmo1", player[playerid][pItemAmmo1], "d");
                                static
                                    fmt_str[] = "*%s достал из-за пазухи карабин M4A1";
                                new
                                    string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                format(string, sizeof(string), fmt_str, GetName(playerid));
                                ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                return true;
                            }
                            case 6:
                            {
                                GivePlayerWeapon(playerid, WEAPON_AK47, GetPlayerAmmo(playerid)+val);
                                if(val < player[playerid][pItemAmmo1])
                                {
                                    player[playerid][pItemAmmo1]-=val;
                                }
                                else
                                {
                                    player[playerid][pItemAmmo1] = 0;
                                    player[playerid][pItem1] = 0;
                                    SavePlayer(playerid, "Item1", player[playerid][pItem1], "d");
                                }
                                SavePlayer(playerid, "ItemAmmo1", player[playerid][pItemAmmo1], "d");
                                static
                                    fmt_str[] = "*%s достал из-за пазухи винтовку AK-47";
                                new
                                    string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                format(string, sizeof(string), fmt_str, GetName(playerid));
                                ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                return true;
                            }
                        }
                    }
                    case 2:
                    {
                        if(val > player[playerid][pItemAmmo2])
                        {
                            SCM(playerid, COLOR_GREY, !"Введите количество патрон, которое имеется у вас в инвентаре!");
                            DialogTakePatrons(playerid);
                            return true;
                        }
                        switch(player[playerid][pItem2])
                        {
                            case 3:
                            {
                                GivePlayerWeapon(playerid, WEAPON_DEAGLE, GetPlayerAmmo(playerid)+val);
                                if(val < player[playerid][pItemAmmo2])
                                {
                                    player[playerid][pItemAmmo2]-=val;
                                }
                                else
                                {
                                    player[playerid][pItemAmmo2] = 0;
                                    player[playerid][pItem2] = 0;
                                    SavePlayer(playerid, "Item2", player[playerid][pItem2], "d");
                                }
                                SavePlayer(playerid, "ItemAmmo2", player[playerid][pItemAmmo2], "d");
                                static
                                    fmt_str[] = "*%s достал из кобуры пистолет Desert Eagle";
                                new
                                    string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                format(string, sizeof(string), fmt_str, GetName(playerid));
                                ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                return true;
                            }
                            case 4:
                            {
                                GivePlayerWeapon(playerid, WEAPON_SHOTGUN, GetPlayerAmmo(playerid)+val);
                                if(val < player[playerid][pItemAmmo2])
                                {
                                    player[playerid][pItemAmmo2]-=val;
                                }
                                else
                                {
                                    player[playerid][pItemAmmo2] = 0;
                                    player[playerid][pItem2] = 0;
                                    SavePlayer(playerid, "Item2", player[playerid][pItem2], "d");
                                }
                                SavePlayer(playerid, "ItemAmmo2", player[playerid][pItemAmmo2], "d");
                                static
                                    fmt_str[] = "*%s достал из-за пазухи дробовик";
                                new
                                    string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                format(string, sizeof(string), fmt_str, GetName(playerid));
                                ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                return true;
                            }
                            case 5:
                            {
                                GivePlayerWeapon(playerid, WEAPON_M4, GetPlayerAmmo(playerid)+val);
                                if(val < player[playerid][pItemAmmo2])
                                {
                                    player[playerid][pItemAmmo2]-=val;
                                }
                                else
                                {
                                    player[playerid][pItemAmmo2] = 0;
                                    player[playerid][pItem2] = 0;
                                    SavePlayer(playerid, "Item2", player[playerid][pItem2], "d");
                                }
                                SavePlayer(playerid, "ItemAmmo2", player[playerid][pItemAmmo2], "d");
                                static
                                    fmt_str[] = "*%s достал из-за пазухи карабин M4A1";
                                new
                                    string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                format(string, sizeof(string), fmt_str, GetName(playerid));
                                ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                return true;
                            }
                            case 6:
                            {
                                GivePlayerWeapon(playerid, WEAPON_AK47, GetPlayerAmmo(playerid)+val);
                                if(val < player[playerid][pItemAmmo2])
                                {
                                    player[playerid][pItemAmmo2]-=val;
                                }
                                else
                                {
                                    player[playerid][pItemAmmo2] = 0;
                                    player[playerid][pItem2] = 0;
                                    SavePlayer(playerid, "Item2", player[playerid][pItem2], "d");
                                }
                                SavePlayer(playerid, "ItemAmmo2", player[playerid][pItemAmmo2], "d");
                                static
                                    fmt_str[] = "*%s достал из-за пазухи винтовку AK-47";
                                new
                                    string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                format(string, sizeof(string), fmt_str, GetName(playerid));
                                ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                return true;
                            }
                        }
                    }
                    case 3:
                    {
                        if(val > player[playerid][pItemAmmo3])
                        {
                            SCM(playerid, COLOR_GREY, !"Введите количество патрон, которое имеется у вас в инвентаре!");
                            DialogTakePatrons(playerid);
                            return true;
                        }
                        switch(player[playerid][pItem3])
                        {
                            case 3:
                            {
                                GivePlayerWeapon(playerid, WEAPON_DEAGLE, GetPlayerAmmo(playerid)+val);
                                if(val < player[playerid][pItemAmmo3])
                                {
                                    player[playerid][pItemAmmo3]-=val;
                                }
                                else
                                {
                                    player[playerid][pItemAmmo3] = 0;
                                    player[playerid][pItem3] = 0;
                                    SavePlayer(playerid, "Item3", player[playerid][pItem3], "d");
                                }
                                SavePlayer(playerid, "ItemAmmo3", player[playerid][pItemAmmo3], "d");
                                static
                                    fmt_str[] = "*%s достал из кобуры пистолет Desert Eagle";
                                new
                                    string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                format(string, sizeof(string), fmt_str, GetName(playerid));
                                ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                return true;
                            }
                            case 4:
                            {
                                GivePlayerWeapon(playerid, WEAPON_DEAGLE, GetPlayerAmmo(playerid)+val);
                                if(val < player[playerid][pItemAmmo3])
                                {
                                    player[playerid][pItemAmmo3]-=val;
                                }
                                else
                                {
                                    player[playerid][pItemAmmo3] = 0;
                                    player[playerid][pItem3] = 0;
                                    SavePlayer(playerid, "Item3", player[playerid][pItem3], "d");
                                }
                                SavePlayer(playerid, "ItemAmmo3", player[playerid][pItemAmmo3], "d");
                                static
                                    fmt_str[] = "*%s достал из-за пазухи дробовик";
                                new
                                    string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                format(string, sizeof(string), fmt_str, GetName(playerid));
                                ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                return true;
                            }
                            case 5:
                            {
                                GivePlayerWeapon(playerid, WEAPON_M4, GetPlayerAmmo(playerid)+val);
                                if(val < player[playerid][pItemAmmo3])
                                {
                                    player[playerid][pItemAmmo3]-=val;
                                }
                                else
                                {
                                    player[playerid][pItemAmmo3] = 0;
                                    player[playerid][pItem3] = 0;
                                    SavePlayer(playerid, "Item3", player[playerid][pItem3], "d");
                                }
                                SavePlayer(playerid, "ItemAmmo3", player[playerid][pItemAmmo3], "d");
                                static
                                    fmt_str[] = "*%s достал из-за пазухи карабин M4A1";
                                new
                                    string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                format(string, sizeof(string), fmt_str, GetName(playerid));
                                ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                return true;
                            }
                            case 6:
                            {
                                GivePlayerWeapon(playerid, WEAPON_AK47, GetPlayerAmmo(playerid)+val);
                                if(val < player[playerid][pItemAmmo3])
                                {
                                    player[playerid][pItemAmmo3]-=val;
                                }
                                else
                                {
                                    player[playerid][pItemAmmo3] = 0;
                                    player[playerid][pItem3] = 0;
                                    SavePlayer(playerid, "Item3", player[playerid][pItem3], "d");
                                }
                                SavePlayer(playerid, "ItemAmmo3", player[playerid][pItemAmmo3], "d");
                                static
                                    fmt_str[] = "*%s достал из-за пазухи винтовку AK-47";
                                new
                                    string[sizeof(fmt_str)-2+MAX_PLAYER_NAME+1];
                                format(string, sizeof(string), fmt_str, GetName(playerid));
                                ProxDetector(playerid, 20.0, COLOR_ACTION, string);
                                return true;
                            }
                        }
                    }
                }
            }
        }

Не совсем понятно в чём заключается вопрос.
Подробнее о том, в каких случаях switch шустрее if, можно почитать тут - https://ziggi.org/pawn-operatory/
А если вопрос про качество кода, то вообще у тебя много повторяющихся кусков, которые можно объединить, помещаяя всякие названия предметов и т.п. в массивы. Впрочем, это усложнит код, так что пока лучше просто делай так, как делал.

Цитата:

---

Сообщение от DeimoS

Подробнее о том, в каких случаях switch шустрее if, можно почитать тут - https://ziggi.org/pawn-operatory/.

---

Спасибо, но vwv (или как там его) делал подобную тему со сравнениями, но конкретно про switch vs if

Цитата:

---

Сообщение от DeimoS

Не совсем понятно в чём заключается вопрос. А если вопрос про качество кода, то вообще у тебя много повторяющихся кусков, которые можно объединить, помещаяя всякие названия предметов и т.п. в массивы. Впрочем, это усложнит код, так что пока лучше просто делай так, как делал.

---

Вопрос: будет ли так работать быстрее, чем тоже самое, но через if? Вдруг я не учел какой-нибудь момент, и там можно было заменить switch на if, и код бы работал быстрее. Но как я понял, таких моментов там нету (я не перечислял значения вовсе, да и зачем - есть default, просто на значения с уникальным диалогом надо написать свой case, а остальные switch возьмет под default). Просто хотел узнать мнение о данном коде.

Цитата:

---

Сообщение от DeimoS

А если вопрос про качество кода, то вообще у тебя много повторяющихся кусков, которые можно объединить, помещаяя всякие названия предметов и т.п. в массивы. Впрочем, это усложнит код, так что пока лучше просто делай так, как делал.

---

Ты намекаешь, что можно сделать проверки быстрее и производительнее? Вроде доступ к массиву медленнее, чем к той же переменной

Цитата:

---

Сообщение от Steve_Stage

Спасибо, но vwv (или как там его) делал подобную тему со сравнениями, но конкретно про switch vs if

---

И в этой же теме я, вроде, отписывался, кидая ссылку на статью Ziggi =) Замеры там идентичные получились.

Цитата:

---

Сообщение от Steve_Stage

Вопрос: будет ли так работать быстрее, чем тоже самое, но через if? Вдруг я не учел какой-нибудь момент, и там можно было заменить switch на if, и код бы работал быстрее. Но как я понял, таких моментов там нету (я не перечислял значения вовсе, да и зачем - есть default, просто на значения с уникальным диалогом надо написать свой case, а остальные switch возьмет под default). Просто хотел узнать мнение о данном коде.

---

В твоём коде вообще не будет существенной разницы между switch и if. Я тебе не просто так кинул тему с замерами и пояснением того, в каких ситуациях switch выигрывает/проигрывает.
Ну либо можно просто знать, что основной прирост скорость у switch происходит за счёт того, что данные из переменной берутся один раз, в отличии от if, в которой проверяется несколько значений. Соответственно, ощутимый прирост будет только тогда, когда в switch будет множество отдельных значений сверяться, а не всего несколько штук. Если значений будет мало, то if и switch по скорости существенно отличаться не будут, а вот места switch занимать будет в разы больше.
Ну и так же если в switch указать промежуток значений ("case 0..10:"), то switch будет сверять значение переменной с каждым значением из промежутка, а не только с начальным и конечным (то бишь, в этом случае if будет быстрее).


Единственное, что тут можно сделать - это, как я уже ранее писал, объединить идентичный код.

Цитата:

---

Сообщение от DeimoS

В твоём коде вообще не будет существенной разницы между switch и if. Я тебе не просто так кинул тему с замерами и пояснением того, в каких ситуациях switch выигрывает/проигрывает.
Единственное, что тут можно сделать - это, как я уже ранее писал, объединить идентичный код.

---

Да, посмотрел зиги, там конструкция

if(i == 40) return true;

идентична

switch(i)
{
    case 40: return true;
}

Я дополнил предыдущий пост. Перепрочти.

Цитата:

---

Сообщение от DeimoS

И в этой же теме я, вроде, отписывался, кидая ссылку на статью Ziggi =) Замеры там идентичные получились.

---

Странно, я говорил про это

  Открыть/закрыть

https://pro-pawn.ru/showthread.php?1...itch-или-if

Там я не нашел ссылки на статью зиги...

Цитата:

---

Сообщение от DeimoS

Я дополнил предыдущий пост. Перепрочти.

---

Знаю, не стоит использовать диапазон значений, но эта система может быть дополнена дополнительными значениями в связи с вводом на сервер дополнительных доступных предметов в инвентарь, и тогда не придется все переписывать с if на switch, а просто вписать новый case

Цитата:

---

Сообщение от Steve_Stage

Странно, я говорил про это

  Открыть/закрыть

https://pro-pawn.ru/showthread.php?1...itch-или-if

Там я не нашел ссылки на статью зиги...

---

А, да. Ссылку на ziggi я ему в вк кидал в конфе.

Цитата:

---

Сообщение от Steve_Stage

Знаю, не стоит использовать диапазон значений, но эта система может быть дополнена дополнительными значениями в связи с вводом на сервер дополнительных доступных предметов в инвентарь, и тогда не придется все переписывать с if на switch, а просто вписать новый case

---

Ну вот подобного рода условия:

                        switch(player[playerid][pItem2])
                        {
                            case 0:
                            {
                                SPD(playerid, 9365, DSM, !"{00C0FF}Ошибка",
                                !"\
                                    {00C0FF}Данный слот пуст!", !"{00C0FF}Понятно", !"\
                                ");
                                return true;
                            }
                            default:
                            {
                                player[playerid][pItemSelect] = 2;
                                 static
                                    fmt_str[] = "{00C0FF}%s";
                                new
                                    string[sizeof(fmt_str)-2+20+8+1];
                                format(string, sizeof(string), fmt_str, invent_items[player[playerid][pItem2]]);
 
                                SPD(playerid, 13, DSL, string,
                                    !"\
                                        {00C0FF}Использовать\n{00C0FF}Выбросить", !"{00C0FF}Далее", !"{00C0FF}Назад\
                                    ");
                            }
                        }

Это точно извращение. Да и зачем дублировать эту проверку для каждого case, если ничего, кроме switch, в диалоге нет? Собственно, как я уже писал, у тебя много дублирующегося кода, который можно объединить.

Цитата:

---

Сообщение от DeimoS

Ну вот подобного рода условия:

                        switch(player[playerid][pItem2])
                        {
                            case 0:
                            {
                                SPD(playerid, 9365, DSM, !"{00C0FF}Ошибка",
                                !"\
                                    {00C0FF}Данный слот пуст!", !"{00C0FF}Понятно", !"\
                                ");
                                return true;
                            }
                            default:
                            {
                                player[playerid][pItemSelect] = 2;
                                 static
                                    fmt_str[] = "{00C0FF}%s";
                                new
                                    string[sizeof(fmt_str)-2+20+8+1];
                                format(string, sizeof(string), fmt_str, invent_items[player[playerid][pItem2]]);
 
                                SPD(playerid, 13, DSL, string,
                                    !"\
                                        {00C0FF}Использовать\n{00C0FF}Выбросить", !"{00C0FF}Далее", !"{00C0FF}Назад\
                                    ");
                            }
                        }

Это точно извращение. Да и зачем дублировать эту проверку для каждого case, если ничего, кроме switch, в диалоге нет? Собственно, как я уже писал, у тебя много дублирующегося кода, который можно объединить.

---

Ты не заметил, что для каждого слота есть своя переменная в энуме игрока?

player[playerid][pItem1] // Предмет в 1 слоте
player[playerid][pItem2] // Во 2 слоте
player[playerid][pItem3] // В третьем

И как я объединю проверки?

Вот сам энум, там 3 переменные для слотов и 3 для запаса

enum pInfo {
    pPass[50+1], pMail[60+1], pMoney, pLevel, pAge, pNational, pSkin, pSex, pID, pReferal[MAX_PLAYER_NAME+1], pDateReg[10+1], pExp, 
        pDeath, pNockout, pItem1, pItem2, pItem3, pItemAmmo1, pItemAmmo2, pItemAmmo3, pItemSelect, pAdmin, pMute, pWarn, pJail, 
        Float:pX, Float:pY, Float:pZ, Float:pA
}
new
    player[MAX_PLAYERS][pInfo];