Login

Cole_William · 20.07.2014, 16:10

So my issue is, that the dialogs if you click on the guns or skins it wont give you or set you any of them, now the "Duty" button works, it sets their duty color corectly but nothing else.

Defines:
http://pastebin.com/3ETsdS8g

Command:
http://pastebin.com/wkn1WqRf

Dialog:
http://pastebin.com/sdP9jNMK

Cole_William · 20.07.2014, 17:21

Anyone?

Scottas · 20.07.2014, 17:35

can't see problem here. Do you have more filterscripts, that uses OnDialogResponse?

Cole_William · 20.07.2014, 17:43

No Filterscripts, everything is coded in the script.

Cole_William · 20.07.2014, 18:28

Ok, so does this mean my script is just screwed without reason?

Cole_William · 20.07.2014, 20:13

I've tried switching the cases too

pawn Код:

if(listitem == 2)

But just returned a lot of errors. Does no one know what's wrong?

Scottas · 20.07.2014, 20:24

Oh, I see the problem. Bad script identation caused it, be carefull with it, look what you have:

pawn Код:

if(dialogid == DIALOG_LOCKER)
{
    if( response )
    {
        switch(listitem)
        {
            case 0: // Duty
            {
                if(PlayerInfo[playerid][pBadge] == 0)
                {
                    PlayerInfo[playerid][pBadge] = 1;
                    SetPlayerColor(playerid, GetFactionColor(playerid));
                }
                else if(PlayerInfo[playerid][pBadge] == 1)
                {
                    PlayerInfo[playerid][pBadge] = 0;
                    SetPlayerColor(playerid, COLOR_WHITE);
                }
            }
            case 1: // Weapons
            {
                if(PlayerInfo[playerid][pFaction] == 1) // GOV Weapon Choice
                {
                    ShowPlayerDialog(playerid, DIALOG_GOVWEAPONS, DIALOG_STYLE_LIST, "GOV Weapons", "Baton\nPepper Spray\nDeagle", "Select","Cancel");
                }
                if(PlayerInfo[playerid][pFaction] == 2) // LSPD Weapons Choice
                {
                    ShowPlayerDialog(playerid, DIALOG_LSPDWEAPONS, DIALOG_STYLE_LIST, "LSPD Weapons", "Baton\nPepper Spray\nDeagle", "Select","Cancel");
                }
            }
            case 2: // Skins
            {
                if(PlayerInfo[playerid][pFaction] == 1) // GOV Skin Choice
                {
                    ShowPlayerDialog(playerid, DIALOG_GOVSKINS, DIALOG_STYLE_LIST, "Skins", "Vacant\nVacant\nVacant", "Select","Cancel");
                }
                if(PlayerInfo[playerid][pFaction] == 2) // LSPD Skin Choice
                {
                    ShowPlayerDialog(playerid, DIALOG_LSPDSKINS, DIALOG_STYLE_LIST, "Skins", "Vacant\nVacant\nVacant", "Select","Cancel");
                }
            }
        }
    }
    if(dialogid == DIALOG_GOVWEAPONS) // Gov Weapons
    {
        if(response)
        {
            switch(listitem)
            {
                case 0: // Baton
                {
                    GivePlayerWeapon(playerid, 3, 0);
                }
                case 1: // Pepperspray
                {
                    GivePlayerWeapon(playerid, 41, 100);
                }
                case 2: // Deagle
                {
                    GivePlayerWeapon(playerid, 24, 100);
                }
            }
        }
    }

    if(dialogid == DIALOG_LSPDWEAPONS) // LSPD Weapons
    {
        if(response)
        {
            switch(listitem)
            {
                case 0: // Baton
                {
                    GivePlayerWeapon(playerid, 3, 0);
                }
                case 1: // Pepperspray
                {
                    GivePlayerWeapon(playerid, 41, 100);
                }
                case 2: // Deagle
                {
                    GivePlayerWeapon(playerid, 24, 100);
                }
            }
        }
    }

    if(dialogid == DIALOG_GOVSKINS) // GOV Skins
    {
        if(response)
        {
            switch(listitem)
            {
                case 0: // Empty
                {
                    PlayerInfo[playerid][pSkin] = 281;
                    SetPlayerSkin(playerid, 281);
                }
                case 1: // Empty
                {
                    PlayerInfo[playerid][pSkin] = 281;
                    SetPlayerSkin(playerid, 281);
                }
                case 2: // Empty
                {
                    PlayerInfo[playerid][pSkin] = 281;
                    SetPlayerSkin(playerid, 281);
                }
            }
        }
    }
    if(dialogid == DIALOG_LSPDSKINS) // LSPD Skins
    {
        if(response)
        {
            switch(listitem)
            {
                case 0: // Empty
                {
                    PlayerInfo[playerid][pSkin] = 281;
                    SetPlayerSkin(playerid, 281);
                }
                case 1: // Empty
                {
                    PlayerInfo[playerid][pSkin] = 281;
                    SetPlayerSkin(playerid, 281);
                }
                case 2: // Empty
                {
                    PlayerInfo[playerid][pSkin] = 281;
                    SetPlayerSkin(playerid, 281);
                }
            }
        }
    }
}

