SA-MP Forums Archive
IsPlayerAdmin... - 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: IsPlayerAdmin... (/showthread.php?tid=431477)


IsPlayerAdmin... - LeeXian99 - 18.04.2013

Well, I just scripted this, but it appears a lot of errors:
pawn Код:
case 898:
        {
        if(response)
            {
            if(IsPlayerAdmin(playerid));
                {
                switch(listitem)
                    {
                    case 0:GivePlayerWeapon(playerid, 35, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a rocket launcher.");
                    case 1:GivePlayerWeapon(playerid, 36, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a heat seeker launcher.");
                    case 2:GivePlayerWeapon(playerid, 37, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a flamethrower.");
                    case 3:GivePlayerWeapon(playerid, 38, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a minigun.");
                    case 4:GivePlayerWeapon(playerid, 39, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a satchel charge.");
                    }
                }
            else return cmd_weapons(playerid,"params") && SendClientMessage(playerid, 0xFF000000, "[ERROR]: You are not a RCON admin");
        }
Errors:
Код:
C:\Documents and Settings\User\Desktop\Scripts\filterscripts\weapon.pwn(272) : error 036: empty statement
C:\Documents and Settings\User\Desktop\Scripts\filterscripts\weapon.pwn(273) : warning 217: loose indentation
C:\Documents and Settings\User\Desktop\Scripts\filterscripts\weapon.pwn(283) : warning 217: loose indentation
C:\Documents and Settings\User\Desktop\Scripts\filterscripts\weapon.pwn(283) : error 029: invalid expression, assumed zero
C:\Documents and Settings\User\Desktop\Scripts\filterscripts\weapon.pwn(283) : warning 215: expression has no effect
C:\Documents and Settings\User\Desktop\Scripts\filterscripts\weapon.pwn(283) : error 001: expected token: ";", but found "return"
C:\Documents and Settings\User\Desktop\Scripts\filterscripts\weapon.pwn(287) : error 002: only a single statement (or expression) can follow each "case"
C:\Documents and Settings\User\Desktop\Scripts\filterscripts\weapon.pwn(287) : warning 215: expression has no effect
C:\Documents and Settings\User\Desktop\Scripts\filterscripts\weapon.pwn(288) : warning 209: function "OnDialogResponse" should return a value
Pawn compiler 3.2.3664	 	 	Copyright © 1997-2006, ITB CompuPhase


4 Errors.


Re: IsPlayerAdmin... - HurtLocker - 18.04.2013

Give us the whole public OnDialogResponse + the command which opens the dialog.

Re: IsPlayerAdmin... - FalconX - 18.04.2013

pawn Код:
cmd_weapons(playerid,"params"), SendClientMessage(playerid, 0xFF000000, "[ERROR]: You are not a RCON admin");
&& (AND) and || (OR) are used in comparisons operators (Control Structures). Which are not used normally. They can be used in if/else statements etc..

You may use "," (comma) or ";" (semi-colon) instead of &&.

Hope this helps.

Regards FalconX

Re: IsPlayerAdmin... - Gamer_007 - 18.04.2013

pawn Код:
switch(listitem)
                    {
                    case 0:GivePlayerWeapon(playerid, 35, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a rocket launcher.");
                    case 1:GivePlayerWeapon(playerid, 36, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a heat seeker launcher.");
                    case 2:GivePlayerWeapon(playerid, 37, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a flamethrower.");
                    case 3:GivePlayerWeapon(playerid, 38, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a minigun.");
                    case 4:GivePlayerWeapon(playerid, 39, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a satchel charge.");
                    }
                }
Why u used '&&' ? I wont use that so i wont suggest.I will suggest this

You need to use braces when you have more then one statement

try this
pawn Код:
switch(listitem)
{
case 0:
{
GivePlayerWeapon(playerid, 35, 9999);
SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a rocket launcher.");
}
case 1:
{
GivePlayerWeapon(playerid, 36, 9999);
SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a heat seeker launcher.");
}
case 2:
{
GivePlayerWeapon(playerid, 37, 9999); 
SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a flamethrower.");
}
case 3:
{
GivePlayerWeapon(playerid, 38, 9999); 
SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a minigun.");
}
case 4:
{
GivePlayerWeapon(playerid, 39, 9999);
SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a satchel charge.");
}
}
Dont forget to do indent or You will get Loose indentation warning

Re: IsPlayerAdmin... - [Ro]DuReX - 18.04.2013

The problem was at the IsPlayerAdmin. Read the title before posting. He put ";" after if(IsPlayerAdmin(playerid))

Try this:
pawn Код:
case 898:
        {
        if(response)
            {
            if(IsPlayerAdmin(playerid))
                {
                switch(listitem)
                    {
                    case 0:GivePlayerWeapon(playerid, 35, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a rocket launcher.");
                    case 1:GivePlayerWeapon(playerid, 36, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a heat seeker launcher.");
                    case 2:GivePlayerWeapon(playerid, 37, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a flamethrower.");
                    case 3:GivePlayerWeapon(playerid, 38, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a minigun.");
                    case 4:GivePlayerWeapon(playerid, 39, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a satchel charge.");
                    }
                }
            else return cmd_weapons(playerid,"params") && SendClientMessage(playerid, 0xFF000000, "[ERROR]: You are not a RCON admin");
        }
EDIT: ****** posted when I was writing this post

Re: IsPlayerAdmin... - LeeXian99 - 18.04.2013

Okay.
pawn Код:
public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
    switch(dialogid)
    {
        case 888:
        {
        if(response)
            {
            switch(listitem)
                {
                case 0:return cmd_melee(playerid,"params");
                case 1:return cmd_pistols(playerid,"params");
                case 2:return cmd_shotguns(playerid,"params");
                case 3:return cmd_machineguns(playerid,"params");
                case 4:return cmd_assaultrifles(playerid,"params");
                case 5:return cmd_snipers(playerid,"params");
                case 6:return cmd_explosive(playerid,"params");
                case 7:return cmd_special(playerid,"params");
                case 8:return cmd_sextoys(playerid,"params");
                case 9:return cmd_heavy(playerid,"params");
                }
            }
        }
        case 889:
        {
        if(response)
            {
            switch(listitem)
                {
                case 0:return GivePlayerWeapon(playerid, 2, 1) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Golf Club.");
                case 1:return GivePlayerWeapon(playerid, 3, 1) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Nightstick.");
                case 2:return GivePlayerWeapon(playerid, 4, 1) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Knife.");
                case 3:return GivePlayerWeapon(playerid, 5, 1) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Baseball Bat.");
                case 4:return GivePlayerWeapon(playerid, 6, 1) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Shovel.");
                case 5:return GivePlayerWeapon(playerid, 7, 1) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Pool Cue.");
                case 6:return GivePlayerWeapon(playerid, 8, 1) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Katana.");
                case 7:return GivePlayerWeapon(playerid, 9, 1) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Chainsaw.");
                }
            }
        else return cmd_weapons(playerid,"params");
        }
        case 890:
        {
        if(response)
            {
            switch(listitem)
                {
                case 0:return GivePlayerWeapon(playerid, 22, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Colt 45.");
                case 1:return GivePlayerWeapon(playerid, 23, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Slienced Colt 45.");
                case 2:return GivePlayerWeapon(playerid, 24, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Desert Eagle.");
                }
            }
        else return cmd_weapons(playerid,"params");
        }
        case 891:
        {
        if(response)
            {
            switch(listitem)
                {
                case 0:return GivePlayerWeapon(playerid, 25, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Shotgun.");
                case 1:return GivePlayerWeapon(playerid, 26, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Sawnoff Shotgun.");
                case 2:return GivePlayerWeapon(playerid, 27, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Combat Shotgun.");
                }
            }
        else return cmd_weapons(playerid,"params");
        }
        case 892:
        {
        if(response)
            {
            switch(listitem)
                {
                case 0:return GivePlayerWeapon(playerid, 28, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Micro Uzi.");
                case 1:return GivePlayerWeapon(playerid, 29, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a SMG.");
                case 2:return GivePlayerWeapon(playerid, 32, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Tec-9.");
                }
            }
        else return cmd_weapons(playerid,"params");
        }
        case 893:
        {
        if(response)
            {
            switch(listitem)
                {
                case 0:return GivePlayerWeapon(playerid, 30, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got an AK-47.");
                case 1:return GivePlayerWeapon(playerid, 31, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a M4.");
                }
            }
        else return cmd_weapons(playerid,"params");
        }
        case 894:
        {
        if(response)
            {
            switch(listitem)
                {
                case 0:return GivePlayerWeapon(playerid, 33, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a country rifle.");
                case 1:return GivePlayerWeapon(playerid, 34, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Sniper rifle.");
                }
            }
        else return cmd_weapons(playerid,"params");
        }
        case 895:
        {
        if(response)
            {
            switch(listitem)
                {
                case 0:GivePlayerWeapon(playerid, 16, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Grenade.");
                case 1:GivePlayerWeapon(playerid, 17, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Tear Gas.");
                case 2:GivePlayerWeapon(playerid, 18, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Molotov.");
                }
            }
        else return cmd_weapons(playerid,"params");
        }
        case 896:
        {
        if(response)
            {
            switch(listitem)
                {
                case 0:GivePlayerWeapon(playerid, 14, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Flower.");
                case 1:GivePlayerWeapon(playerid, 15, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Cane.");
                case 2:GivePlayerWeapon(playerid, 41, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Spray Can.");
                case 3:GivePlayerWeapon(playerid, 42, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Fire Extinguisher.");
                case 4:GivePlayerWeapon(playerid, 43, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Camera.");
                case 5:GivePlayerWeapon(playerid, 46, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a Parachute.");
                }
            }
        else return cmd_weapons(playerid,"params");
        }
        case 897:
        {
        if(response)
            {
            switch(listitem)
                {
                case 0:GivePlayerWeapon(playerid, 30, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a double-ended dildo.");
                case 1:GivePlayerWeapon(playerid, 30, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a dildo.");
                case 2:GivePlayerWeapon(playerid, 30, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a vibrator.");
                case 3:GivePlayerWeapon(playerid, 30, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a sliver vibrator.");
                }
            }
        else return cmd_weapons(playerid,"params");
        }
        case 898:
        {
        if(response)
                {
                switch(listitem)
                    {
                    case 0:GivePlayerWeapon(playerid, 35, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a rocket launcher.");
                    case 1:GivePlayerWeapon(playerid, 36, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a heat seeker launcher.");
                    case 2:GivePlayerWeapon(playerid, 37, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a flamethrower.");
                    case 3:GivePlayerWeapon(playerid, 38, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a minigun.");
                    case 4:GivePlayerWeapon(playerid, 39, 9999) && SendClientMessage(playerid, 0xFF9900AA, "[INFO]: You've got a satchel charge.");
                    }
                }
            else return cmd_weapons(playerid,"params");
        }
                
     }
    return 1;
}


Re: IsPlayerAdmin... - LeeXian99 - 18.04.2013

Quote:
Originally Posted by ******
Посмотреть сообщение
Since no-one has actually mentioned the real issue:



Remove the semicolon.
******, thanks for the help, but I tested and the code is above me. It just doesn't respond after clicking one of them.(Sorry for double posting >.<)