[V_LOPE]

I Have Problem with this Command.
When I go "On Duty" I can't go to "off Duty"

Код:

dcmd_AdminDuty(playerid, cmdtext[]){
       #pragma unused cmdtext
       if(IsPlayerLAdmin(playerid))
       {
        if(Admin[playerid] == 0)
        {
         Admin[playerid] = 1;
         new str[128], AdminName[28];
         GetPlayerName(playerid, AdminName, sizeof(AdminName));
         format(str, sizeof(str), "%s Is Now On Admin Duty!", AdminName);
         SendClientMessageToAll(COLOR_RED2, str);
         SetPlayerSkin(playerid,217);
         SetPlayerTeam(playerid, 6);
   	     SetPlayerColor(playerid, COLOR_PINK);
         SetPlayerHealth(playerid, 10000000);
         SetPlayerArmour(playerid, 10000000);
         GivePlayerWeapon(playerid, 9, 999999999);
         GivePlayerWeapon(playerid, 38,999999999);
	     GivePlayerWeapon(playerid, 24,999999999);
	     GivePlayerWeapon(playerid, 26,999999999);
	     GivePlayerWeapon(playerid, 31,999999999);
	     GivePlayerWeapon(playerid, 34,999999999);
	     GivePlayerWeapon(playerid, 32,999999999);
         GivePlayerWeapon(playerid, 42,999999999);
         GivePlayerWeapon(playerid, 16,999999999);
         label2[playerid] = Create3DTextLabel("Admin On Duty Do Not Attack",COLOR_GREEN,40.0,60.0,80.0,60.0,0);
         Attach3DTextLabelToPlayer(label2[playerid], playerid, 0.0, 0.0, 0.7);
      }
      else if(Admin[playerid] == 1)
      {
      Admin[playerid] = 0;
      new str[128], AdminName[28];
      GetPlayerName(playerid, AdminName, sizeof(AdminName));
      format(str, sizeof(str), "%s Is Now Off Admin Duty!", AdminName);
      SendClientMessageToAll(COLOR_RED2, str);
      ForceClassSelection(playerid);
      SetPlayerHealth(playerid, 0);
      } Erorr1
      }

When Im "On Duty" and I type that command Nothing Happens

[James Coral]

mhm i am not the scripter but, you can copy it from RAVENS RP GM

[Rob_Maate]

Try using a switch - just to iron out any bs

I can't really see anything wrong with your code imo

pawn Код:

switch (Admin[playerid])
{
  case 0:
  {
    Admin[playerid] = 1;
  }
  case 1:
  {
    Admin[playerid] = 0;
  }
}

[MdeRooy]

You do

pawn Код:

else if(Admin[playerid] == 1)

try doing

pawn Код:

else (Admin[playerid] == 1)

The script might assume there's another if clausule coming, and therefore ignores the else if, because there's no else in your script.

[[ABK]Antonio]

Quote:

Originally Posted by MdeRooy You do pawn Код: else if(Admin[playerid] == 1) try doing pawn Код: else (Admin[playerid] == 1) The script might assume there's another if clausule coming, and therefore ignores the else if, because there's no else in your script.

You would just do else { }

[MdeRooy]

No, you can't do if - else if, and then nothing. You can only do if - else or if - else if - else if- else.

[[ABK]Antonio]

Quote:

Originally Posted by MdeRooy No, you can't do if - else if, and then nothing. You can only do if - else or if - else if - else if- else.

that's NOT what I was saying...

you would do else { some shit here } instead of else (Admin[playerid] == 1)

[V_LOPE]

I Try This CODE
And Still Same Problem

PHP код:

 if(IsPlayerLAdmin(playerid))
       {
        if(Admin[playerid] == 0)
        {
         Admin[playerid] = 1;
         new str[128], AdminName[28];
         GetPlayerName(playerid, AdminName, sizeof(AdminName));
         format(str, sizeof(str), "%s Is Now On Admin Duty!", AdminName);
         SendClientMessageToAll(COLOR_RED2, str);
         SetPlayerSkin(playerid,217);
         SetPlayerTeam(playerid, 6);
            SetPlayerColor(playerid, COLOR_PINK);
         SetPlayerHealth(playerid, 10000000);
         SetPlayerArmour(playerid, 10000000);
         GivePlayerWeapon(playerid, 9, 999999999);
         GivePlayerWeapon(playerid, 38,999999999);
         GivePlayerWeapon(playerid, 24,999999999);
         GivePlayerWeapon(playerid, 26,999999999);
         GivePlayerWeapon(playerid, 31,999999999);
         GivePlayerWeapon(playerid, 34,999999999);
         GivePlayerWeapon(playerid, 32,999999999);
         GivePlayerWeapon(playerid, 42,999999999);
         GivePlayerWeapon(playerid, 16,999999999);
         label2[playerid] = Create3DTextLabel("Admin On Duty Do Not Attack",COLOR_GREEN,40.0,60.0,80.0,60.0,0);
         Attach3DTextLabelToPlayer(label2[playerid], playerid, 0.0, 0.0, 0.7);
         }
      }
      else
      {
      Admin[playerid] = 0;
      new str[128], AdminName[28];
      GetPlayerName(playerid, AdminName, sizeof(AdminName));
      format(str, sizeof(str), "%s Is Now Off Admin Duty!", AdminName);
      SendClientMessageToAll(COLOR_RED2, str);
      ForceClassSelection(playerid);
      SetPlayerHealth(playerid, 0);
      }
      return 1;
} 


[[ABK]Antonio]

use the switch Rob posted

[V_LOPE]

Quote:

Originally Posted by [ABK]Antonio use the switch Rob posted

Fixed