How to fix this warning
#1

pawn Код:
if(dialogid == DSB)
    {
        if(response)
        {
        if(PlayerInfo[playerid][pConnectTime] < 2) return SendClientMessage(playerid, COLOR_GRAD2, "You can't use this as you're currently restricted from possessing weapons!");
            switch(listitem) //HERE'S THE WARNING
            {
            case 0: GivePlayerValidWeapon(playerid, 24, 99999);
            case 1: GivePlayerValidWeapon(playerid, 27, 99999);
            case 2: GivePlayerValidWeapon(playerid, 31, 99999);
            case 3: GivePlayerValidWeapon(playerid, 34, 99999);
            case 4: GivePlayerValidWeapon(playerid, 29, 99999);
            case 5: SetPlayerArmourEx(playerid, 100);
            case 6: SetPlayerHealth(playerid, 100);
            }
        }
    }
Quote:

warning 217: loose indentation

Reply
#2

<removed>
Reply
#3

That's the worse way of doing that. Please indent them with TAB and SHIFT+TAB.
Reply
#4

<removed>

or

pawn Код:
if(dialogid == DSB)
    {
        if(response)
        {
            if(PlayerInfo[playerid][pConnectTime] < 2) return SendClientMessage(playerid, COLOR_GRAD2, "You can't use this as you're currently restricted from possessing weapons!");
            switch(listitem)
            {
                case 0: GivePlayerValidWeapon(playerid, 24, 99999);
                case 1: GivePlayerValidWeapon(playerid, 27, 99999);
                case 2: GivePlayerValidWeapon(playerid, 31, 99999);
                case 3: GivePlayerValidWeapon(playerid, 34, 99999);
                case 4: GivePlayerValidWeapon(playerid, 29, 99999);
                case 5: SetPlayerArmourEx(playerid, 100);
                case 6: SetPlayerHealth(playerid, 100);
            }
        }
    }
Reply
#5

Pragma tabsize isn't really recommended at all!

How do you fix the problem? Intend your code properly

Intended Code and No need to use <removed>
pawn Код:
if(dialogid == DSB)
{
    if(response)
    {
        if(PlayerInfo[playerid][pConnectTime] < 2) return SendClientMessage(playerid, COLOR_GRAD2, "You can't use this as you're currently restricted from possessing weapons!");
        switch(listitem)
        {
            case 0: GivePlayerValidWeapon(playerid, 24, 99999);
            case 1: GivePlayerValidWeapon(playerid, 27, 99999);
            case 2: GivePlayerValidWeapon(playerid, 31, 99999);
            case 3: GivePlayerValidWeapon(playerid, 34, 99999);
            case 4: GivePlayerValidWeapon(playerid, 29, 99999);
            case 5: SetPlayerArmourEx(playerid, 100);
            case 6: SetPlayerHealth(playerid, 100);
        }
    }
}
Reply
#6

Fixed, I have another warning
Quote:

warning 219: local variable "playerid" shadows a variable at a preceding level

pawn Код:
CMD:fixmyvw(playerid, params[])
{
    new string[128], playerid; //Warn here.
    if(IsPlayerConnected(playerid))
    {
        GetPlayerPos(playerid, StopaniFloats[playerid][0], StopaniFloats[playerid][1], StopaniFloats[playerid][2]);
        SetTimerEx("StopaniTimer", 5000, 0, "d", playerid);
        SendClientMessage (playerid, COLOR_YELLOW, "Do not move for 5 seconds to have your VW Fixed!");
        {
        PlayerInfo[playerid][pVW] =  0;
        SetPlayerVirtualWorld(playerid,  0);
        format(string, sizeof(string), "You have fix your virtual world.");
        SendClientMessage(playerid, COLOR_GRAD2, string);
        }
    }
    return 1;
}
Reply
#7

pawn Код:
new string[128], playerid;
to
pawn Код:
new string[128];
playerid is already defined, can't you see? CMD:fixmyvw(playerid, params[])
Reply
#8

Quote:
Originally Posted by pds2012
Посмотреть сообщение
Pragma tabsize isn't really recommended at all!

How do you fix the problem? Intend your code properly

Intended Code and No need to use <removed>
pawn Код:
if(dialogid == DSB)
{
    if(response)
    {
        if(PlayerInfo[playerid][pConnectTime] < 2) return SendClientMessage(playerid, COLOR_GRAD2, "You can't use this as you're currently restricted from possessing weapons!");
        switch(listitem)
        {
            case 0: GivePlayerValidWeapon(playerid, 24, 99999);
            case 1: GivePlayerValidWeapon(playerid, 27, 99999);
            case 2: GivePlayerValidWeapon(playerid, 31, 99999);
            case 3: GivePlayerValidWeapon(playerid, 34, 99999);
            case 4: GivePlayerValidWeapon(playerid, 29, 99999);
            case 5: SetPlayerArmourEx(playerid, 100);
            case 6: SetPlayerHealth(playerid, 100);
        }
    }
}
You see this... The second code...

Quote:
Originally Posted by xganyx
Посмотреть сообщение
<removed>

or

pawn Код:
if(dialogid == DSB)
    {
        if(response)
        {
            if(PlayerInfo[playerid][pConnectTime] < 2) return SendClientMessage(playerid, COLOR_GRAD2, "You can't use this as you're currently restricted from possessing weapons!");
            switch(listitem)
            {
                case 0: GivePlayerValidWeapon(playerid, 24, 99999);
                case 1: GivePlayerValidWeapon(playerid, 27, 99999);
                case 2: GivePlayerValidWeapon(playerid, 31, 99999);
                case 3: GivePlayerValidWeapon(playerid, 34, 99999);
                case 4: GivePlayerValidWeapon(playerid, 29, 99999);
                case 5: SetPlayerArmourEx(playerid, 100);
                case 6: SetPlayerHealth(playerid, 100);
            }
        }
    }
Why do you -rep me... Bitch....
Reply
#9

Quote:
Originally Posted by xganyx
Посмотреть сообщение
You see this... The second code...



Why do you -rep me... Bitch....
I didn't -rep you by the way, you could ask the admin, when you -rep me, I don't care it's alright, but you forced me to do it again cuz of your behaviour.
Reply


Forum Jump:


Users browsing this thread: 1 Guest(s)