Команда / Проверка

[66]

Всем привет, суть проблемы такая. При вводе команды - пишет "Вам недоступна данная команда".

Проверка:

CMD:guninvite(playerid, params[])
{
	if(PI[playerid][pMember] != TEAM_MERIA || PI[playerid][pRank] < 7) return SendClientMessage(playerid, COLOR_DARKORANGE, YouCanNot);
    if(PI[playerid][pMember] != TEAM_SFMAYOR || PI[playerid][pMember] != TEAM_SFMAYOR || PI[playerid][pRank] < 9) return SendClientMessage(playerid, COLOR_DARKORANGE, YouCanNot);

П.с. с проверкой косяк, но никак не могу сообразить, как исправить.

Отредактировано 6 марта, 2019 пользователем HAPKOMAH
Причина: Исправил опечатку

[525]

@HAPKOMAH, 

if(!IsACop(playerid) && PI[playerid][pMember] != TEAM_MERIA && PI[playerid][pMember] != TEAM_SFMAYOR && PI[playerid][pMember] != TEAM_LVMAYOR) return SendClientMessage(playerid, COLOR_DARKORANGE, YouCanNot);

 

[268]

Ты состоишь в фракции мэрия и твой ранг равен 7 или больше?

Ты состоишь в фракции TEAM_SFMAYOR(не понял, что это, ищи в моде) и твой ранг равен 9 или больше?

У тебя кстати повторяется проверка.

CMD:guninvite(playerid, params[])
{
	if(PI[playerid][pMember] != TEAM_MERIA && PI[playerid][pRank] < 7 || PI[playerid][pMember] != TEAM_SFMAYOR && PI[playerid][pRank] < 9) 
      	return SendClientMessage(playerid, COLOR_DARKORANGE, YouCanNot);

Попробуй так, возможно выдаст ошибку при компиляции, у тебя проверка идёт так: если одно условие выполняется, то второе не будет точно, попробуй так. Я только заменил, теперь будет так:

Если(игрок не состоит в мэрии и его ранг меньше 7 или если игрок не состоит в фракции sfmayor и его ранг меньше 9) выводит сообщение.

 

UPD: протести, потому что я запутался

Отредактировано 6 марта, 2019 пользователем SCRIPTMAN

[66]

Кхм, смысл такой:

Игрок состоящей в "Мэрии 1" если имеет свыше 7 ранга - может использовать команда.

Если игрок состоит в "Мэрии 2" или "Мэрии 3", то использовать команду он может только с 9 ранга. Надеюсь понятно объяснил.

 

Протестил, все работает, за что Вам огромное спасибо, но есть еще вопросик:

Почему в проверки нет TEAM_LVMAOYR, но для нее команда так-же работает? 

 

Примечание: Кстати, в своей теме опять нашел ошибку, код который я дал немного неверный, вот исправленная строчка ( Красным я пометил исправление ):

    if(PI[playerid][pMember] != TEAM_SFMAYOR || PI[playerid][pMember] != TEAM_LVMAOYR|| PI[playerid][pRank] < 9) return SendClientMessage(playerid, COLOR_DARKORANGE, YouCanNot);

То есть у меня два раза было "TEAM_SFMAOYR", а должно быть "TEAM_SFMAYOR" и "TEAM_LVMAOYR"

[268]

@HAPKOMAH 

	if(PI[playerid][pMember] != TEAM_MERIA && PI[playerid][pRank] < 7 || PI[playerid][pMember] != TEAM_SFMAYOR && PI[playerid][pRank] < 9 || PI[playerid][pMember] != TEAM_LVMAYOR && PI[playerid][pRank] < 9) 
      	return SendClientMessage(playerid, COLOR_DARKORANGE, YouCanNot);

Вот так попробуй

[525]

2 минуты назад, HAPKOMAH сказал:

Почему в проверки нет TEAM_LVMAOYR, но для нее команда так-же работает? 

 

Скорее всего, вы как лидер фракции TEAM_LVMAOYR пытаетесь использовать эту команду. А в проверке дважды проверяется на члена фракции (не лидера) - опечатка, стало быть.

Да и гораздо удобнее будет с использованием соответствующих функций (причем не только для данной команды):

stock bool:IsPlayerInTeam(playerid, team, minrank = 0) // ID игрока, ID фракции, минимальный ранг (необязательно)
{
	// если игрок не лидер и не подчиненный, выходим из функции
	if(gPlayer[playerid][pLeader] != team && gPlayer[playerid][pMember] != team) {
		return false;
	}
	
	// если игрок подчиненный и его ранг ниже необходимого, выходим
	if(gPlayer[playerid][pRank] < minrank && gPlayer[playerid][pLeader] == 0) { 
		return false;
	}
	
	return true; // в остальных случаях возвращаем успех
}

CMD:guninvite(playerid, params[])
{
	if(!IsPlayerInTeam(playerid, TEAM_MERIA, 7) && !IsPlayerInTeam(playerid, TEAM_SFMAYOR, 9)) {
		return SendClientMessage(playerid, COLOR_DARKORANGE, YouCanNot);
	}
	
	// здесь код команды
}

 

[66]

@SCRIPTMAN  Пока писал уже сам допер, все работает, спасибо) 

1 час назад, SCRIPTMAN сказал:

@HAPKOMAH 

	if(PI[playerid][pMember] != TEAM_MERIA && PI[playerid][pRank] < 7 || PI[playerid][pMember] != TEAM_SFMAYOR && PI[playerid][pRank] < 9 || PI[playerid][pMember] != TEAM_LVMAYOR && PI[playerid][pRank] < 9) 
      	return SendClientMessage(playerid, COLOR_DARKORANGE, YouCanNot);

Вот так 

 

[66]

@SCRIPTMAN

Тэк-с, поторопился я. Тут еще одна команда есть, проверку в ней сделал по твоему принципу, но при ее вводе пишет "Вам недоступная данная команда"

CMD:tazer(playerid, params[])
{
    if(!IsACop(playerid) || PI[playerid][pMember] != TEAM_MERIA || PI[playerid][pMember] != TEAM_SFMAYOR || PI[playerid][pMember] != TEAM_LVMAYOR) return SendClientMessage(playerid, COLOR_DARKORANGE, YouCanNot);

 

[268]

@HAPKOMAH а ты в какой сейчас фракции состоишь?

[66]

@SCRIPTMAN Пробовал состоять в Полиции, и в Мэрии

[268]

Скинь команду

[66]

@SCRIPTMAN 

CMD:tazer(playerid, params[])
{
    if(!IsACop(playerid) || PI[playerid][pMember] != TEAM_MERIA || PI[playerid][pMember] != TEAM_SFMAYOR || PI[playerid][pMember] != TEAM_LVMAYOR) return SendClientMessage(playerid, COLOR_DARKORANGE, YouCanNot);
	else if(sscanf(params, "u", params[0])) return SendClientMessage(playerid, COLOR_LIME, "Информация: {ffffff}/tazer [id игрока]");
 	else if(!IsPlayerConnected(params[0])) return SendClientMessage(playerid, COLOR_DARKORANGE, PlayerIsNotOnline);
	else if(PlayerLogged[params[0]] == false) return SendClientMessage(playerid, COLOR_DARKORANGE, PlayerIsNotAuthorized);
	else if(playerid == params[0]) return SendClientMessage(playerid, COLOR_DARKORANGE, "Нельзя себя ударить электрошокером");
	else if(!ProxDetectorS(8.0, playerid, params[0])) return SendClientMessage(playerid, COLOR_DARKORANGE, "Вы далеко друг от друга");
	new string[128];
	if(PI[playerid][pSex] == 1) format(string, sizeof(string), "ударил электрошоком %s'a", GetName(params[0]));
	else format(string, sizeof(string), "ударила электрошоком %s'a", GetName(params[0]));
	MeAction(playerid, string);
	ClearAnimations(params[0], 1);
	ApplyAnimation(params[0], "CRACK", "crckdeth2", 4.0, 0, 1, 1, 1, 0, 1);
	SetTimerEx("TazerOff", 15000, 0, "i", params[0]);
	SetPVarInt(params[0], "Tazer", 1);
	TogglePlayerControllable(params[0], 0);
	return 1;
}

 

[268]

@HAPKOMAH а если так, я просто не помню уже этот сток:

 if(IsACop(playerid) || PI[playerid][pMember] != TEAM_MERIA || PI[playerid][pMember] != TEAM_SFMAYOR || PI[playerid][pMember] != TEAM_LVMAYOR) return SendClientMessage(playerid, COLOR_DARKORANGE, YouCanNot);

Хотя не должно работать

[66]

@SCRIPTMAN  Не-а, не работает

[66]

@odosenok Хм, работает, благодарю