[Jing_Chan]

pawn Код:

if(strcmp(cmd, "/mask", true) == 0)
    {
        if(PlayerInfo[playerid][pMember] != 8 || PlayerInfo[playerid][pMember] != 11)
        {
            SendClientMessage(playerid, COLOR_GREY, "   You are not a member of the Hitman Agency / Bratva !");
            return 1;
        }
        if(PlayerInfo[playerid][pMask] == 0)
        {
            PlayerInfo[playerid][pMask] = 1;
            foreach (Player, i)
            {
                if(IsPlayerConnected(i))
                {
                    ShowPlayerNameTagForPlayer(i, playerid, 0);
                }
            }
            strmid(sendername, PlayerRPName(playerid), 0, MAX_PLAYER_NAME);
            format(string, sizeof(string), "* %s has put a mask on.", sendername);
            ProxDetector(15.0, playerid, string, COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE);
        }
        else if(PlayerInfo[playerid][pMask] == 1)
        {
            PlayerInfo[playerid][pMask] = 0;
            foreach (Player, i)
            {
                if(IsPlayerConnected(i))
                {
                    ShowPlayerNameTagForPlayer(i, playerid, 1);
                }
            }
            strmid(sendername, PlayerRPName(playerid), 0, MAX_PLAYER_NAME);
            format(string, sizeof(string), "* %s has put their mask away.", sendername);
            ProxDetector(15.0, playerid, string, COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE);
        }
    }

When in either faction 8, or faction 11 /mask still doesn't work. What's wrong?

[Schneider]

You are checking if the player not 8 OR not 11, so if for example the player is in faction 8, he is not in faction 11, so he will still get the error-message.

The way to go is to check if the player is not from 8 AND not from 11, so use && instead of ||

[Jing_Chan]

Quote:

Originally Posted by Schneider You are checking if the player not 8 OR not 11, so if for example the player is in faction 8, he is not in faction 11, so he will still get the error-message. The way to go is to check if the player is not from 8 AND not from 11, so use && instead of ||

I'll give it a go and get back to you, thanks.

edit: worked ty