1. SA-MP Forums Archive
  2. SA-MP Scripting and Plugins
  3. Scripting Help

Threaded Mode
some errors
thefatshizms
High-roller
*****
Posts: 1,745
Threads: 252
Joined: Dec 2011
Reputation: 0
#1

13.05.2012, 01:45
hi guys im back lol with MORE errors


C:\Users\iphone\Desktop\0.3e\filterscripts\dialogs ss.pwn(71) : warning 209: function "OnPlayerCommandText" should return a value
C:\Users\iphone\Desktop\0.3e\filterscripts\dialogs ss.pwn(211) : warning 217: loose indentation
C:\Users\iphone\Desktop\0.3e\filterscripts\dialogs ss.pwn(305) : warning 202: number of arguments does not match definition
C:\Users\iphone\Desktop\0.3e\filterscripts\dialogs ss.pwn(321) : warning 217: loose indentation
C:\Users\iphone\Desktop\0.3e\filterscripts\dialogs ss.pwn(324) : error 030: compound statement not closed at the end of file (started at line 204)
Pawn compiler 3.2.3664 Copyright © 1997-2006, ITB CompuPhase


1 Error.

from line 204 to end of file:
pawn Код:
switch(listitem)// Checking which listitem was selected
            {
              case 0 : ShowPlayerDialog(playerid, DIALOG_WEPS, DIALOG_STYLE_LIST, "{FF0000}Weapons", "{00FF00}9mm\n{00FF00}Silenced 9mm\n{00FF00}Desert Eagle\n{00FF00}Tec-9\n{00FF00}Micro SMG\n{00FF00}SMG\n{00FF00}Shotgun\n{00FF00}Sawnoff Shotgun\n{00FF00}Combat Shotgun\n{00FF00}M4\n{00FF00}Ak47\n{00FF00}Sniper Rifle\n{00FF00}Molotov Cocktail\n{00FF00}Grenade\n{00FF00}Remote Explosives\n{00FF00}Mini Gun\n{00FF00}Rocket Launcher", "ok", "cancel");
              case 1 : ShowPlayerDialog(playerid, DIALOG_CARS, DIALOG_STYLE_LIST, "{FF0000}Cars", "{00FF00}Airplanes\n{00FF00}Helicopters\n{00FF00}Bikes\n{00FF00}Convertibles\n{00FF00}Industrial\n{00FF00}Lowriders\n{00FF00}Off Road\n{00FF00}Public Service Vehicles\n{00FF00}Saloons\n{00FF00}Sport Vehicles\n{00FF00}Station Wagons\n{00FF00}Boats\n{00FF00}Trailers\n{00FF00}Unique Vehicles", "Select", "Cancel" );
              case 2 : ShowPlayerDialog(playerid, DIALOG_USERS, DIALOG_STYLE_LIST, "{FF0000}Administrate Users", "{00FF00}Kick\n{00FF00}Ban\n{00FF00}Slap", "ok", "cancel");
              
            }
         if(dialogid == DIALOG_WEPS)
         {
         if(response)
         {
         switch(listitem)
         {
                case 0:
                {
                    GivePlayerWeapon(playerid, 22, 1000);

                }
                case 1:
                {


                    GivePlayerWeapon(playerid, 23, 1000);
                }
                case 2:
                {


                    GivePlayerWeapon(playerid, 24, 1000);
                }
                  case 3:
                {


                    GivePlayerWeapon(playerid, 32, 1000);
                }
                  case 4:
                {


                    GivePlayerWeapon(playerid, 28, 1000);
                }
                  case 5:
                {


                    GivePlayerWeapon(playerid, 29, 1000);
                }
                  case 6:
                {


                    GivePlayerWeapon(playerid, 25, 1000);
                }
                case 7:
                {


                    GivePlayerWeapon(playerid, 26, 1000);
                }
                case 8:
                {


                    GivePlayerWeapon(playerid, 27, 1000);
                }
                case 9:
                {


                    GivePlayerWeapon(playerid, 31, 1000);
                }
                case 10:
                {


                    GivePlayerWeapon(playerid, 30, 1000);
                }
                case 11:
                {


                    GivePlayerWeapon(playerid, 34, 1000);
                }
                case 12:
                {


                    GivePlayerWeapon(playerid, 18, 1000);
                }
                case 13:
                {


                    GivePlayerWeapon(playerid, 16, 1000);
                }
                case 14:
                {


                    GivePlayerWeapon(playerid, 39, 1000);
                    GivePlayerWeapon(playerid, 40);
                }
                case 15:
                {


                    GivePlayerWeapon(playerid, 38, 1000);
                }
                case 16:
                {


                    GivePlayerWeapon(playerid, 35, 1000);
            }
      }
 }
    return 1;
}
#endif
plz help
Find
Reply
ToiletDuck
Huge Clucker
****
Posts: 400
Threads: 52
Joined: Apr 2012
Reputation: 0
#2

