[bylins]

Жалоба

Игрок заметил: бьешь моба оружием с беленой — первое наложение проходит гарантированно, как и задумано. Но если отбежать и вернуться, следующий удар снова кладет яд со стопроцентной вероятностью, хотя прежний еще держится. «Стоя в бою накладываются с мелким шансом, а если бегать — друг за другом».

Причина

Гарантированное наложение в PerformPoisonedWeapom висело на боевой защелке:

if (((success) && (number(1, 100) <= 25)) ||
    (!vict->battle_affects.get(kEafFirstPoison) && !AFF_FLAGGED(vict, EAffect::kPoisoned))) {

kEafFirstPoison лежит в battle_affects, а они обнуляются и при входе в бой (fight.cpp:233), и при выходе из него (fight.cpp:306). То есть каждый новый бой для цели — «первый», и первый удар опять дает гарантию. При этом poison_affect_join стакает модификатор до трех раз и каждый раз ставит полную длительность, а сам яд помечен kAfSameTime и на мобе держится около семи минут. Выходило, что беготня туда-сюда набивает полный стак быстрее честного боя, где повторные наложения идут с шансом 25%.

Аконита это не касалось: один из его узлов имеет тип kPoisoned, поэтому AFF_FLAGGED(vict, kPoisoned) блокировал вторую половину условия. У белены, скополии и дурмана типы свои (kBelenaPoison и прочие), и вся защита держалась только на защелке.

Правка

Гарантия теперь проверяет не «первый раз за бой», а «этого яда на цели еще нет» — по опознавательному аффекту кластера:

static EAffect PoisonClusterAffect(ESpell spell_id);   // kAconitumPoison / kScopolaPoison / kBelenaPoison / kDaturaPoison
...
if (((success) && (number(1, 100) <= 25)) ||
    (!AFF_FLAGGED(vict, PoisonClusterAffect(spell_id)) && !AFF_FLAGGED(vict, EAffect::kPoisoned))) {

Проверка !AFF_FLAGGED(vict, kPoisoned) оставлена как была, так что для аконита поведение не меняется вовсе. Защелка kEafFirstPoison стала не нужна и убрана из EExtraAttackFlag — в коде на нее больше нет ни одной ссылки.

Что меняется в игре: первое наложение каждого яда по-прежнему гарантировано, повторные внутри действия яда — честные 25% за удар, и выбегание из боя этого больше не обнуляет.

Проверка

Сборка release + build_tests=true чистая, без предупреждений; юнит-тесты 796 прошли, 3 отключены.

🤖 Generated with Claude Code