SA-MP Forums Archive
loose identation, again - Printable Version

+- SA-MP Forums Archive (https://sampforum.blast.hk)
+-- Forum: SA-MP Scripting and Plugins (https://sampforum.blast.hk/forumdisplay.php?fid=8)
+--- Forum: Scripting Help (https://sampforum.blast.hk/forumdisplay.php?fid=12)
+--- Thread: loose identation, again (/showthread.php?tid=650768)


loose identation, again - enzulikeS - 06.03.2018

D:\server\gamemodes\ruleta.pwn(75 : warning 217: loose indentation

Код:
case 29..55:
                {
                        switch(step)
                        {
                                case 1: prohodtime = SetTimer("TextDrawProhod3", 450, true),TextDrawSetString(CasinoDraw[11], "Rolling roulette..");
                                case 10:
                                {
                    foreach(new i:Player)
                                        {
                            HideCasino(i);
                            TextDrawBoxColor(CasinoDraw[1], 0x000000AA),TextDrawBoxColor(CasinoDraw[2], 0xFF0000AA),TextDrawBoxColor(CasinoDraw[3], 0x000000AA),TextDrawBoxColor(CasinoDraw[4], 0xFF0000AA),TextDrawBoxColor(CasinoDraw[5], 0x000000AA),TextDrawBoxColor(CasinoDraw[6], 0xFF0000AA),TextDrawBoxColor(CasinoDraw[7], 0x000000AA);
                            ShowCasino(i);
                        	if(betplayer[i] == 14)
                        	{
                         		SendClientMessage(i,-1,"?? ???????? x2 ??????. ??????? ??????? ????.");
                           		GivePlayerMoney(i,betsumma[i]*2),PlayerPlaySound(i,43001,0.0,0.0,0.0);
							} else if(betplayer[i] == 10 || betplayer[i] == 4) SendClientMessage(i,-1,"?? ?????????. ??????? ??????? ????."),PlayerPlaySound(i,1085,0.0,0.0,0.0);
							betsumma[i] = 0,betplayer[i] = 0,PlayerTextDrawSetString(i,CasinoDrawPlayer[i][1], "BET: NONE");
       				}
       				
           			RouletteTime = 20,TotalBet = 0,RouletteStatus = 0,step = 0,step2 = 0,step3 = 0,podkrut = 0;
                    for(new i; i<3; i++) totalbet[i] = 0;
                    for(new i; i<3; i++) totalsumma[i] = 0;
                    KillTimer(playtime),KillTimer(prohodtime);
                    TextDrawSetString(CasinoDraw[11], "Win red");
                    TextDrawSetString(CasinoDraw[13], "0 users total 0$");
                    TextDrawSetString(CasinoDraw[15], "0 users total 0$");
                    TextDrawSetString(CasinoDraw[17], "0 users total 0$");
                    return true;
                                }
                        }
                }
                default: foreach(new i:Player) SendClientMessage(i,-1,"???? ??????????.");
		}
		return true;
    }
tryed all combinations, didnt found one to work

Re: loose identation, again - MrThomas - 06.03.2018

Try this:

PHP код:
case 29..55: 
                { 
                    switch(step) 
                    { 
                        case 1: prohodtime = SetTimer("TextDrawProhod3", 450, true),TextDrawSetString(CasinoDraw[11], "Rolling roulette.."); 
                        case 10: 
                    { 
                    foreach(new i:Player) 
                        { 
                            HideCasino(i); 
                            TextDrawBoxColor(CasinoDraw[1], 0x000000AA),TextDrawBoxColor(CasinoDraw[2], 0xFF0000AA),TextDrawBoxColor(CasinoDraw[3], 0x000000AA),TextDrawBoxColor(CasinoDraw[4], 0xFF0000AA),TextDrawBoxColor(CasinoDraw[5], 0x000000AA),TextDrawBoxColor(CasinoDraw[6], 0xFF0000AA),TextDrawBoxColor(CasinoDraw[7], 0x000000AA); 
                            ShowCasino(i); 
                            if(betplayer[i] == 14) 
                            { 
                                SendClientMessage(i,-1,"?? ???????? x2 ??????. ??????? ??????? ????."); 
                                GivePlayerMoney(i,betsumma[i]*2),PlayerPlaySound(i,43001,0.0,0.0,0.0); 
                            }
                            else if(betplayer[i] == 10 || betplayer[i] == 4) SendClientMessage(i,-1,"?? ?????????. ??????? ??????? ????."),PlayerPlaySound(i,1085,0.0,0.0,0.0); 
                            betsumma[i] = 0,betplayer[i] = 0,PlayerTextDrawSetString(i,CasinoDrawPlayer[i][1], "BET: NONE"); 
                        } 
                        
                    RouletteTime = 20,TotalBet = 0,RouletteStatus = 0,step = 0,step2 = 0,step3 = 0,podkrut = 0; 
                    for(new i; i<3; i++) totalbet[i] = 0; 
                    for(new i; i<3; i++) totalsumma[i] = 0; 
                    KillTimer(playtime),KillTimer(prohodtime); 
                    TextDrawSetString(CasinoDraw[11], "Win red"); 
                    TextDrawSetString(CasinoDraw[13], "0 users total 0$"); 
                    TextDrawSetString(CasinoDraw[15], "0 users total 0$"); 
                    TextDrawSetString(CasinoDraw[17], "0 users total 0$"); 
                    return true; 
                         } 
                    } 
                } 
                    default: foreach(new i:Player) SendClientMessage(i,-1,"???? ??????????."); 
        } 
        return true; 
    } 
You can fix it easily by removing all the extra spaces before the lines, did this fast but worked for me.

Re: loose identation, again - enzulikeS - 06.03.2018