13.05.2012, 02:39
Try this one
pawn Код:
switch(listitem)// Checking which listitem was selected
            {
              case 0 : ShowPlayerDialog(playerid, DIALOG_WEPS, DIALOG_STYLE_LIST, "{FF0000}Weapons", "{00FF00}9mm\n{00FF00}Silenced 9mm\n{00FF00}Desert Eagle\n{00FF00}Tec-9\n{00FF00}Micro SMG\n{00FF00}SMG\n{00FF00}Shotgun\n{00FF00}Sawnoff Shotgun\n{00FF00}Combat Shotgun\n{00FF00}M4\n{00FF00}Ak47\n{00FF00}Sniper Rifle\n{00FF00}Molotov Cocktail\n{00FF00}Grenade\n{00FF00}Remote Explosives\n{00FF00}Mini Gun\n{00FF00}Rocket Launcher", "ok", "cancel");
              case 1 : ShowPlayerDialog(playerid, DIALOG_CARS, DIALOG_STYLE_LIST, "{FF0000}Cars", "{00FF00}Airplanes\n{00FF00}Helicopters\n{00FF00}Bikes\n{00FF00}Convertibles\n{00FF00}Industrial\n{00FF00}Lowriders\n{00FF00}Off Road\n{00FF00}Public Service Vehicles\n{00FF00}Saloons\n{00FF00}Sport Vehicles\n{00FF00}Station Wagons\n{00FF00}Boats\n{00FF00}Trailers\n{00FF00}Unique Vehicles", "Select", "Cancel" );
              case 2 : ShowPlayerDialog(playerid, DIALOG_USERS, DIALOG_STYLE_LIST, "{FF0000}Administrate Users", "{00FF00}Kick\n{00FF00}Ban\n{00FF00}Slap", "ok", "cancel");
            }
