Короткие вопросы [Вопрос:Ответ]

[DeimoS]

Макрос на существование игрока:

#define AvaliablePlayer(%0,%1) if(%1  == INVALID_PLAYER_ID || !IsPlayerConnected(%1)) \
    return SendClientMessage(%0, COLOR_RED, ">> {FFFFFF}Не верный id!");  

Эмм, а какой смысл проверять значение переменной на совпадение с INVALID_PLAYER_ID, если IsPlayerConnected и так вернёт 0, если в неё попадёт любой ID, которого нет на сервере (в том числе и INVALID_PLAYER_ID). Никогда не понимал эту проверку.

У меня есть один вариант - ты используешь sscanf со спецификатором "u". Хотя "INVALID_PLAYER_ID" sscanf вернёт только в том случае, когда игрок не найден на сервере (в спецификатор "u" вшита проверка на то, находится ли игрок на сервере). Других вариантов не знаю. Нужно смотреть конкретный код, где появляются данные проблемы. Бага такого в SA-MP замечено никем не было, а значит это кривой код

[Nexius_Tailer]

А разница?И так и так должно работать, не?!

Как вариант. Хотя если работало до этого, тогда, возможно, проблема в другом.

[ziggi]

Эмм, а какой смысл проверять значение переменной на совпадение с INVALID_PLAYER_ID, если IsPlayerConnected и так вернёт 0, если в неё попадёт любой ID, которого нет на сервере (в том числе и INVALID_PLAYER_ID). Никогда не понимал эту проверку.

Да, смысла нет, лучше сделать такую проверку:

stock FIX_IsPlayerConnected(playerid)
{
    return 0 <= playerid < MAX_PLAYERS && IsPlayerConnected(playerid);
}
#if defined _ALS_IsPlayerConnected
    #undef IsPlayerConnected
#else
    #define _ALS_IsPlayerConnected
#endif

#define IsPlayerConnected FIX_IsPlayerConnected  

Ведь, как известно, IsPlayerConnected работает только с двумя младшими байтами переданного значения, что в ряде случаях (при числах больше, чем 2^16 - 1) может приводить к ложным результатам.

[Profyan]

У меня есть один вариант - ты используешь sscanf со спецификатором "u". Хотя "INVALID_PLAYER_ID" sscanf вернёт только в том случае, когда игрок не найден на сервере (в спецификатор "u" вшита проверка на то, находится ли игрок на сервере). Других вариантов не знаю. Нужно смотреть конкретный код, где появляются данные проблемы. Бага такого в SA-MP замечено никем не было, а значит это кривой код

Да, использую u. Но вначале я думал,что sscanf сам все проверит, а как оказалось нет. Было очень много выходов за пределы массива, поэтому сделал такой костыль.

Спасибо ziggi, воспользуюсь твоим решением!

[vovandolg]

Да, смысла нет, лучше сделать такую проверку:

stock FIX_IsPlayerConnected(playerid) 
{ 
    return 0 <= playerid < MAX_PLAYERS && IsPlayerConnected(playerid); 
} 
#if defined _ALS_IsPlayerConnected 
    #undef IsPlayerConnected 
#else 
    #define _ALS_IsPlayerConnected 
#endif 
 
#define IsPlayerConnected FIX_IsPlayerConnected  

Ведь, как известно, IsPlayerConnected работает только с двумя младшими байтами переданного значения, что в ряде случаях (при числах больше, чем 2^16 - 1) может приводить к ложным результатам.

А что он будет за числа возвращать в такой проверке, ID игроков? прежнею цифру 1?

[ziggi]

А что он будет за числа возвращать в такой проверке, ID игроков? прежнею цифру 1?

1 если игрок на сервере и 0, если нет.

[DeimoS]

А что он будет за числа возвращать в такой проверке, ID игроков? прежнею цифру 1?

Всё просто же:

return 0 <= playerid < MAX_PLAYERS && IsPlayerConnected(playerid);  

Тут возвращается результат условия. Условия возвращают либо истину (1), либо ложь (0).
Так как стоит оператор "&&", учитываются результаты обоих условий, то бишь, например, если представить, что возврат имеет такой вид:

return 0 <= playerid < 10 && IsPlayerConnected(playerid);  

То:

• Если в функцию попадёт значение 15 (и такой игрок будет на сервере), условие 0 <= playerid < 10 вернёт 0 (ибо 15 больше 10), но IsPlayerConnected вернёт 1. Но так как в условии имеется оператор "&&", срабатывает компьютерная логика, при которой, если хоть один из аргументов возвращает "ложь" (0/false), то возвращается "ложь".
  (для оператора "||" (или) это, например, звучит так: если хоть один из аргументов возвращает "истина" (1/true), то возвращается "истина". То бишь, совершенно противоположная логика. Это, кстати, должны на информатике в школе рассказывать)
• Если в функцию попадёт значение 5 (но такого игрока не будет на сервере), условие 0 <= playerid < 10 вернёт 1, но функция IsPlayerConnected вернёт 0, из-за чего результат опять будет равен 0.
• Если же в функцию попадёт значение, например, 3 (и такой игрок будет на сервере), и условие 0 <= playerid < 10, и функция IsPlayerConnected вернут 1, из-за чего результат уже будет равен 1.

В этой строке (return ...) нет ничего сверхсложного. Это, фактически, то же самое, что и:

if((0 <= playerid < MAX_PLAYERS && IsPlayerConnected(playerid)) == true) return 1;
else  return 0;  

Просто записано в более компактном виде.

[vovandolg]

new count = 0,
    rand = Iter_Random(Player);
foreach(new i: Player)
{
    if(team{i} == 1) count ++;
    if(count == Iter_Count(Player)) SetupFG(rand);
}  

Знаю что этот отрывок делает, но хочу узнать как он работает, на английском переводил так и не понял,
вопрос в общем: что делают/как работают Iter_Random и Iter_Count в данном коде?

[DeimoS]

Iter_Random возвращает рандомное значение из указанного в нём итератора (массива, в которое записаны ID игроков, в данном случае)
Iter_Count возвращает количество записанных данных в итератор (количество заполненных ячеек в массиве-итераторе Player)
Всё связано с foreach

[vovandolg]

Iter_Random возвращает рандомное значение из указанного в нём итератора (массива, в которое записаны ID игроков, в данном случае)
Iter_Count возвращает количество записанных данных в итератор (количество заполненных ячеек в массиве-итераторе Player)
Всё связано с foreach

А в этом отрывке кода может быть такое что рандом выберет не зависимо состоял игрок в тиме 1 или состоял в другой любой?
Во обще есть смысл такого перебора? или можно было просто вот так сделать как ниже:

SetupFG(Iter_Random(Player));