[Lixyde]

So in my gamemode, i've created Vehicle Spawner. But there is other problem!

I get: Loose Indentation.

But i don't know how to fix it!

Here is the code:

https://pastebin.com/kD367JCt

And the line in which i got warning is:

if(dialogid == 5681)

Please help!

[RxErT]

Yes, poor indentation.

What is poor indentation

-Poor indentation be like this:

Код:

public OnPlayerSpawn(playerid)
{
SendClientMessage(playerid, -1,"Welcome bud");
GivePlayerWeapon(playerid, 24, 500000000);
SetPlayerArmour(playerid, 100000000);
     GivePlayerMoney(playerid, 41000);
     SetPlayerHealth(playerid, 50);
return 1;
}

To fix it, you should do this for ex:

Код:

public OnPlayerSpawn(playerid)
{
     SendClientMessage(playerid, -1,"Welcome bud");
     GivePlayerWeapon(playerid, 24, 500000000);
     SetPlayerArmour(playerid, 100000000);
     GivePlayerMoney(playerid, 41000);
     SetPlayerHealth(playerid, 50);
     return 1;
}

I did my best on fixing the indentation;

PHP код:

if (response)
   {
     switch (dialogid)
     {
       case 3434:
       {
            switch (listitem)
            {
               case 0: ShowPlayerDialog(playerid, 3435, DIALOG_STYLE_LIST, "Airplanes", "Andromada\nAT-400\nBeagle\nCropduster\nDodo\nHydra\nNevada\nRustler\nShamal\nSkimmer\nStuntplane\nBack", "Select", "Cancel");
               case 1: ShowPlayerDialog(playerid, 3436, DIALOG_STYLE_LIST, "Helicopters", "Cargobob\nHunter\nLeviathan\nMaverick\nNews Maverick\nPolice Maverick\nRaindance\nSeasparrow\nSparrow\nBack", "Select", "Cancel");
               case 2: ShowPlayerDialog(playerid, 3437, DIALOG_STYLE_LIST, "Bikes", "BF-400\nBike\nBMX\nFaggio\nFCR-900\nFreeway\nMountain Bike\nNRG-500\nPCJ-600\nPizzaboy\nQuad\nSanchez\nWayfarer\nBack", "Select", "Cancel");
               case 3: ShowPlayerDialog(playerid, 3438, DIALOG_STYLE_LIST, "Convertibles", "Comet\nFeltzer\nStallion\nWindsor\nBack", "Select", "Cancel" );
               case 4: ShowPlayerDialog(playerid, 3439, DIALOG_STYLE_LIST, "Industrial", "Benson\nBobcat\nBurrito\nBoxville\nBoxburg\nCement Truck\nDFT-30\nFlatbed\nLinerunner\nMule\nNewsvan\nPacker\nPetrol Tanker\nPony\nRoadtrain\nRumpo\nSadler\nSadler Shit\nTopfun\nTractor\nTrashmaster\nUtility Van\nWalton\nYankee\nYosemite\nBack", "Select", "Cancel");
               case 5: ShowPlayerDialog(playerid, 3440, DIALOG_STYLE_LIST, "Lowriders", "Blade\nBroadway\nRemington\nSavanna\nSlamvan\nTahoma\nTornado\nVoodoo\nBack", "Select", "Cancel" );
               case 6: ShowPlayerDialog(playerid, 3441, DIALOG_STYLE_LIST, "Off Road", "Bandito\nBF Injection\nDune\nHuntley\nLandstalker\nMesa\nMonster\nMonster A\nMonster B\nPatriot\nRancher A\nRancher B\nSandking\nBack", "Select", "Cancel");
               case 7: ShowPlayerDialog(playerid, 3442, DIALOG_STYLE_LIST, "Public Service Vehicles", "Ambulance\nBarracks\nBus\nCabbie\nCoach\nCop Bike (HPV-1000)\nEnforcer\nFBI Rancher\nFBI Truck\nFiretruck\nFiretruck LA\nPolice Car (LSPD)\nPolice Car (LVPD)\nPolice Car (SFPD)\nRanger\nRhino\nS.W.A.T\nTaxi\nBack", "Select", "Cancel");
               case 8: ShowPlayerDialog(playerid, 3443, DIALOG_STYLE_LIST, "Saloons", "Admiral\nBloodring Banger\nBravura\nBuccaneer\nCadrona\nClover\nElegant\nElegy\nEmperor\nEsperanto\nFortune\nGlendale Shit\nGlendale\nGreenwood\nHermes\nIntruder\nMajestic\nManana\nMerit\nNebula\nOceanic\nPicador\nPremier\nPrevion\nPrimo\nSentinel\nStafford\nSultan\nSunrise\nTampa\nVincent\nVirgo\nWillard\nWashington\nBack", "Select", "Cancel");
               case 9: ShowPlayerDialog(playerid, 3444, DIALOG_STYLE_LIST, "Sport Vehicles", "Alpha\nBanshee\nBlista Compact\nBuffalo\nBullet\nCheetah\nClub\nEuros\nFlash\nHotring Racer\nHotring Racer A\nHotring Racer B\nInfernus\nJester\nPhoenix\nSabre\nSuper GT\nTurismo\nUranus\nZR-350\nBack", "Select", "Cancel");
               case 10: ShowPlayerDialog(playerid, 3445, DIALOG_STYLE_LIST, "Station Wagons", "Moonbeam\nPerenniel\nRegina\nSolair\nStratum\nBack", "Select", "Cancel");
               case 11: ShowPlayerDialog(playerid, 3446, DIALOG_STYLE_LIST, "Boats", "Coastguard\nDinghy\nJetmax\nLaunch\nMarquis\nPredator\nReefer\nSpeeder\nSquallo\nTropic\nBack", "Select", "Cancel");
               case 12: ShowPlayerDialog(playerid, 3447, DIALOG_STYLE_LIST, "Trailers", "Article Trailer\nArticle Trailer 2\nArticle Trailer 3\nBaggage Trailer A\nBaggage Trailer B\nFarm Trailer\nFreight Flat Trailer (Train)\nFreight Box Trailer (Train)\nPetrol Trailer\nStreak Trailer (Train)\nStairs Trailer\nUtility Trailer\nBack", "Select", "Cancel");
               case 13: ShowPlayerDialog(playerid, 3448, DIALOG_STYLE_LIST, "Unique Vehicles", "Baggage\nBrownstreak (Train)\nCaddy\nCamper\nCamper A\nCombine Harvester\nDozer\nDumper\nForklift\nFreight (Train)\nHotknife\nHustler\nHotdog\nKart\nMower\nMr Whoopee\nRomero\nSecuricar\nStretch\nSweeper\nTram\nTowtruck\nTug\nVortex\nBack", "Select", "Cancel");
               case 14: ShowPlayerDialog(playerid, 3449, DIALOG_STYLE_LIST, "RC Vehicles", "RC Bandit\nRC Baron\nRC Raider\nRC Goblin\nRC Tiger\nRC Cam\nBack", "Select", "Cancel");
            }
        }
            case 3435:
            {
                if ( listitem > 10 ) return ShowPlayerDefaultDialog(playerid);
                new model_array[] = {592, 577, 511, 512, 593, 520, 553, 476, 519, 460, 513};
                return CreatePlayerVehicle(playerid, model_array[listitem]);
            }
            case 3436:
            {
                if (listitem > 8) return ShowPlayerDefaultDialog(playerid);
                new model_array[] = { 548, 425, 417, 487, 488, 497, 563, 447, 469 };
                return CreatePlayerVehicle( playerid, model_array[ listitem ] );
            }
            case 3437:
            {
                if (listitem > 12) return ShowPlayerDefaultDialog(playerid);
                new model_array[] = {581, 509, 481, 462, 521, 463, 510, 522, 461, 448, 471, 468, 586};
                return CreatePlayerVehicle(playerid, model_array[listitem ]);
            }
            case 3438:
            {
                if (listitem > 3) return ShowPlayerDefaultDialog(playerid);
                new model_array[] = {480, 533, 439, 555};
                return CreatePlayerVehicle(playerid, model_array[listitem]);
            }
            case 3439:
            {
                if (listitem > 24) return ShowPlayerDefaultDialog(playerid);
                new model_array[] = {499, 422, 482, 498, 609, 524, 578, 455, 403, 414, 582, 443, 514, 413, 515, 440, 543, 605, 459, 531, 408, 552, 478, 456, 554};
                return CreatePlayerVehicle(playerid, model_array[listitem]);
            }
            case 3440:
            {
                if (listitem > 7) return ShowPlayerDefaultDialog(playerid);
                new model_array[] = {536, 575, 534, 567, 535, 566, 576, 412};
                return CreatePlayerVehicle( playerid, model_array[listitem]);
            }
            case 3441:
            {
                if (listitem > 12) return ShowPlayerDefaultDialog(playerid);
                new model_array[] = {568, 424, 573, 579, 400, 500, 444, 556, 557, 470, 489, 505, 495};
                return CreatePlayerVehicle(playerid, model_array[listitem]);
            }
            case 3442:
            {
                if (listitem > 17) return ShowPlayerDefaultDialog(playerid);
                new model_array[] = {416, 433, 431, 438, 437, 523, 427, 490, 528, 407, 544, 596, 598, 597, 599, 432, 601, 420};
                return CreatePlayerVehicle( playerid, model_array[ listitem ] );
            }
            case 3443:
            {
                if (listitem > 33) return ShowPlayerDefaultDialog(playerid);
                new model_array[] = {445, 504, 401, 518, 527, 542, 507, 562, 585, 419, 526, 604, 466, 492, 474, 546, 517, 410, 551, 516, 467, 600, 426, 436, 547, 405, 580, 560, 550, 549, 540, 491, 529, 421};
                return CreatePlayerVehicle( playerid, model_array[ listitem ] );
            }
            case 3444:
            {
                if (listitem > 19) return ShowPlayerDefaultDialog(playerid);
                new model_array[] = {602, 429, 496, 402, 541, 415, 589, 587, 565, 494, 502, 503, 411, 559, 603, 475, 506, 451, 558, 477};
                return CreatePlayerVehicle(playerid, model_array[listitem]);
            }
            case 3445:
            {
                if (listitem > 4) return ShowPlayerDefaultDialog(playerid);
                new model_array[] = {418, 404, 479, 458, 561};
                return CreatePlayerVehicle(playerid, model_array[listitem]);
            }
            case 3446:
            {
                if (listitem > 9) return ShowPlayerDefaultDialog(playerid);
                new model_array[] = {472, 473, 493, 595, 484, 430, 453, 452, 446, 454};
                return CreatePlayerVehicle(playerid, model_array[listitem]);
            }
            case 3447:
            {
                if (listitem > 11) return ShowPlayerDefaultDialog(playerid);
                new model_array[] = {435, 450, 591, 606, 607, 610, 569, 590, 584, 570, 608, 611};
                return CreatePlayerVehicle( playerid, model_array[ listitem ] );
            }
            case 3448:
            {
                if (listitem > 23) return ShowPlayerDefaultDialog(playerid);
                new model_array[] = {485, 537, 457, 483, 508, 532, 486, 406, 530, 538, 434, 545, 588, 571, 572, 423, 442, 428, 409, 574, 449, 525, 583, 539};
                return CreatePlayerVehicle(playerid, model_array[listitem]);
            }
            case 3449:
            {
                if (listitem > 5) return ShowPlayerDefaultDialog(playerid);
                new model_array[] = {441, 464, 465, 501, 564, 594};
                return CreatePlayerVehicle(playerid, model_array[listitem]);
            }
        }
    }
    if(dialogid == 5681) 


