SA-MP Forums Archive
warning 225: unreachable code - 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: warning 225: unreachable code (/showthread.php?tid=133238)



warning 225: unreachable code - Torran - 11.03.2010

pawn Код:
CMD:makeadmin(playerid, params[])
{
  new targetid;
  new adminlevel;

  if (PlayerInfo[playerid][AdminLevel] < 5) return SendClientMessage(playerid, COLOR_RED, "Error ~ You are not an administrator with the required level");
  if (sscanf(params, "ud", targetid, adminlevel)) return SendClientMessage(playerid, COLOR_RED, "Usage ~ /makeadmin [PlayerID/PartOfName] [AdminLevel]");
  if (!IsPlayerConnected(targetid)) return SendClientMessage(playerid, COLOR_RED, "Error ~ Player not Connected");
  if (adminlevel == PlayerInfo[targetid][AdminLevel]) return SendClientMessage(playerid, COLOR_RED, "Error ~ Player is already this Level");
  else {
  }
  return 1;
}
Код:
C:\Users\Torran\Desktop\tRoleplay\filterscripts\tadmin.pwn(193) : warning 225: unreachable code
Pawn compiler 3.2.3664	 	 	Copyright © 1997-2006, ITB CompuPhase


1 Warning.
Line:
pawn Код:
return 1;
Anyone know?


Re: warning 225: unreachable code - westre - 11.03.2010

Код:
else {
return 1;
}
or just delete the whole else statement if you're not going to use it.


Re: warning 225: unreachable code - Torran - 11.03.2010

Im going to use it i just always leave the else empty until i edit later,
Anyway i fixed it!
It just needed a code in between the brackets..

pawn Код:
CMD:makeadmin(playerid, params[])
{
  new targetid;
  new adminlevel;

  if (PlayerInfo[playerid][AdminLevel] < 5) return SendClientMessage(playerid, COLOR_RED, "Error ~ You are not an administrator with the required level");
  if (sscanf(params, "ud", targetid, adminlevel)) return SendClientMessage(playerid, COLOR_RED, "Usage ~ /makeadmin [PlayerID/PartOfName] [AdminLevel]");
  if (!IsPlayerConnected(targetid)) return SendClientMessage(playerid, COLOR_RED, "Error ~ Player not Connected");
  if (adminlevel == PlayerInfo[targetid][AdminLevel]) return SendClientMessage(playerid, COLOR_RED, "Error ~ Player is already this Level");
  else {
    print("Makeadmin command");
  }
  return 1;
}



Re: warning 225: unreachable code - kLx - 11.03.2010

if you use return on the other checks, you don't need to use else {
just delete it, and write the code.

SORRY DID'NT NOTICED U FIXED IT.


Re: warning 225: unreachable code - FireScript - 07.01.2012

Use:

PHP код:
CMD:makeadmin(playeridparams[])
{
    new 
targetid;
      new 
adminlevel;
      if (
PlayerInfo[playerid][AdminLevel] < 5) return SendClientMessage(playeridCOLOR_RED"Error ~ You are not an administrator with the required level");
      if (
sscanf(params"ud"targetidadminlevel)) return SendClientMessage(playeridCOLOR_RED"Usage ~ /makeadmin [PlayerID/PartOfName] [AdminLevel]");
      if (!
IsPlayerConnected(targetid)) return SendClientMessage(playeridCOLOR_RED"Error ~ Player not Connected");
      if (
adminlevel == PlayerInfo[targetid][AdminLevel]) return SendClientMessage(playeridCOLOR_RED"Error ~ Player is already this Level");
      else
     {
        print(
"Makeadmin command");
        return 
1;
      }

I am Brazilian, Reputation if I helped ^ ^


Re: warning 225: unreachable code - Snowman12 - 28.01.2012

pawn Код:
CMD:makeadmin(playerid, params[])
{
    new targetid;
      new adminlevel;
      if (PlayerInfo[playerid][AdminLevel] < 5) return SendClientMessage(playerid, COLOR_RED, "Error ~ You are not an administrator with the required level");
      if (sscanf(params, "ud", targetid, adminlevel)) return SendClientMessage(playerid, COLOR_RED, "Usage ~ /makeadmin [PlayerID/PartOfName] [AdminLevel]");
      if (!IsPlayerConnected(targetid)) return SendClientMessage(playerid, COLOR_RED, "Error ~ Player not Connected");
      if (adminlevel == PlayerInfo[targetid][AdminLevel]) return SendClientMessage(playerid, COLOR_RED, "Error ~ Player is already this Level");
      else
     {
        print("Makeadmin command");
        return 1;  // Would clearly fail.
      }
//return 1 here or BAM SERVER: Unknown command.
}



Re: warning 225: unreachable code - 2KY - 28.01.2012

Quote:
Originally Posted by FireScript
Посмотреть сообщение
Use:

PHP код:
CMD:makeadmin(playeridparams[])
{
    new 
targetid;
      new 
adminlevel;
      if (
PlayerInfo[playerid][AdminLevel] < 5) return SendClientMessage(playeridCOLOR_RED"Error ~ You are not an administrator with the required level");
      if (
sscanf(params"ud"targetidadminlevel)) return SendClientMessage(playeridCOLOR_RED"Usage ~ /makeadmin [PlayerID/PartOfName] [AdminLevel]");
      if (!
IsPlayerConnected(targetid)) return SendClientMessage(playeridCOLOR_RED"Error ~ Player not Connected");
      if (
adminlevel == PlayerInfo[targetid][AdminLevel]) return SendClientMessage(playeridCOLOR_RED"Error ~ Player is already this Level");
      else
     {
        print(
"Makeadmin command");
        return 
1;
      }

I am Brazilian, Reputation if I helped ^ ^
You bumped a two year old thread. HOW?!