if(dialogid == DIALOG_WEPS)
{
    if(response)
    {
        switch(listitem)
        {
            case 0:
            {
                GivePlayerWeapon(playerid, 22, 1000);
            }
            case 1:
            {
                GivePlayerWeapon(playerid, 23, 1000);
            }
            case 2:
            {
                GivePlayerWeapon(playerid, 24, 1000);
            }
            case 3:

            {
                GivePlayerWeapon(playerid, 32, 1000);
            }
            case 4:

            {
                GivePlayerWeapon(playerid, 28, 1000);
            }
            case 5:
            {
                GivePlayerWeapon(playerid, 29, 1000);
            }
            case 6:
            {
                GivePlayerWeapon(playerid, 25, 1000);
            }
            case 7:
            {
                GivePlayerWeapon(playerid, 26, 1000);
            }
            case 8:
            {
                GivePlayerWeapon(playerid, 27, 1000);
            }
            case 9:
            {
                GivePlayerWeapon(playerid, 31, 1000);
            }
            case 10:
            {
                GivePlayerWeapon(playerid, 30, 1000);
            }
            case 11:
            {
                GivePlayerWeapon(playerid, 34, 1000);
            }
            case 12:
            {
                GivePlayerWeapon(playerid, 18, 1000);
            }
            case 13:
            {
                GivePlayerWeapon(playerid, 16, 1000);
            }
            case 14:
            {
                GivePlayerWeapon(playerid, 39, 1000);
                GivePlayerWeapon(playerid, 40);
            }
            case 15:
            {
                GivePlayerWeapon(playerid, 38, 1000);
            }
            case 16:
            {
                GivePlayerWeapon(playerid, 35, 1000);
            }
        }

    return 1;
}
Find
Reply
kazekene
Little Clucker
**
Posts: 28
Threads: 0
Joined: Oct 2009
Reputation: 0
#3

13.05.2012, 02:43
Do it like this to fix the loose indentation


switch(listitem)// Checking which listitem was selected
{
{
case 0 : ShowPlayerDialog(playerid, DIALOG_WEPS, DIALOG_STYLE_LIST, "{FF0000}Weapons", "{00FF00}9mm\n{00FF00}Silenced 9mm\n{00FF00}Desert Eagle\n{00FF00}Tec-9\n{00FF00}Micro SMG\n{00FF00}SMG\n{00FF00}Shotgun\n{00FF00}Sawnoff Shotgun\n{00FF00}Combat Shotgun\n{00FF00}M4\n{00FF00}Ak47\n{00FF00}Sniper Rifle\n{00FF00}Molotov Cocktail\n{00FF00}Grenade\n{00FF00}Remote Explosives\n{00FF00}Mini Gun\n{00FF00}Rocket Launcher", "ok", "cancel");
case 1 : ShowPlayerDialog(playerid, DIALOG_CARS, DIALOG_STYLE_LIST, "{FF0000}Cars", "{00FF00}Airplanes\n{00FF00}Helicopters\n{00FF00}B ikes\n{00FF00}Convertibles\n{00FF00}Industrial\n{0 0FF00}Lowriders\n{00FF00}Off Road\n{00FF00}Public Service Vehicles\n{00FF00}Saloons\n{00FF00}Sport Vehicles\n{00FF00}Station Wagons\n{00FF00}Boats\n{00FF00}Trailers\n{00FF00}U nique Vehicles", "Select", "Cancel" );
case 2 : ShowPlayerDialog(playerid, DIALOG_USERS, DIALOG_STYLE_LIST, "{FF0000}Administrate Users", "{00FF00}Kick\n{00FF00}Ban\n{00FF00}Slap", "ok", "cancel");
}
if(dialogid == DIALOG_WEPS)
{
if(response)
{
switch(listitem)
{
case 0:
{
GivePlayerWeapon(playerid, 22, 1000);

}
case 1:
{


GivePlayerWeapon(playerid, 23, 1000);
}
case 2:
{


GivePlayerWeapon(playerid, 24, 1000);
}
case 3:
{


GivePlayerWeapon(playerid, 32, 1000);
}
case 4:
{


GivePlayerWeapon(playerid, 28, 1000);
}
case 5:
{


GivePlayerWeapon(playerid, 29, 1000);
}
case 6:
{


GivePlayerWeapon(playerid, 25, 1000);
}
case 7:
{


GivePlayerWeapon(playerid, 26, 1000);
}
case 8:
{


GivePlayerWeapon(playerid, 27, 1000);
}
case 9:
{


GivePlayerWeapon(playerid, 31, 1000);
}
case 10:
{


GivePlayerWeapon(playerid, 30, 1000);
}
case 11:
{


GivePlayerWeapon(playerid, 34, 1000);
}
case 12:
{


GivePlayerWeapon(playerid, 18, 1000);
}
case 13:
{


GivePlayerWeapon(playerid, 16, 1000);
}
case 14:
{


GivePlayerWeapon(playerid, 39, 1000);
GivePlayerWeapon(playerid, 40);
}
case 15:
{


GivePlayerWeapon(playerid, 38, 1000);
}
case 16:
{

GivePlayerWeapon(playerid, 35, 1000);
}
}
}
return 1;
}
#endif
Find
Reply
Faisal_khan
High-roller
*****
Posts: 1,826
Threads: 52
Joined: Feb 2012
Reputation: 0
#4

13.05.2012, 03:02
Here you are missing the amount of ammo to be given:
pawn Код:
case 14:
                {


                    GivePlayerWeapon(playerid, 39, 1000);
                    GivePlayerWeapon(playerid, 40);//here
                }
So change it to:
pawn Код:
case 14:
                {


                    GivePlayerWeapon(playerid, 39, 1000);
                    GivePlayerWeapon(playerid, 40, /*amount of ammo*/);
                }
The identation part was done with the help of DarcoBlue's project TidyPawn I use that

You were missing some brackets in the end line, your complete code:
pawn Код:
switch(listitem) {                            // Checking which listitem was selected
        case 0 : ShowPlayerDialog(playerid, DIALOG_WEPS, DIALOG_STYLE_LIST, "{FF0000}Weapons", "{00FF00}9mm\n{00FF00}Silenced 9mm\n{00FF00}Desert Eagle\n{00FF00}Tec-9\n{00FF00}Micro SMG\n{00FF00}SMG\n{00FF00}Shotgun\n{00FF00}Sawnoff Shotgun\n{00FF00}Combat Shotgun\n{00FF00}M4\n{00FF00}Ak47\n{00FF00}Sniper Rifle\n{00FF00}Molotov Cocktail\n{00FF00}Grenade\n{00FF00}Remote Explosives\n{00FF00}Mini Gun\n{00FF00}Rocket Launcher", "ok", "cancel");
        case 1 : ShowPlayerDialog(playerid, DIALOG_CARS, DIALOG_STYLE_LIST, "{FF0000}Cars", "{00FF00}Airplanes\n{00FF00}Helicopters\n{00FF00}Bikes\n{00FF00}Convertibles\n{00FF00}Industrial\n{00FF00}Lowriders\n{00FF00}Off Road\n{00FF00}Public Service Vehicles\n{00FF00}Saloons\n{00FF00}Sport Vehicles\n{00FF00}Station Wagons\n{00FF00}Boats\n{00FF00}Trailers\n{00FF00}Unique Vehicles", "Select", "Cancel" );
        case 2 : ShowPlayerDialog(playerid, DIALOG_USERS, DIALOG_STYLE_LIST, "{FF0000}Administrate Users", "{00FF00}Kick\n{00FF00}Ban\n{00FF00}Slap", "ok", "cancel");

    }
    if(dialogid == DIALOG_WEPS) {
        if(response) {
            switch(listitem) {
                case 0:
                {
                    GivePlayerWeapon(playerid, 22, 1000);

                }
                case 1:
                {

                    GivePlayerWeapon(playerid, 23, 1000);
                }
                case 2:
                {

                    GivePlayerWeapon(playerid, 24, 1000);
                }
                case 3:
                {

                    GivePlayerWeapon(playerid, 32, 1000);
                }
                case 4:
                {

                    GivePlayerWeapon(playerid, 28, 1000);
                }
                case 5:
                {

                    GivePlayerWeapon(playerid, 29, 1000);
                }
                case 6:
                {

                    GivePlayerWeapon(playerid, 25, 1000);
                }
                case 7:
                {

                    GivePlayerWeapon(playerid, 26, 1000);
                }
                case 8:
                {

                    GivePlayerWeapon(playerid, 27, 1000);
                }
                case 9:
                {

                    GivePlayerWeapon(playerid, 31, 1000);
                }
                case 10:
                {

                    GivePlayerWeapon(playerid, 30, 1000);
                }
                case 11:
                {

                    GivePlayerWeapon(playerid, 34, 1000);
                }
                case 12:
                {

                    GivePlayerWeapon(playerid, 18, 1000);
                }
                case 13:
                {

                    GivePlayerWeapon(playerid, 16, 1000);
                }
                case 14:
                {

                    GivePlayerWeapon(playerid, 39, 1000);
                    GivePlayerWeapon(playerid, 40, 1000);
                }
                case 15:
                {

                    GivePlayerWeapon(playerid, 38, 1000);
                }
                case 16:
                {

                    GivePlayerWeapon(playerid, 35, 1000);
                }
            }
        }
    }
    return 1;
}
And for the first error, you are missing return. Must be something like this:
pawn Код:
public OnPlayerCommandText(playerid, cmdtext[])
{
    if (strcmp("/butler", cmdtext, true, 10) == 0)
    {
        SendClientMessage(playerid, COLOR_RED, "You are a good friend of mine");
        return 1;
    }
    return 0;
}
Wish Ya luck.
Find
Reply
« Next Oldest | Next Newest »


Forum Jump:


Users browsing this thread: 1 Guest(s)
  1. SA-MP Forums Archive
  2. SA-MP Scripting and Plugins
  3. Scripting Help