SA-MP Forums Archive
Why cant compile this ? - 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: Why cant compile this ? (/showthread.php?tid=461916)


Why cant compile this ? - bustern - 04.09.2013

PHP код:
CMD:dance(playerid, params[])
{
if(id == 1)
            {
                SetPlayerSpecialAction(playerid,SPECIAL_ACTION_DANCE1);// Dancing Styles
            }
            else if(id == 2)
            {
                SetPlayerSpecialAction(playerid,SPECIAL_ACTION_DANCE2);// Dancing Styles
            }
            else if(id == 3)
            {
                SetPlayerSpecialAction(playerid,SPECIAL_ACTION_DANCE3);// Dancing Styles
            }
            else if(id == 4)
            {
                SetPlayerSpecialAction(playerid,SPECIAL_ACTION_DANCE4);// Dancing Styles
            }
            else
            {
                SendClientMessage(playerid, COLOR_WHITE, "USAGE: /an dance [ID]");
                SendClientMessage(playerid, COLOR_LIGHTBLUE, "1,2,3,4");
            }
return 1;
} 
I got:
PHP код:
C:\Users\Niko\Desktop\DeathMatch server by bustern\gamemodes\DeathMatch.pwn(823) : error 017: undefined symbol "id"
C:\Users\Niko\Desktop\DeathMatch server by bustern\gamemodes\DeathMatch.pwn(827) : error 017: undefined symbol "id"
C:\Users\Niko\Desktop\DeathMatch server by bustern\gamemodes\DeathMatch.pwn(831) : error 017: undefined symbol "id"
C:\Users\Niko\Desktop\DeathMatch server by bustern\gamemodes\DeathMatch.pwn(835) : error 017: undefined symbol "id" 


Re: Why cant compile this ? - TonyII - 04.09.2013

This will work
pawn Код:
CMD:dance(playerid, params[]) 
{ 
new id;
if(id == 1) 
            { 
                SetPlayerSpecialAction(playerid,SPECIAL_ACTION_DANCE1);// Dancing Styles 
            } 
            else if(id == 2) 
            { 
                SetPlayerSpecialAction(playerid,SPECIAL_ACTION_DANCE2);// Dancing Styles 
            } 
            else if(id == 3) 
            { 
                SetPlayerSpecialAction(playerid,SPECIAL_ACTION_DANCE3);// Dancing Styles 
            } 
            else if(id == 4) 
            { 
                SetPlayerSpecialAction(playerid,SPECIAL_ACTION_DANCE4);// Dancing Styles 
            } 
            else 
            { 
                SendClientMessage(playerid, COLOR_WHITE, "USAGE: /an dance [ID]"); 
                SendClientMessage(playerid, COLOR_LIGHTBLUE, "1,2,3,4"); 
            } 
return 1; 
}


Re: Why cant compile this ? - Threshold - 04.09.2013

No, it definitely will not.

You are creating a variable 'id', but you haven't given it a value. Thus giving it a default value of 0, meaning you will just get the error messages/syntax message every time you use the command, regardless of the parameters you enter. Also, you are missing a bracket which would just create more errors within the script.

The correct code for this would be:
With SSCANF include/plugin (RECOMMENDED):
pawn Код:
CMD:dance(playerid, params[])
{
    new selection;
    if(sscanf(params, "d", selection))
    {
        SendClientMessage(playerid, COLOR_WHITE, "USAGE: /dance [ID]");
        SendClientMessage(playerid, COLOR_LIGHTBLUE, "IDs: 1, 2, 3, 4");
        return 1;
    }
    switch(selection)
    {
        case 1: SetPlayerSpecialAction(playerid, SPECIAL_ACTION_DANCE1);
        case 2: SetPlayerSpecialAction(playerid, SPECIAL_ACTION_DANCE2);
        case 3: SetPlayerSpecialAction(playerid, SPECIAL_ACTION_DANCE3);
        case 4: SetPlayerSpecialAction(playerid, SPECIAL_ACTION_DANCE4);
    }
    return 1;
}
Without SSCANF Plugin/Include (NOT RECOMMENDED):
pawn Код:
CMD:dance(playerid, params[])
{
    if(!strlen(params))
    {
        SendClientMessage(playerid, COLOR_WHITE, "USAGE: /dance [ID]");
        SendClientMessage(playerid, COLOR_LIGHTBLUE, "IDs: 1, 2, 3, 4");
        return 1;
    }
    new selection = strval(params);
    switch(selection)
    {
        case 1: SetPlayerSpecialAction(playerid, SPECIAL_ACTION_DANCE1);
        case 2: SetPlayerSpecialAction(playerid, SPECIAL_ACTION_DANCE2);
        case 3: SetPlayerSpecialAction(playerid, SPECIAL_ACTION_DANCE3);
        case 4: SetPlayerSpecialAction(playerid, SPECIAL_ACTION_DANCE4);
        default:
        {
            SendClientMessage(playerid, COLOR_WHITE, "USAGE: /dance [ID]");
            SendClientMessage(playerid, COLOR_LIGHTBLUE, "IDs: 1, 2, 3, 4");
        }
    }
    return 1;
}