[HELP]ERROR
#2

If you had been indenting your code properly you would've noticed that you were missing a bracket under almost every if statement in this snippet of code, here is the fixed version:

pawn Код:
public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
    //============================DIALOGO TELETRASPORTI
    if(dialogid == DIALOGID)
    {
        if(response)
        {
            if(listitem == 0) // TELE LS
            {
                ShowPlayerDialog(playerid, DIALOGID+1, DIALOG_STYLE_LIST, "Teles Los Santos", "1. Groove Street\n2. Police Station\n3. El Corona\n4. WillowField\n5. Skate Park\n6. Glen Park\n7. Winewood\n8. Verdant Bluffs\n9. Aeroporto LS", "Ok", "Esci");
            }
            /*if(listitem == 1) // TELE SF
            {
                ShowPlayerDialog(playerid, DIALOGID+2, DIALOG_STYLE_LIST, " ", " ", "Ok", "Cancella");
            }
            if(listitem == 2) //TELE LV
            {
                ShowPlayerDialog(playerid, DIALOGID+3, DIALOG_STYLE_LIST, " ", " ", "Ok", "Cancella");
        }*/

        }
        return 1;
    }
    if(dialogid == DIALOGID+1) //TELES LOS SANTOS
    {
        if(response)
        {
            if(listitem == 0) //GROVE STREET
            {
                if (GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
                {
                    LinkVehicleToInterior(GetPlayerVehicleID(playerid), 0);
                    SetVehiclePos(GetPlayerVehicleID(playerid), 2507.4324,-1679.7229,13.5469);
                    SetVehicleZAngle(GetPlayerVehicleID(playerid), 0);
                }
                else
                {
                    SetPlayerInterior(playerid,0);
                    SetPlayerPos(playerid, 2507.4324,-1679.7229,13.5469);
                    SetPlayerFacingAngle(playerid, 0);
                }
            }
            if(listitem == 1) //police station
            {
                if (GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
                {
                    LinkVehicleToInterior(GetPlayerVehicleID(playerid), 0);
                    SetVehiclePos(GetPlayerVehicleID(playerid), 1510.8280,-1671.2766,14.0469);
                    SetVehicleZAngle(GetPlayerVehicleID(playerid), 0);
                }
                else
                {
                    SetPlayerInterior(playerid,0);
                    SetPlayerPos(playerid, 1510.8280,-1671.2766,14.0469);
                    SetPlayerFacingAngle(playerid, 0);
                }
            }
            if(listitem == 2) //el corona
            {
                if (GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
                {
                    LinkVehicleToInterior(GetPlayerVehicleID(playerid), 0);
                    SetVehiclePos(GetPlayerVehicleID(playerid), 1812.9939,-1886.6304,13.4141);
                    SetVehicleZAngle(GetPlayerVehicleID(playerid), 0);
                }
                else
                {
                    SetPlayerInterior(playerid,0);
                    SetPlayerPos(playerid, 1812.9939,-1886.6304,13.4141);
                    SetPlayerFacingAngle(playerid, 0);
                }
            }
            if(listitem == 3) //willowfied
            {
                if (GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
                {
                    LinkVehicleToInterior(GetPlayerVehicleID(playerid), 0);
                    SetVehiclePos(GetPlayerVehicleID(playerid), 2451.9905,-2020.5287,13.5469);
                    SetVehicleZAngle(GetPlayerVehicleID(playerid), 0);
                }
                else
                {
                    SetPlayerInterior(playerid,0);
                    SetPlayerPos(playerid, 2451.9905,-2020.5287,13.5469);
                    SetPlayerFacingAngle(playerid, 0);
                }
            }
            if(listitem == 4) //willowfied
            {
                if (GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
                {
                    LinkVehicleToInterior(GetPlayerVehicleID(playerid), 0);
                    SetVehiclePos(GetPlayerVehicleID(playerid), 1965.3345,-1448.1998,13.4840);
                    SetVehicleZAngle(GetPlayerVehicleID(playerid), 0);
                }
                else
                {
                    SetPlayerInterior(playerid,0);
                    SetPlayerPos(playerid, 1965.3345,-1448.1998,13.4840);
                    SetPlayerFacingAngle(playerid, 0);
                }
            }
            if(listitem == 5) //glen park
            {
                if (GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
                {
                    LinkVehicleToInterior(GetPlayerVehicleID(playerid), 0);
                    SetVehiclePos(GetPlayerVehicleID(playerid), 1900.3505,-1171.6853,24.2914);
                    SetVehicleZAngle(GetPlayerVehicleID(playerid), 0);
                }
                else
                {
                    SetPlayerInterior(playerid,0);
                    SetPlayerPos(playerid, 1900.3505,-1171.6853,24.2914);
                    SetPlayerFacingAngle(playerid, 0);
                }
            }
            if(listitem == 6) //winewood
            {
                if (GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
                {
                    LinkVehicleToInterior(GetPlayerVehicleID(playerid), 0);
                    SetVehiclePos(GetPlayerVehicleID(playerid), 1415.2092,-845.7033,47.6591);
                    SetVehicleZAngle(GetPlayerVehicleID(playerid), 0);
                }
                else
                {
                    SetPlayerInterior(playerid,0);
                    SetPlayerPos(playerid, 1415.2092,-845.7033,47.6591);
                    SetPlayerFacingAngle(playerid, 0);
                }
            }
            if(listitem == 7) //verdant buffs
            {
                if (GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
                {
                    LinkVehicleToInterior(GetPlayerVehicleID(playerid), 0);
                    SetVehiclePos(GetPlayerVehicleID(playerid), 1126.0853,-2036.4387,69.8839);
                    SetVehicleZAngle(GetPlayerVehicleID(playerid), 0);
                }
                else
                {
                    SetPlayerInterior(playerid,0);
                    SetPlayerPos(playerid, 1126.0853,-2036.4387,69.8839);
                    SetPlayerFacingAngle(playerid, 0);
                }
            }
            if(listitem == 8) //aereoporto ls
            {
                if (GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
                {
                    LinkVehicleToInterior(GetPlayerVehicleID(playerid), 0);
                    SetVehiclePos(GetPlayerVehicleID(playerid), 1788.3025,-2423.0300,13.5547);
                    SetVehicleZAngle(GetPlayerVehicleID(playerid), 0);
                }
                else
                {
                    SetPlayerInterior(playerid,0);
                    SetPlayerPos(playerid, 1788.3025,-2423.0300,13.5547);
                    SetPlayerFacingAngle(playerid, 0);
                }
            }
        }
        return 1;
    }
    return 1;
}
In future I suggest you indent your code. Also I suggest you use a switch statement for checking the listitem instead of a lot of if statements, especially when you don't even use else if statements!
Reply


Messages In This Thread
[HELP]ERROR - by TheBluec0de - 07.08.2011, 20:53
Re: [HELP]ERROR - by JaTochNietDan - 07.08.2011, 21:01
Re: [HELP]ERROR - by TheBluec0de - 07.08.2011, 21:05
Re: [HELP]ERROR - by JaTochNietDan - 07.08.2011, 21:09
Re: [HELP]ERROR - by TheBluec0de - 07.08.2011, 21:10

Forum Jump:


Users browsing this thread: 1 Guest(s)