Неверный алгоритм работы кода

[punkochel]

Уже вторые сутки не могу разобраться с ситуацией. При первичном входе на сервер, когда я сажусь в автомобиль и нажимаю "2", то выполняются функции garage и vehicle одновременно, хотя по идее должна выполниться только одна из них и вероятнее всего это garage (т.к. выполнение ее стоит впереди выполнения vehicle).
Я грешил сначала на неверный возврат одной из функций (думал что возвращает где-то false), но при повторном вызове OnPlayerKeyStateChange - все работает как нужно, а именно, выполняется функция garage, а vehicle уже не выполняется.

Это код вызова функций:

  Открыть/закрыть

public OnPlayerKeyStateChange(playerid, newkeys, oldkeys)
{
        if (garage_OnPlayerKeyStateChange(playerid, newkeys)) {
 		return 1;
 	}
	if (vehicle_OnPlayerKeyStateChange(playerid, newkeys)) {
		return 1;
	}
        //etc
	return 1;
}

Это код функции vehicle_OnPlayerKeyStateChange:

  Открыть/закрыть

vehicle_OnPlayerKeyStateChange(playerid, newkeys)
{
	if(newkeys == KEY_SUBMISSION && GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
	{
		return callcmd::engine(playerid);
	}
	return 0;
}

Это код выполнения функции garage_OnPlayerKeyStateChange:

  Открыть/закрыть

garage_OnPlayerKeyStateChange(playerid, newkeys)
{
	if(newkeys == KEY_SUBMISSION && GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
	{
		if(true)
		{
			for(new i; i < MAX_VEHICLE_PLAYER; i++)
			{
				if(true)
				{
					if(true)
					{
						printf("выполнено");
						return 1;
					}
					else
					{
						printf("не выполнено");
						return 1;
					}
				}
			}
		}
	}
	return 0;
}

Попробовал сделать без создания отдельных функций, а впихнул код в колбэк - все работает как нужно (в том числе и с первичного входа).
Код:

  Открыть/закрыть

public OnPlayerKeyStateChange(playerid, newkeys, oldkeys)
{
	if(newkeys == KEY_SUBMISSION && GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
	{
		if(true)
		{
			for(new i; i < MAX_VEHICLE_PLAYER; i++)
			{
				if(true)
				{
					if(true)
					{
						printf("выполнено");
						return 1;
					}
					else
					{
						printf("не выполнено");
						return 1;
					}
				}
			}
		}
	}
	if(newkeys == KEY_SUBMISSION && GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
	{
		return callcmd::engine(playerid);
	}
 	//etc
	return 1;
}

[L0ndl3m]

В команде "engine" есть возврат единицы? Есть предположение, что обе функции всегда возвращают ноль. Первое условие не сработало, и начала выполняться следующая проверка.

[Pa4enka]

if (0 == garage_OnPlayerKeyStateChange(playerid, newkeys)) {    return 1;}if (0 == vehicle_OnPlayerKeyStateChange(playerid, newkeys)) {    return 1;}

[L0ndl3m]

Pa4enka, смысл тогда проверять возвращаемое значений функций, если оно всегда возвращает 0?

[Pa4enka]

Pa4enka, смысл тогда проверять возвращаемое значений функций, если оно всегда возвращает 0?

0 будет возвращён, если проверка в стоке не пройдет. В ином случае, функция возвратит 1, исходя из его кода.

[L0ndl3m]

0 будет возвращён, если проверка в стоке не пройдет. В ином случае, функция возвратит 1, исходя из его кода.

Не факт, что

return callcmd::engine(playerid);

вернёт 1. Если команда возвращает нуль, то при любом раскладе "сток" вернёт 0, независимо от проверок.
Поэтому я и спросил у ТС:

В команде "engine" есть возврат единицы?

[Pa4enka]

Если команда возвращает нуль, то при любом раскладе "сток" вернёт 0, независимо от проверок.

Если команда возвращает 0, то она, как минимум, не сработает, разве нет?)
А кто-то в наше время возвращает 0? Ниразу такого не видел.

[DeimoS]

Если команда возвращает 0, то она, как минимум, не сработает, разве нет?)
А кто-то в наше время возвращает 0? Ниразу такого не видел.

Ты и нормальной работы с MySQL не увидишь в паблик-работах :)

А возврат нуля нужен для того, чтоб плагин не прекращал поиск команды. Например, это может быть полезно в FS, делая своеобразный перехват команды из мода (сначала будет срабатывать команды из скриптов, а потом уже из мода).

Так же возвращаемое значение передаётся в OnPlayerCommandPerformed, в параметр "result". При том, это не обязательно может быть 0 или 1. И это может быть полезно для создания различных сообщений об ошибке. То бишь, чтоб не плодить в каждой команде свой текст ошибки, сделать так:

CMD:pay(playerid, params[]){    new giveplayerid, money;    if(sscanf(params, "ri", giveplayerid, money))        return SendClientMessage(playerid, 0x888888FF, "Использование: /pay [ID игрока] [Сумма]");    if(giveplayerid == INVALID_PLAYER_ID)        return 2;    if(money < 1)        return 3;    GivePlayerMoney(playerid, money);    return 1;}   CMD:tpto(playerid, params[]){    new giveplayerid;    if(sscanf(params, "r", giveplayerid))        return SendClientMessage(playerid, 0x888888FF, "Использование: /tpto [ID игрока]");    if(giveplayerid == INVALID_PLAYER_ID)        return 2;    new Float:x, Float:y, Float:z;    GetPlayerPos(giveplayerid, x, y, z);    SetPlayerPos(playerid, x, y, z);    return 1;} public OnPlayerCommandPerformed(playerid, cmd[], params[], result, flags){    switch(result)    {        case 1:// Если команда найдена и успешно выполнена (в команде возвращена единица)            return 1;// Прекращаем поиск команды в других скриптах         case 2:// Если в команде возвращена двойка        {            SendClientMessage(playerid, 0x888888FF, "Ошибка: Указанный игрок не найден!");// Показываем сообщение            return 1;// Прекращаем поиск команды в других скриптах        }        case 3:// Если тройка        {            SendClientMessage(playerid, 0x888888FF, "Ошибка: Введено невалидное значение!");// Сообщение            return 1;// Прекращаем поиск команды в других скриптах        }    }    return 0;// Иначе "говорим" плагину продолжать поиск в других скриптах}

В идеале, конечно, код ошибок обёртывать в именные константы, например, через enum, и уже использовать их, чтоб не запоминать ворох чисел. Я числа указал лишь для наглядности :)


Ну а касаемо случая автора - 0 может возвращаться просто по незнанию (автор для красоты влепил "return false;" при сообщении об ошибке, например, что можно встретить во всяких паблик-модах).

[punkochel]

В команде "engine" есть возврат единицы? Есть предположение, что обе функции всегда возвращают ноль. Первое условие не сработало, и начала выполняться следующая проверка.

Без разницы что возвращает "engine", сама конструкция return callcmd::engine(playerid); вернет 1, как не крути. В engine есть возврат единицы, но это никак не повлияет на возврат значения из сток.

Ну а касаемо случая автора - 0 может возвращаться просто по незнанию (автор для красоты влепил "return false;" при сообщении об ошибке, например, что можно встретить во всяких паблик-модах).

Не увидел у себя в коде "return false". Единицу сток возвращает в случае успешного выполнения, иначе 0. В случае с проверкой, проверяется именно значение true (1), и если (false(0)), то переходим к следующему условию.
Пример конечно хороший, спасибо, но он никак не отвечает моему запросу.

Я не могу понять почему после авторизации, при вызове колбека KeyStateChange (нажатии на 2), обе функции vehicle и garage выполняются. По идее должно быть так, когда нажал на "2" - идет выполнение garage, если garage вернул 1, то vehicle уже не выполняется, так как колбэк вернул значение больше 0 (то бишь остановил дальнейшую работу). Но в моем случае после авторизации срабатываю обе функции.

[DeimoS]

Без разницы что возвращает "engine", сама конструкция return callcmd::engine(playerid); вернет 1, как не крути. В engine есть возврат единицы, но это никак не повлияет на возврат значения из сток.




Не увидел у себя в коде "return false". Единицу сток возвращает в случае успешного выполнения, иначе 0. В случае с проверкой, проверяется именно значение true (1), и если (false(0)), то переходим к следующему условию.
Пример конечно хороший, спасибо, но он никак не отвечает моему запросу.

Я не могу понять почему после авторизации, при вызове колбека KeyStateChange (нажатии на 2), обе функции vehicle и garage выполняются. По идее должно быть так, когда нажал на "2" - идет выполнение garage, если garage вернул 1, то vehicle уже не выполняется, так как колбэк вернул значение больше 0 (то бишь остановил дальнейшую работу). Но в моем случае после авторизации срабатываю обе функции.

Скинь весь код, связанный с проблемой, целиком: начиная с команды и заканчивая функциями.