Проверка валидности игрока

[thegame]

Вечер добрый уважаемые форумчане, хотел попросить помощи подсказать, что из ниже приведенного кода будет правильным и оптимизированным в плане проверки валидности игрока:

// Вариант через сток
public CBasePlayer_TakeDamage_Pre(const this, pevInflictor, pevAttacker, Float:flDamage, bitsDamageType)
{
    if(!IsValidPlayer(pevAttacker) || pevAttacker != pevInflictor)
        return HC_CONTINUE;

    SetHookChainArg(4, ATYPE_FLOAT, flDamage * 1.1);
   
    return HC_CONTINUE;
}

stock bool:IsValidPlayer(const pPlayer)
    return bool:(1 <= pPlayer <= MaxClients);

// Вариант через is_entity от reapi
public CBasePlayer_TakeDamage_Pre(const this, pevInflictor, pevAttacker, Float:flDamage, bitsDamageType)
{
    if(!is_entity(pevAttacker) || pevAttacker != pevInflictor)
        return HC_CONTINUE;

    SetHookChainArg(4, ATYPE_FLOAT, flDamage * 1.1);
   
    return HC_CONTINUE;
}

 

[kucklovod]

!is_user_alive()

 

[fantom]

kucklovod, is_user_connected лучше использовать. Аттакер не обьязательно омжет быть живым (граната например)

 

[kucklovod]

fantom, он спросил про валидность игрока. окей, пусть is_user_connected

 

[d3m37r4]

Еще посмотри натив rg_is_player_can_takedamage и FPlayerCanTakeDamage

 

[voed]

В данном куске кода вообще не нужны никакие проверки
Всё зависит от ситуации. Например если у вас есть массив переменных для каждого игрока(размером в MaxClients, то достаточно проверить
(1 <= pPlayer <= MaxClients), чтоб индекс не выходил за границы массива. Если нужно получить приватдату игрока(get_pdata_*/get_member), то тут уже обязательно нужно is_user_connected, иначе возможны краши сервера. Для проверки на возможность нанесения урона можно использовать нативы, которые указал d3m37r4
В целом, не стоит заморачиваться и просто использовать везде is_user_connected. Исключения - Client_Pre/PostThink, AddToFullPack и еще пару экзотических форвардов, которые вызываются с большой частотой.

 

[hajimura]

Аттакер не обьязательно омжет быть живым (граната например)

pevInflictor же только может быть гранатой, нет?

 

[thegame]

shadow, у меня исключено повышение урона от гранаты pevAttacker != pevInflictor.

по этому мне подходит (1 <= pPlayer <= MaxClients) не выходит ли индекс за границы массива MaxClients.

Остался вопрос только как правильно через

дефайн

#define IsValidPlayer (%0) (1 <= (%0) <= MaxClients)

или сток
stock bool:IsValidPlayer(const pPlayer)
return bool:(1 <= pPlayer <= MaxClients);

 

[Minni]

Остался вопрос только как правильно через

дефайн

#define IsValidPlayer (%0) (1 <= (%0) <= MaxClients)

или сток
stock bool:IsValidPlayer(const pPlayer)
return bool:(1 <= pPlayer <= MaxClients);

Как тебе удобно. Имхо, макросом удобнее будет.

 

[thegame]

Minni, принял, спасибо)