[Lixyde]

I tried it over 3 times and it doesn't work. The warning is on the same line the same thing.
...

[Lokii]

you can ignore it it wont cause any problems

[Lixyde]

But if i can fix the warrning there's no use of it. So please if anyone know how to fix it, tell me

[rfr]

try:

PHP код:

if (response)
{
    switch (dialogid)
    {
        case 3434:
        {
            switch (listitem)
            {
                case 0: ShowPlayerDialog(playerid, 3435, DIALOG_STYLE_LIST, "Airplanes", "Andromada\nAT-400\nBeagle\nCropduster\nDodo\nHydra\nNevada\nRustler\nShamal\nSkimmer\nStuntplane\nBack", "Select", "Cancel");
                case 1: ShowPlayerDialog(playerid, 3436, DIALOG_STYLE_LIST, "Helicopters", "Cargobob\nHunter\nLeviathan\nMaverick\nNews Maverick\nPolice Maverick\nRaindance\nSeasparrow\nSparrow\nBack", "Select", "Cancel");
                case 2: ShowPlayerDialog(playerid, 3437, DIALOG_STYLE_LIST, "Bikes", "BF-400\nBike\nBMX\nFaggio\nFCR-900\nFreeway\nMountain Bike\nNRG-500\nPCJ-600\nPizzaboy\nQuad\nSanchez\nWayfarer\nBack", "Select", "Cancel");
                case 3: ShowPlayerDialog(playerid, 3438, DIALOG_STYLE_LIST, "Convertibles", "Comet\nFeltzer\nStallion\nWindsor\nBack", "Select", "Cancel" );
                case 4: ShowPlayerDialog(playerid, 3439, DIALOG_STYLE_LIST, "Industrial", "Benson\nBobcat\nBurrito\nBoxville\nBoxburg\nCement Truck\nDFT-30\nFlatbed\nLinerunner\nMule\nNewsvan\nPacker\nPetrol Tanker\nPony\nRoadtrain\nRumpo\nSadler\nSadler Shit\nTopfun\nTractor\nTrashmaster\nUtility Van\nWalton\nYankee\nYosemite\nBack", "Select", "Cancel");
                case 5: ShowPlayerDialog(playerid, 3440, DIALOG_STYLE_LIST, "Lowriders", "Blade\nBroadway\nRemington\nSavanna\nSlamvan\nTahoma\nTornado\nVoodoo\nBack", "Select", "Cancel" );
                case 6: ShowPlayerDialog(playerid, 3441, DIALOG_STYLE_LIST, "Off Road", "Bandito\nBF Injection\nDune\nHuntley\nLandstalker\nMesa\nMonster\nMonster A\nMonster B\nPatriot\nRancher A\nRancher B\nSandking\nBack", "Select", "Cancel");
                case 7: ShowPlayerDialog(playerid, 3442, DIALOG_STYLE_LIST, "Public Service Vehicles", "Ambulance\nBarracks\nBus\nCabbie\nCoach\nCop Bike (HPV-1000)\nEnforcer\nFBI Rancher\nFBI Truck\nFiretruck\nFiretruck LA\nPolice Car (LSPD)\nPolice Car (LVPD)\nPolice Car (SFPD)\nRanger\nRhino\nS.W.A.T\nTaxi\nBack", "Select", "Cancel");
                case 8: ShowPlayerDialog(playerid, 3443, DIALOG_STYLE_LIST, "Saloons", "Admiral\nBloodring Banger\nBravura\nBuccaneer\nCadrona\nClover\nElegant\nElegy\nEmperor\nEsperanto\nFortune\nGlendale Shit\nGlendale\nGreenwood\nHermes\nIntruder\nMajestic\nManana\nMerit\nNebula\nOceanic\nPicador\nPremier\nPrevion\nPrimo\nSentinel\nStafford\nSultan\nSunrise\nTampa\nVincent\nVirgo\nWillard\nWashington\nBack", "Select", "Cancel");
                case 9: ShowPlayerDialog(playerid, 3444, DIALOG_STYLE_LIST, "Sport Vehicles", "Alpha\nBanshee\nBlista Compact\nBuffalo\nBullet\nCheetah\nClub\nEuros\nFlash\nHotring Racer\nHotring Racer A\nHotring Racer B\nInfernus\nJester\nPhoenix\nSabre\nSuper GT\nTurismo\nUranus\nZR-350\nBack", "Select", "Cancel");
                case 10: ShowPlayerDialog(playerid, 3445, DIALOG_STYLE_LIST, "Station Wagons", "Moonbeam\nPerenniel\nRegina\nSolair\nStratum\nBack", "Select", "Cancel");
                case 11: ShowPlayerDialog(playerid, 3446, DIALOG_STYLE_LIST, "Boats", "Coastguard\nDinghy\nJetmax\nLaunch\nMarquis\nPredator\nReefer\nSpeeder\nSquallo\nTropic\nBack", "Select", "Cancel");
                case 12: ShowPlayerDialog(playerid, 3447, DIALOG_STYLE_LIST, "Trailers", "Article Trailer\nArticle Trailer 2\nArticle Trailer 3\nBaggage Trailer A\nBaggage Trailer B\nFarm Trailer\nFreight Flat Trailer (Train)\nFreight Box Trailer (Train)\nPetrol Trailer\nStreak Trailer (Train)\nStairs Trailer\nUtility Trailer\nBack", "Select", "Cancel");
                case 13: ShowPlayerDialog(playerid, 3448, DIALOG_STYLE_LIST, "Unique Vehicles", "Baggage\nBrownstreak (Train)\nCaddy\nCamper\nCamper A\nCombine Harvester\nDozer\nDumper\nForklift\nFreight (Train)\nHotknife\nHustler\nHotdog\nKart\nMower\nMr Whoopee\nRomero\nSecuricar\nStretch\nSweeper\nTram\nTowtruck\nTug\nVortex\nBack", "Select", "Cancel");
                case 14: ShowPlayerDialog(playerid, 3449, DIALOG_STYLE_LIST, "RC Vehicles", "RC Bandit\nRC Baron\nRC Raider\nRC Goblin\nRC Tiger\nRC Cam\nBack", "Select", "Cancel");
            }
        }
        case 3435:
        {
            if ( listitem > 10 ) return ShowPlayerDefaultDialog(playerid);
            new model_array[] = {592, 577, 511, 512, 593, 520, 553, 476, 519, 460, 513};
            return CreatePlayerVehicle(playerid, model_array[listitem]);
        }
        case 3436:
        {
            if (listitem > 8) return ShowPlayerDefaultDialog(playerid);
            new model_array[] = { 548, 425, 417, 487, 488, 497, 563, 447, 469 };
            return CreatePlayerVehicle( playerid, model_array[ listitem ] );
        }
        case 3437:
        {
            if (listitem > 12) return ShowPlayerDefaultDialog(playerid);
            new model_array[] = {581, 509, 481, 462, 521, 463, 510, 522, 461, 448, 471, 468, 586};
            return CreatePlayerVehicle(playerid, model_array[listitem ]);
        }
        case 3438:
        {
            if (listitem > 3) return ShowPlayerDefaultDialog(playerid);
            new model_array[] = {480, 533, 439, 555};
            return CreatePlayerVehicle(playerid, model_array[listitem]);
        }
        case 3439:
        {
            if (listitem > 24) return ShowPlayerDefaultDialog(playerid);
            new model_array[] = {499, 422, 482, 498, 609, 524, 578, 455, 403, 414, 582, 443, 514, 413, 515, 440, 543, 605, 459, 531, 408, 552, 478, 456, 554};
            return CreatePlayerVehicle(playerid, model_array[listitem]);
        }
        case 3440:
        {
            if (listitem > 7) return ShowPlayerDefaultDialog(playerid);
            new model_array[] = {536, 575, 534, 567, 535, 566, 576, 412};
            return CreatePlayerVehicle( playerid, model_array[listitem]);
        }
        case 3441:
        {
            if (listitem > 12) return ShowPlayerDefaultDialog(playerid);
            new model_array[] = {568, 424, 573, 579, 400, 500, 444, 556, 557, 470, 489, 505, 495};
            return CreatePlayerVehicle(playerid, model_array[listitem]);
        }
        case 3442:
        {
            if (listitem > 17) return ShowPlayerDefaultDialog(playerid);
            new model_array[] = {416, 433, 431, 438, 437, 523, 427, 490, 528, 407, 544, 596, 598, 597, 599, 432, 601, 420};
            return CreatePlayerVehicle( playerid, model_array[ listitem ] );
        }
        case 3443:
        {
            if (listitem > 33) return ShowPlayerDefaultDialog(playerid);
            new model_array[] = {445, 504, 401, 518, 527, 542, 507, 562, 585, 419, 526, 604, 466, 492, 474, 546, 517, 410, 551, 516, 467, 600, 426, 436, 547, 405, 580, 560, 550, 549, 540, 491, 529, 421};
            return CreatePlayerVehicle( playerid, model_array[ listitem ] );
        }
        case 3444:
        {
            if (listitem > 19) return ShowPlayerDefaultDialog(playerid);
            new model_array[] = {602, 429, 496, 402, 541, 415, 589, 587, 565, 494, 502, 503, 411, 559, 603, 475, 506, 451, 558, 477};
            return CreatePlayerVehicle(playerid, model_array[listitem]);
        }
        case 3445:
        {
            if (listitem > 4) return ShowPlayerDefaultDialog(playerid);
            new model_array[] = {418, 404, 479, 458, 561};
            return CreatePlayerVehicle(playerid, model_array[listitem]);
        }
        case 3446:
        {
            if (listitem > 9) return ShowPlayerDefaultDialog(playerid);
            new model_array[] = {472, 473, 493, 595, 484, 430, 453, 452, 446, 454};
            return CreatePlayerVehicle(playerid, model_array[listitem]);
        }
        case 3447:
        {
            if (listitem > 11) return ShowPlayerDefaultDialog(playerid);
            new model_array[] = {435, 450, 591, 606, 607, 610, 569, 590, 584, 570, 608, 611};
            return CreatePlayerVehicle( playerid, model_array[ listitem ] );
        }
        case 3448:
        {
            if (listitem > 23) return ShowPlayerDefaultDialog(playerid);
            new model_array[] = {485, 537, 457, 483, 508, 532, 486, 406, 530, 538, 434, 545, 588, 571, 572, 423, 442, 428, 409, 574, 449, 525, 583, 539};
            return CreatePlayerVehicle(playerid, model_array[listitem]);
        }
        case 3449:
        {
            if (listitem > 5) return ShowPlayerDefaultDialog(playerid);
            new model_array[] = {441, 464, 465, 501, 564, 594};
            return CreatePlayerVehicle(playerid, model_array[listitem]);
        }
    }
}
if(dialogid == 5681)            return CreatePlayerVehicle(playerid, model_array[listitem]);
        }
        case 3449:
        {
            if (listitem > 5) return ShowPlayerDefaultDialog(playerid);
            new model_array[] = {441, 464, 465, 501, 564, 594};
            return CreatePlayerVehicle(playerid, model_array[listitem]);
        }
    }
}
if(dialogid == 5681) 