Do you see what's wrong? All code is inside first check, so other dialogid checks allways returns false, because the first one is true.

Cole_William · 20.07.2014, 20:27

Quote:

Originally Posted by Scottas

Oh, I see the problem. Bad script identation caused it, be carefull with it, look what you have:

pawn Код:

if(dialogid == DIALOG_LOCKER)
{
    if( response )
    {
        switch(listitem)
        {
            case 0: // Duty
            {
                if(PlayerInfo[playerid][pBadge] == 0)
                {
                    PlayerInfo[playerid][pBadge] = 1;
                    SetPlayerColor(playerid, GetFactionColor(playerid));
                }
                else if(PlayerInfo[playerid][pBadge] == 1)
                {
                    PlayerInfo[playerid][pBadge] = 0;
                    SetPlayerColor(playerid, COLOR_WHITE);
                }
            }
            case 1: // Weapons
            {
                if(PlayerInfo[playerid][pFaction] == 1) // GOV Weapon Choice
                {
                    ShowPlayerDialog(playerid, DIALOG_GOVWEAPONS, DIALOG_STYLE_LIST, "GOV Weapons", "Baton\nPepper Spray\nDeagle", "Select","Cancel");
                }
                if(PlayerInfo[playerid][pFaction] == 2) // LSPD Weapons Choice
                {
                    ShowPlayerDialog(playerid, DIALOG_LSPDWEAPONS, DIALOG_STYLE_LIST, "LSPD Weapons", "Baton\nPepper Spray\nDeagle", "Select","Cancel");
                }
            }
            case 2: // Skins
            {
                if(PlayerInfo[playerid][pFaction] == 1) // GOV Skin Choice
                {
                    ShowPlayerDialog(playerid, DIALOG_GOVSKINS, DIALOG_STYLE_LIST, "Skins", "Vacant\nVacant\nVacant", "Select","Cancel");
                }
                if(PlayerInfo[playerid][pFaction] == 2) // LSPD Skin Choice
                {
                    ShowPlayerDialog(playerid, DIALOG_LSPDSKINS, DIALOG_STYLE_LIST, "Skins", "Vacant\nVacant\nVacant", "Select","Cancel");
                }
            }
        }
    }
    if(dialogid == DIALOG_GOVWEAPONS) // Gov Weapons
    {
        if(response)
        {
            switch(listitem)
            {
                case 0: // Baton
                {
                    GivePlayerWeapon(playerid, 3, 0);
                }
                case 1: // Pepperspray
                {
                    GivePlayerWeapon(playerid, 41, 100);
                }
                case 2: // Deagle
                {
                    GivePlayerWeapon(playerid, 24, 100);
                }
            }
        }
    }

    if(dialogid == DIALOG_LSPDWEAPONS) // LSPD Weapons
    {
        if(response)
        {
            switch(listitem)
            {
                case 0: // Baton
                {
                    GivePlayerWeapon(playerid, 3, 0);
                }
                case 1: // Pepperspray
                {
                    GivePlayerWeapon(playerid, 41, 100);
                }
                case 2: // Deagle
                {
                    GivePlayerWeapon(playerid, 24, 100);
                }
            }
        }
    }

    if(dialogid == DIALOG_GOVSKINS) // GOV Skins
    {
        if(response)
        {
            switch(listitem)
            {
                case 0: // Empty
                {
                    PlayerInfo[playerid][pSkin] = 281;
                    SetPlayerSkin(playerid, 281);
                }
                case 1: // Empty
                {
                    PlayerInfo[playerid][pSkin] = 281;
                    SetPlayerSkin(playerid, 281);
                }
                case 2: // Empty
                {
                    PlayerInfo[playerid][pSkin] = 281;
                    SetPlayerSkin(playerid, 281);
                }
            }
        }
    }
    if(dialogid == DIALOG_LSPDSKINS) // LSPD Skins
    {
        if(response)
        {
            switch(listitem)
            {
                case 0: // Empty
                {
                    PlayerInfo[playerid][pSkin] = 281;
                    SetPlayerSkin(playerid, 281);
                }
                case 1: // Empty
                {
                    PlayerInfo[playerid][pSkin] = 281;
                    SetPlayerSkin(playerid, 281);
                }
                case 2: // Empty
                {
                    PlayerInfo[playerid][pSkin] = 281;
                    SetPlayerSkin(playerid, 281);
                }
            }
        }
    }
}

