[LeeXian99]

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.

[HurtLocker]

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

[FalconX]

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

[Gamer_007]

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

[[Ro]DuReX]

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

[LeeXian99]

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;
}

[LeeXian99]

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 >.<)