Player_TakeDamage (fix error_log)

Написал не большой код кое-как, не могли бы помочь исправить error, появляется ошибка если становлюсь на место trigger_hurt и получаю урон от него. Так же если есть желания сделать доброе дело, помогите оптимизировать, как бы вы правильно написали=)) спасибо:)

L 06/23/2017 - 03:34:33: [ZP] Invalid Player (84)
L 06/23/2017 - 03:34:33: [AMXX] Displaying debug trace (plugin "zombie_bers.amxx")
L 06/23/2017 - 03:34:33: [AMXX] Run time error 10: native error (native "zp_get_user_zombie")
L 06/23/2017 - 03:34:33: [AMXX] [0] zombie_bers.sma::Player_TakeDamage (line 28)

Код:

public Player_TakeDamage(victim, inflicator, attacker, Float:damage, damage_type, bitsDamage)
{
if(!is_user_alive(attacker) || !is_user_alive(victim)) // сначала лучше проверить на alive атакующего и жертвы. Alive проверит на валидность игроков, поэтому проверку на коннект можно пропустить
    return;

    if(  (zp_get_user_zombie(attacker)) && (zp_get_user_zombie_class(attacker) == g_zclassidqex) && (bitsDamage & DMG_SLASH  ))
    {
        if(pev(attacker, pev_button) & IN_ATTACK || pev(attacker, pev_button) & IN_ATTACK2)
        {
            user_slap(victim, 0, 0)
        }
    }
}

DeMNiX, данная функция не работает(
user_slap(victim, 0, 0)

хорошо, логи есть какие-либо?

Код:

user_slap(victim, 1, 0)

Нету, попробовал разными способами ударять на всякий случай.. логи не появились(

Код:

public Player_TakeDamage(victim, inflicator, attacker, Float:damage, damage_type, bitsDamage)
{
if(!is_user_alive(attacker) || !is_user_alive(victim))
    return HAM_IGNORED;
 
    if(  (zp_get_user_zombie(attacker)) && (zp_get_user_zombie_class(attacker) == g_zclassidqex) && (bitsDamage & (DMG_SLASH | DMG_CLUB)  ))
    {
            user_slap(victim, 0, 0)
    }
    
     return HAM_IGNORED;
}

А сам по себе slap работает? Пример:

@nevermind
я думаю дело в dmgbits
в принципе я могу убрать его вовсе, но если сторонние плагины будут атаковать - жертва будет пинаться, что не есть хорошо
потому что у зм основное оружие это нож. Если появится какая-нибудь базука ( как на немезисе ), т.е. это уже совсем другая история

Не работает, нечего не изменилось(

Код:

public Player_TakeDamage(victim, inflicator, attacker, Float:damage, damage_type, bitsDamage)

регистрацию покажи

neverminde, Способ, который я в 1 посту писал работает.


Да работает.

DeMNiX, RegisterHam(Ham_TakeDamage, "player", "Player_TakeDamage")

Регистер в public plugin_precache()

Код:

public Player_TakeDamage(victim, inflicator, attacker, Float:damage, damage_type, bitsDamage)
{
if(!is_user_alive(attacker) || !is_user_alive(victim))
    return HAM_IGNORED;
  
    if(  (zp_get_user_zombie(attacker)) && (zp_get_user_zombie_class(attacker) == g_zclassidqex) )
    {
            user_slap(victim, 0, 0)
            return HAM_HANDLED;
    }
     
     return HAM_IGNORED;
}

попробуй так, потом скажешь работает или нет

DeMNiX, да, так работает.

Код:

 
public Player_TakeDamage(victim, inflicator, attacker, Float:damage, damage_type, bitsDamage)
{
if(!is_user_alive(attacker) || !is_user_alive(victim))
    return HAM_IGNORED;
   
    if(  (zp_get_user_zombie(attacker)) && (zp_get_user_zombie_class(attacker) == g_zclassidqex) && (bitsDamage & (DMG_NEVERGIB | DMG_BULLET) ) )
    {
            user_slap(victim, 0, 0)
            return HAM_HANDLED;
    }
      
     return HAM_IGNORED;
}

должно работать, спасибо D34

DeMNiX, а так не работает((

Код:

#define DMG_KNIFE (DMG_NEVERGIB | DMG_BULLET)

public Player_TakeDamage(victim, inflicator, attacker, Float:damage, damage_type, bitsDamage)
{
if(!is_user_alive(attacker) || !is_user_alive(victim))
    return HAM_IGNORED;
    
    if(  (zp_get_user_zombie(attacker)) && (zp_get_user_zombie_class(attacker) == g_zclassidqex) && (bitsDamage & DMG_KNIFE ) )
    {
            user_slap(victim, 0, 0)
            return HAM_HANDLED;
    }
       
     return HAM_IGNORED;
}

а по факту должна

DeMNiX, то же не работает..

Такой вопрос, скажи пожалуйста, лучше будет если поставить проверку, чтобы на немезиду это не распространялось или не ставить? ( в плане нагрузки на сервер, так то пофиг мне стукает немезида при убийстве или нет :))

В принципе данный вариант нормальный) Чем он плох?
Код:

public Player_TakeDamage(victim, inflicator, attacker, Float:damage, damage_type, bitsDamage)
{
if(!is_user_alive(attacker) || !is_user_alive(victim))
    return HAM_IGNORED;
   
    if(  (zp_get_user_zombie(attacker)) && (zp_get_user_zombie_class(attacker) == g_zclassidqex) )
    {
            user_slap(victim, 0, 0)
            return HAM_HANDLED;
    }
      
     return HAM_IGNORED;
}

если у немца атака больше чем хп у хумана, то не вижу смысла ставить условие. А так особо разницы не заметишь, т.к. вызывается функция не так уж и часто
кстати, проверь: если атакуешь зм-союзника = отскок тоже должен быть, потому как проверку не поставили на это

тем что нет проверки на атаку с ножа. Ударишь каким-либо другим способом - жертва получит пинок

DeMNiX, По поводу проверки, а вот лучше поставить проверку или нет? Просто у меня такое бывает и я думаю сижу, ставить или не ставить))
Так у зомби кроме инфекционной гранаты нечего не может быть)) Ну по крайне у меня так и будет всегда)

если у тебя кроме этого ничего не будет, тогда не критично
инфекционка не вызывает ham_takedamage