fixed
i put ur code then 3 identations shown, fixed them, ty

Re: loose identation, again - OneDay - 06.03.2018

PHP код:
        case 29..55:
        {
            switch (step)
            {
                case 1:
                {
                    prohodtime = SetTimer("TextDrawProhod3", 450, true);
                    TextDrawSetString(CasinoDraw[11], "Rolling roulette..");
                }
                case 10:
                {
                    foreach (new i : Player)
                    {
                        HideCasino(i);
                        TextDrawBoxColor(CasinoDraw[1], 0x000000AA);
                        TextDrawBoxColor(CasinoDraw[2], 0xFF0000AA);
                        TextDrawBoxColor(CasinoDraw[3], 0x000000AA);
                        TextDrawBoxColor(CasinoDraw[4], 0xFF0000AA);
                        TextDrawBoxColor(CasinoDraw[5], 0x000000AA);
                        TextDrawBoxColor(CasinoDraw[6], 0xFF0000AA);
                        TextDrawBoxColor(CasinoDraw[7], 0x000000AA);
                        ShowCasino(i);
                        if (betplayer[i] == 14)
                        {
                            SendClientMessage(i, -1, "?? ???????? x2 ??????. ??????? ??????? ????.");
                            GivePlayerMoney(i, betsumma[i]*2);
                            PlayerPlaySound(i, 43001, 0.0, 0.0, 0.0);
                        }
                        else if (betplayer[i] == 10 || betplayer[i] == 4)
                        {
                            SendClientMessage(i, -1, "?? ?????????. ??????? ??????? ????.");
                            PlayerPlaySound(i, 1085, 0.0, 0.0, 0.0);
                        }
                        betsumma[i] = 0;
                        betplayer[i] = 0;
                        PlayerTextDrawSetString(i, CasinoDrawPlayer[i][1], "BET: NONE");
                    }
                    RouletteTime = 20;
                    TotalBet = 0;
                    RouletteStatus = 0;
                    step = 0;
                    step2 = 0;
                    step3 = 0;
                    podkrut = 0;
                    for (new i; i < 3; i++)
                    {
                        totalbet[i] = 0;
                        totalsumma[i] = 0;
                    }
                    KillTimer(playtime);
                    KillTimer(prohodtime);
                    TextDrawSetString(CasinoDraw[11], "Win red");
                    TextDrawSetString(CasinoDraw[13], "0 users total 0$");
                    TextDrawSetString(CasinoDraw[15], "0 users total 0$");
                    TextDrawSetString(CasinoDraw[17], "0 users total 0$");
                    return true;
                }
            }
        }
        default:
        {
            foreach (new i : Player)
            {
                SendClientMessage(i, -1, "???? ??????????.");
            }
        }
    }
    return true;
} 
do you hate spacing?

Re: loose identation, again - enzulikeS - 06.03.2018

Quote:
Originally Posted by OneDay
Посмотреть сообщение
PHP код:
        case 29..55:
        {
            switch (step)
            {
                case 1:
                {
                    prohodtime = SetTimer("TextDrawProhod3", 450, true);
                    TextDrawSetString(CasinoDraw[11], "Rolling roulette..");
                }
                case 10:
                {
                    foreach (new i : Player)
                    {
                        HideCasino(i);
                        TextDrawBoxColor(CasinoDraw[1], 0x000000AA);
                        TextDrawBoxColor(CasinoDraw[2], 0xFF0000AA);
                        TextDrawBoxColor(CasinoDraw[3], 0x000000AA);
                        TextDrawBoxColor(CasinoDraw[4], 0xFF0000AA);
                        TextDrawBoxColor(CasinoDraw[5], 0x000000AA);
                        TextDrawBoxColor(CasinoDraw[6], 0xFF0000AA);
                        TextDrawBoxColor(CasinoDraw[7], 0x000000AA);
                        ShowCasino(i);
                        if (betplayer[i] == 14)
                        {
                            SendClientMessage(i, -1, "?? ???????? x2 ??????. ??????? ??????? ????.");
                            GivePlayerMoney(i, betsumma[i]*2);
                            PlayerPlaySound(i, 43001, 0.0, 0.0, 0.0);
                        }
                        else if (betplayer[i] == 10 || betplayer[i] == 4)
                        {
                            SendClientMessage(i, -1, "?? ?????????. ??????? ??????? ????.");
                            PlayerPlaySound(i, 1085, 0.0, 0.0, 0.0);
                        }
                        betsumma[i] = 0;
                        betplayer[i] = 0;
                        PlayerTextDrawSetString(i, CasinoDrawPlayer[i][1], "BET: NONE");
                    }
                    RouletteTime = 20;
                    TotalBet = 0;
                    RouletteStatus = 0;
                    step = 0;
                    step2 = 0;
                    step3 = 0;
                    podkrut = 0;
                    for (new i; i < 3; i++)
                    {
                        totalbet[i] = 0;
                        totalsumma[i] = 0;
                    }
                    KillTimer(playtime);
                    KillTimer(prohodtime);
                    TextDrawSetString(CasinoDraw[11], "Win red");
                    TextDrawSetString(CasinoDraw[13], "0 users total 0$");
                    TextDrawSetString(CasinoDraw[15], "0 users total 0$");
                    TextDrawSetString(CasinoDraw[17], "0 users total 0$");
                    return true;
                }
            }
        }
        default:
        {
            foreach (new i : Player)
            {
                SendClientMessage(i, -1, "???? ??????????.");
            }
        }
    }
    return true;
} 
do you hate spacing?
getting mindfucked when doing it )

fixed, ty

Re: loose identation, again - Sew_Sumi - 06.03.2018

Uncrustify handles pawn. It fixes the tab errors, and can find missing braces as well.