[Lixyde]

I think it won't working and this time the problem is on me.

The (dialogid == 5681) creates a new dialog it doesn't supposed to be in the dialog where i gave on Pastebin.

I just gave (dialogid == 5681) because there is the error. Not (dialogid == 5681) supposed to be here. Whit this i start a new dialog.

And what i see on your code is i need to use (dialogid == 5681) on the dialog for spawning cars.

(This time the error is on me), didn't note that (dialogid == 5681) is on other dialog its not in this dialog for spawning cars.

(If you understand me what im trying to say)

[RxErT]

Quote:

Originally Posted by Lixyde I think it won't working and this time the problem is on me. The (dialogid == 5681) creates a new dialog it doesn't supposed to be in the dialog where i gave on Pastebin. I just gave (dialogid == 5681) because there is the error. Not (dialogid == 5681) supposed to be here. Whit this i start a new dialog. And what i see on your code is i need to use (dialogid == 5681) on the dialog for spawning cars. (This time the error is on me), didn't note that (dialogid == 5681) is on other dialog its not in this dialog for spawning cars. (If you understand me what im trying to say)

Well he did it wrong, he duplicated it, just ignore it, it won't affect on your script.

[RogueDrifter]

Search : https://sampforum.blast.hk/showthread.php?tid=256961

[Lixyde]

The problem was that if(response) needs to be in the same line as if(dialogid