[9872][patch] Life Tap (Жизнеотвод)

[Insider42]

Решил реализовать фичу последних патчей, формулы взяты с вовхеда, опрашивал игроков с оффа.
Как рассказал один из игроков с оффа:
Получаем базовый урон+урон зависящий от духа
Получаем базовую ману+ману зависящую от силы заклинаний

diff --git a/src/game/SpellEffects.cpp b/src/game/SpellEffects.cpp
index f8fcfea..65d4f06 100644
--- a/src/game/SpellEffects.cpp
+++ b/src/game/SpellEffects.cpp
@@ -1851,30 +1851,19 @@ void Spell::EffectDummy(SpellEffectIndex eff_idx)
             // Life Tap
             if (m_spellInfo->SpellFamilyFlags & UI64LIT(0x0000000000040000))
             {
+                int32 damagebase = damage;
                 // In 303 exist spirit depend
                 uint32 spirit = uint32(m_caster->GetStat(STAT_SPIRIT));
-                switch (m_spellInfo->Id)
-                {
-                    case  1454: damage+=spirit; break;
-                    case  1455: damage+=spirit*15/10; break;
-                    case  1456: damage+=spirit*2; break;
-                    case 11687: damage+=spirit*25/10; break;
-                    case 11688:
-                    case 11689:
-                    case 27222:
-                    case 57946: damage+=spirit*3; break;
-                    default:
-                        sLog.outError("Spell::EffectDummy: %u Life Tap need set spirit multipler", m_spellInfo->Id);
-                        return;
-                }
-//              Think its not need (also need remove Life Tap from SpellDamageBonus or add new value)
-//              damage = m_caster->SpellDamageBonus(m_caster, m_spellInfo,uint32(damage > 0 ? damage : 0), SPELL_DIRECT_DAMAGE);
+                damage+= spirit*15/10;
+
                 if (unitTarget && (int32(unitTarget->GetHealth()) > damage))
                 {
                     // Shouldn't Appear in Combat Log
                     unitTarget->ModifyHealth(-damage);
 
-                    int32 mana = damage;
+                    int32 spell_power = m_caster->SpellBaseDamageBonusDone(GetSpellSchoolMask(m_spellInfo));
+                    int32 mana = damagebase + (spell_power * 5/10);
+
                     // Improved Life Tap mod
                     Unit::AuraList const& auraDummy = m_caster->GetAurasByType(SPELL_AURA_DUMMY);
                     for(Unit::AuraList::const_iterator itr = auraDummy.begin(); itr != auraDummy.end(); ++itr)

также можно взять тут http://paste2.org/p/809556

Я совершенно не в курсе, нужно ли учитывать различные "штрафы" от ранга способности... если нужно то меняем

int32 mana = damagebase + (spell_power * 5/10);

на

int32 mana = damagebase + (spell_power * 5/10 * m_caster->CalculateLevelPenalty(m_spellInfo));

а весь

switch (m_spellInfo->Id)

оставляем какой был

ЗЫ: патч не тестировался!

[Viste]

у меня в

switch (m_spellInfo->Id)

так

                    case  1454: damage += int32(0 + 1.5*sp); break;
                    case  1455: damage += int32(6 + 1.5*sp); break;
                    case  1456: damage += int32(24 + 1.5*sp); break;
                    case 11687: damage += int32(37 + 1.5*sp); break;
                    case 11688: damage += int32(42 + 1.5*sp); break;
                    case 11689: damage += int32(500 + 1.5*sp); break;
                    case 27222: damage += int32(710 + 1.5*sp); break;
                    case 57946: damage += int32(1490 + 1.5*sp); break;

sp берется отсуда

float sp = m_caster->SpellBaseDamageBonus(GetSpellSchoolMask(m_spellInfo));

[DiffuSer]

При таком решении мана становится бесконечной на начальных уровнях. Он практический не наносит урон, а маны дает значительное количество.

[Insider42]

Сообщение от Viste

у меня в

switch (m_spellInfo->Id)

так

                    case  1454: damage += int32(0 + 1.5*sp); break;
                    case  1455: damage += int32(6 + 1.5*sp); break;
                    case  1456: damage += int32(24 + 1.5*sp); break;
                    case 11687: damage += int32(37 + 1.5*sp); break;
                    case 11688: damage += int32(42 + 1.5*sp); break;
                    case 11689: damage += int32(500 + 1.5*sp); break;
                    case 27222: damage += int32(710 + 1.5*sp); break;
                    case 57946: damage += int32(1490 + 1.5*sp); break;

sp берется отсуда

float sp = m_caster->SpellBaseDamageBonus(GetSpellSchoolMask(m_spellInfo));

А разве базовый задаток damage (первое число у вас в скобке) не получен на более ранней стадии?

[Viste]

да вы правы, я что-то затупил
add
можно еще так вместо всего

switch (m_spellInfo->Id)

просто добавить

damage+=spirit*15/10;

[Insider42]

Сообщение от Viste

можно еще так вместо всего

switch (m_spellInfo->Id)

просто добавить

damage+=spirit*15/10;

А у меня разве не так?

[Insider42]

Протестировал таки свой патч. C последним рангом способности работает четко. Единственное отличие, скорее даже "фича" мангоса - восстанавливает на 1 маны меньше и отнимает на 1 здоровья меньше

А вот с навыками пониже уровня странность - на способности пишет 701 маны например, но даёт 710. Причем разница на всех кроме последнего ранга составляет всего 9 маны. Врятли это какой-либо penalty для бонуса заклинаний, скорее всего формула для низких рангов чуть чуть иная, либо же косяк на стороне клиента.

[LordJZ]

А ничего, что сами близзы сказали, что на способности неправильно написано?

[Vladimir]

Patch 3.3.3: "This spell no longer scales with spirit, and instead scales with spell power."

[Insider42]

Сообщение от Vladimir

Patch 3.3.3: "This spell no longer scales with spirit, and instead scales with spell power."

близы часто говорят одно а делают другое. У меня даже с подсказками сходится на последнем ранге и на вовхеде формула такая же как в моем патче.

Игроки с оффа тоже подтвердили, что наносимый по игроку урон всё ещё базируется на духе, а мана уже на силе заклинаний.

Converts [0 + SPI * 1.5] health into [0 + SPS * 0.5 * 1] mana. Spell power increases the amount of mana returned.

Преобразование [0 + SPI * 1.5] ед. здоровья в [0 + SPS * 0.5 * 1] ед. маны. Сила заклинаний повышает количество получаемой маны.

http://www.wowhead.com/spell=1454

[Vladimir]

Я не увидел нигде в комментариях опровержений к изменениям в 3.3.3

Добавлено с удалением спирит-части в [9872]. Спасибо.