Do you see what's wrong? All code is inside first check, so other dialogid checks allways returns false, because the first one is true.

I don't see nor understand, i am a new scripter so you gotta explain a bit more please.

Scottas · 20.07.2014, 20:30

I've explained as I could, there is working version of your code:

pawn Код:

if(dialogid == DIALOG_LOCKER)
{
    if( response )
    {
        switch(listitem)
        {
            case 0: // Duty
            {
                if(PlayerInfo[playerid][pBadge] == 0)
                {
                    PlayerInfo[playerid][pBadge] = 1;
                    SetPlayerColor(playerid, GetFactionColor(playerid));
                }
                else if(PlayerInfo[playerid][pBadge] == 1)
                {
                    PlayerInfo[playerid][pBadge] = 0;
                    SetPlayerColor(playerid, COLOR_WHITE);
                }
            }
            case 1: // Weapons
            {
                if(PlayerInfo[playerid][pFaction] == 1) // GOV Weapon Choice
                {
                    ShowPlayerDialog(playerid, DIALOG_GOVWEAPONS, DIALOG_STYLE_LIST, "GOV Weapons", "Baton\nPepper Spray\nDeagle", "Select","Cancel");
                }
                if(PlayerInfo[playerid][pFaction] == 2) // LSPD Weapons Choice
                {
                    ShowPlayerDialog(playerid, DIALOG_LSPDWEAPONS, DIALOG_STYLE_LIST, "LSPD Weapons", "Baton\nPepper Spray\nDeagle", "Select","Cancel");
                }
            }
            case 2: // Skins
            {
                if(PlayerInfo[playerid][pFaction] == 1) // GOV Skin Choice
                {
                    ShowPlayerDialog(playerid, DIALOG_GOVSKINS, DIALOG_STYLE_LIST, "Skins", "Vacant\nVacant\nVacant", "Select","Cancel");
                }
                if(PlayerInfo[playerid][pFaction] == 2) // LSPD Skin Choice
                {
                    ShowPlayerDialog(playerid, DIALOG_LSPDSKINS, DIALOG_STYLE_LIST, "Skins", "Vacant\nVacant\nVacant", "Select","Cancel");
                }
            }
        }
    }
}
if(dialogid == DIALOG_GOVWEAPONS) // Gov Weapons
{
    if(response)
    {
        switch(listitem)
        {
            case 0: // Baton
            {
                GivePlayerWeapon(playerid, 3, 0);
            }
            case 1: // Pepperspray
            {
                GivePlayerWeapon(playerid, 41, 100);
            }
            case 2: // Deagle
            {
                GivePlayerWeapon(playerid, 24, 100);
            }
        }
    }
}

if(dialogid == DIALOG_LSPDWEAPONS) // LSPD Weapons
{
    if(response)
    {
        switch(listitem)
        {
            case 0: // Baton
            {
                GivePlayerWeapon(playerid, 3, 0);
            }
            case 1: // Pepperspray
            {
                GivePlayerWeapon(playerid, 41, 100);
            }
            case 2: // Deagle
            {
                GivePlayerWeapon(playerid, 24, 100);
            }
        }
    }
}

if(dialogid == DIALOG_GOVSKINS) // GOV Skins
{
    if(response)
    {
        switch(listitem)
        {
            case 0: // Empty
            {
                PlayerInfo[playerid][pSkin] = 281;
                SetPlayerSkin(playerid, 281);
            }
            case 1: // Empty
            {
                PlayerInfo[playerid][pSkin] = 281;
                SetPlayerSkin(playerid, 281);
            }
            case 2: // Empty
            {
                PlayerInfo[playerid][pSkin] = 281;
                SetPlayerSkin(playerid, 281);
            }
        }
    }
}
if(dialogid == DIALOG_LSPDSKINS) // LSPD Skins
{
    if(response)
    {
        switch(listitem)
        {
            case 0: // Empty
            {
                PlayerInfo[playerid][pSkin] = 281;
                SetPlayerSkin(playerid, 281);
            }
            case 1: // Empty
            {
                PlayerInfo[playerid][pSkin] = 281;
                SetPlayerSkin(playerid, 281);
            }
            case 2: // Empty
            {
                PlayerInfo[playerid][pSkin] = 281;
                SetPlayerSkin(playerid, 281);
            }
        }
    }
}

Login
Username:
Password:	Lost Password?
	Remember me