My problem

//====================
#include <a_samp>
//====================
#define COLOR_NARANCS 0xFF9900AA
//====================
public OnPlayerCommandText(playerid, cmdtext[])
{
    if(strcmp(cmdtext, "/guns", true) == 0)
    {
        ShowPlayerDialog(playerid, 1, DIALOG_STYLE_LIST, "Weapon spawner", "Knife = $500 \nGranade = $2000 \n9mm = $2500 \nSilenced 9mm = $2800 \nDesert Eagle = $3500 \nTec9 = $4000 \nShotgun = $4000 \nSawnoff Shotgun = $4600 \nCombat Shotgun = $4600 \nUzi = $4000 \nMP5 = $4000 \nAK-47 = $8000 \nM4 = $8000 \nSniper = $9000", "Buy", "Cancel");
        return 1;
    }
    return 0;
}
//====================
public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
    if(dialogid == 1)
    {
        if(response)
        {
            if(listitem == 0)
            {
                GivePlayerWeapon(playerid,4, 1);
                GivePlayerMoney(playerid, -500);
                new name[256];
                new string[256];
                GetPlayerName(playerid, name, sizeof(name));
                format(string, sizeof(string), "[INFO] %s got a Knife.", name);
                SendClientMessageToAll(COLOR_NARANCS, string);
                TogglePlayerControllable(playerid,1);
            }
            if(listitem == 1)
            {
                GivePlayerWeapon(playerid,16, 5);
                GivePlayerMoney(playerid, -2000);
                new name[256];
                new string[256];
                GetPlayerName(playerid, name, sizeof(name));
                format(string, sizeof(string), "[INFO] %s got Granades.", name);
                SendClientMessageToAll(COLOR_NARANCS, string);
                TogglePlayerControllable(playerid,1);
            }
            if(listitem == 2)
            {
                GivePlayerWeapon(playerid,22, 200);
                GivePlayerMoney(playerid, -2500);
                new name[256];
                new string[256];
                GetPlayerName(playerid, name, sizeof(name));
                format(string, sizeof(string), "[INFO] %s got a 9mm Pistol.", name);
                SendClientMessageToAll(COLOR_NARANCS, string);
                TogglePlayerControllable(playerid,1);
            }
            if(listitem == 3)
            {
                GivePlayerWeapon(playerid,23, 200);
                GivePlayerMoney(playerid, -2800);
                new name[256];
                new string[256];
                GetPlayerName(playerid, name, sizeof(name));
                format(string, sizeof(string), "[INFO] %s got a 9mm Silenced Pistol.", name);
                SendClientMessageToAll(COLOR_NARANCS, string);
                TogglePlayerControllable(playerid,1);
            }
            if(listitem == 4)
            {
                GivePlayerWeapon(playerid,24, 200);
                GivePlayerMoney(playerid, -3500);
                new name[256];
                new string[256];
                GetPlayerName(playerid, name, sizeof(name));
                format(string, sizeof(string), "[INFO] %s got a Desert Eagle.", name);
                SendClientMessageToAll(COLOR_NARANCS, string);
                TogglePlayerControllable(playerid,1);
            }
            if(listitem == 5)
            {
                GivePlayerWeapon(playerid,32, 200);
                GivePlayerMoney(playerid, -4000);
                new name[256];
                new string[256];
                GetPlayerName(playerid, name, sizeof(name));
                format(string, sizeof(string), "[INFO] %s got a Tec9.", name);
                SendClientMessageToAll(COLOR_NARANCS, string);
                TogglePlayerControllable(playerid,1);
            }
            if(listitem == 6)
            {
                GivePlayerWeapon(playerid,25, 20);
                GivePlayerMoney(playerid, -4000);
                new name[256];
                new string[256];
                GetPlayerName(playerid, name, sizeof(name));
                format(string, sizeof(string), "[INFO] %s got a Shotgun.", name);
                SendClientMessageToAll(COLOR_NARANCS, string);
                TogglePlayerControllable(playerid,1);
            }
            if(listitem == 7)
            {
                GivePlayerWeapon(playerid,26, 20);
                GivePlayerMoney(playerid, -4600);
                new name[256];
                new string[256];
                GetPlayerName(playerid, name, sizeof(name));
                format(string, sizeof(string), "[INFO] %s got a Sawnoff Shotgun.", name);
                SendClientMessageToAll(COLOR_NARANCS, string);
                TogglePlayerControllable(playerid,1);
            }
            if(listitem == 8)
            {
                GivePlayerWeapon(playerid,27, 20);
                GivePlayerMoney(playerid, -4600);
                new name[256];
                new string[256];
                GetPlayerName(playerid, name, sizeof(name));
                format(string, sizeof(string), "[INFO] %s got a Combat Shotgun.", name);
                SendClientMessageToAll(COLOR_NARANCS, string);
                TogglePlayerControllable(playerid,1);
            }
            if(listitem == 9)
            {
                GivePlayerWeapon(playerid,28, 200);
                GivePlayerMoney(playerid, -4000);
                new name[256];
                new string[256];
                GetPlayerName(playerid, name, sizeof(name));
                format(string, sizeof(string), "[INFO] %s got an Uzi.", name);
                SendClientMessageToAll(COLOR_NARANCS, string);
                TogglePlayerControllable(playerid,1);
            }
            if(listitem == 10)
            {
                GivePlayerWeapon(playerid,29, 200);
                GivePlayerMoney(playerid, -4000);
                new name[256];
                new string[256];
                GetPlayerName(playerid, name, sizeof(name));
                format(string, sizeof(string), "[INFO] %s got an MP5.", name);
                SendClientMessageToAll(COLOR_NARANCS, string);
                TogglePlayerControllable(playerid,1);
            }
            if(listitem == 11)
            {
                GivePlayerWeapon(playerid,30, 300);
                GivePlayerMoney(playerid, -8000);
                new name[256];
                new string[256];
                GetPlayerName(playerid, name, sizeof(name));
                format(string, sizeof(string), "[INFO] %s got an AK-47.", name);
                SendClientMessageToAll(COLOR_NARANCS, string);
                TogglePlayerControllable(playerid,1);
            }
            if(listitem == 12)
            {
                GivePlayerWeapon(playerid,31, 300);
                GivePlayerMoney(playerid, -8000);
                new name[256];
                new string[256];
                GetPlayerName(playerid, name, sizeof(name));
                format(string, sizeof(string), "[INFO] %s got a M4.", name);
                SendClientMessageToAll(COLOR_NARANCS, string);
                TogglePlayerControllable(playerid,1);
            }
            if(listitem == 13)
            {
                GivePlayerWeapon(playerid,34, 20);
                GivePlayerMoney(playerid, -9000);
                new name[256];
                new string[256];
                GetPlayerName(playerid, name, sizeof(name));
                format(string, sizeof(string), "[INFO] %s got a Sniper Rifle.", name);
                SendClientMessageToAll(COLOR_NARANCS, string);
                TogglePlayerControllable(playerid,1);
            }
        }
    }
    return 1;
}
//====================