Восстановление Стража Скверны

[virusav]

До определенного момента пет чернокнижника Страж Скверны восстанавливался полностью, когда игрок слезал с маунта, т.е. призываем пета, лезем в бой, садимся на маунта, слезаем - пет полностью восстановлен.

С определенного момента пет восстанавливается только при нахождении в мире.
Если игрок сел на маунта и долгое время не слезал с него, то никакого восстановления пета нет.

Так и должно быть?

[evilstar]

Проверю на оффе

Добавил: Проверил, да пет восстанавливается полностью после слезания с маунта

[virusav]

Значит, в каком-то коммите поломали, т.к. раньше работало.

[rsa]

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

Проверю на оффе

Добавил: Проверил, да пет восстанавливается полностью после слезания с маунта

Хм. Я себе даже специальный участок кода писал раньше чтобы так было. Потому как меня целая толпа народа уверяла со скринами что пока пет утемпсаммонен - он не восстанавливается. Еще на 3.3.5а.

[virusav]

Т.е. пет не должен полностью восстанавливаться после слезания с маунта?

[rsa]

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

Т.е. пет не должен полностью восстанавливаться после слезания с маунта?

Меня уверяли (и даже скрины были, только я их давно прибил) что во время нахождения на маунте на оффе 3.3.5 ни хп ни мана у петов не ресаются, остаются такими же как и при залезании. И я заимплементил это себе полгода назад. Поскольку своей информации с оффа у меня нет, пришлось поверить.
С учетом того, что на моем ядре петы - весьма мощная боевая единица (почти как на оффе), такое шустрое восстановление ХП демонам лока и хантам их весьма усиливало, особенно в пвп. Попробуй нахилять войду его 35к ХП (при всех глифах) или дождаться пока он сам отресается...

[virusav]

На текущий момент проще застрелиться, чем дождаться восстановления пета.
Тем более, что иногда этого вообще не происходит, особенно в инстах.

[rsa]

Ну я уже с трудом представляю как там идет восстановление ХП на чистом ядре, слишком давно этот кусок кода ковыряю. У меня бес восстанавливает ману полностью 2 минуты, войд свое ХП (без магии) - 4-5 минут. Что соответствует замерам что для меня делали на 3.3.5. В бою тоже, хоть и с другой скоростью (функцией от скорости хозяина).

[virusav]

Еще раз спросил теперь уже у другого игрока с офа: когда игрок слезает с маунта, пет полностью восстанавливается.
Так сейчас, так было на 335а.
На какой-то версии проскакивал баг с неполным восстановлением, но его потом поправили.

[Йоха]

так и есть только что проверил, покоцаный страж скверны появляется со 100% хп если залезть и слезть с маунта
При чем заметил мелкий нюанс, после спешивания страж всегда появляется с очень малым количеством хп, а буквально через секунду у него резко становится 100%. При повышенном пинге этот промежуток времени становится явно различимым. Похоже пет самонится с минимумом хп, а потом приходит апдейт на восстановление полного здоровья.

[virusav]

Сейчас:

void Unit::Unmount()
...
((Player*)this)->ResummonPetTemporaryUnSummonedIfAny();
...
NewPet->LoadPetFromDB(this, 0, m_temporaryUnsummonedPetNumber, true)

Т.е. при слезании с маунта происходит загрузка пета из базы, где прописаны оставшиеся данные.

Как должна исправляться проблема:
1. При посадке на маунта прописывать в базу полные параметры пета?
2. При слезании с маунта восстанавливать все параметры пета?

Лично мне ближе второй вариант.

Как правильнее исправить баг с невосстановлением петов при слезании с маунта?

[virusav]

Для теста попробовал так: в конец

void Player::ResummonPetTemporaryUnSummonedIfAny()

прописал

    if (NewPet)
    {
        NewPet->InitStatsForLevel(this->getLevel(), ((Unit *)this));
    }

В итоге при слезании с маунта пет полностью восстанавливается, как и должно быть.

Какие мысли по этому поводу?

[Estella]

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

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

Для теста попробовал так: в конец

void Player::ResummonPetTemporaryUnSummonedIfAny()

прописал

    if (NewPet)
    {
        NewPet->InitStatsForLevel(this->getLevel(), ((Unit *)this));
    }

В итоге при слезании с маунта пет полностью восстанавливается, как и должно быть.

Какие мысли по этому поводу?

Насчёт Хантов и Локов подтверждаю с оффа, всё правильно, когда слезаешь с маунта у пета ханта и саммонера лока хп восстанавливалось и восстанавливается до 100%. Баг был на оффе с петами, в какой-то из версий 3.3.х когда слезаешь с маунта у твоего пета или саммонера 50%, ещё раз садишься, слезаешь 25%. Но Близы пофиксили примерно через 2-3 недели его.

rca, на счёт твоего поста :

С учетом того, что на моем ядре петы - весьма мощная боевая единица (почти как на оффе), такое шустрое восстановление ХП демонам лока и хантам их весьма усиливало, особенно в пвп. Попробуй нахилять войду его 35к ХП (при всех глифах) или дождаться пока он сам отресается...

Смотря в каком спеке лок или хант, если демон (Лок) или БМ (хант)... то петы дамажут так, что фиг ты его просадишь по хп, быстрее сам ляжешь если не убьёшь самих лока или ханта, там петов вообще не реал убить. Если конечно игрок мешать не будет, то убьёшь, но а так быстрее сам ляжешь. ХП во время боя регенится не быстро, а как только садишься на ездового, то у пета ХП сразу 100%.

[rsa]

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

Для теста попробовал так: в конец

void Player::ResummonPetTemporaryUnSummonedIfAny()

прописал

    if (NewPet)
    {
        NewPet->InitStatsForLevel(this->getLevel(), ((Unit *)this));
    }

В итоге при слезании с маунта пет полностью восстанавливается, как и должно быть.

Какие мысли по этому поводу?

вообще-то это в LoadPetFromDB должно быть... Тем более после реинита статов надо еще каки-то операции сделать как минимум.

[virusav]

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

вообще-то это в LoadPetFromDB должно быть... Тем более после реинита статов надо еще каки-то операции сделать как минимум.

Я выложил пример для теста и спросил, как правильно надо исправить баг.

Операций может быть больше, т.к., возможно, еще надо учитывать бафы на пете и т.д.

В конечном итоге нужен патч для исправления бага, а то сейчас от пета толку мало, т.к. в первом же бою мана закончилась, а дальше агрить он не особо хочет.

[rsa]

я исправил так:
https://github.com/rsa/mangos/commit...d9834b1669c052
насколько я помню, этот участок кода в чистом ядре весьма похож.
PS сразу получил жалобы от роговладельцев, мол ханты с локами их теперь ушибают как умеют.

[virusav]

В чистом ядре в Pet::LoadPetFromDB есть блок:

    if (getPetType() == SUMMON_PET && !current)             //all (?) summon pets come with full health when called, but not when they are current
    {
        SetHealth(GetMaxHealth());
        SetPower(POWER_MANA, GetMaxPower(POWER_MANA));
    }
    else
    {
        SetHealth(savedhealth > GetMaxHealth() ? GetMaxHealth() : savedhealth);
        SetPower(POWER_MANA, savedmana > GetMaxPower(POWER_MANA) ? GetMaxPower(POWER_MANA) : savedmana);
    }

Как я понимаю, надо просто изменить или убрать условие.

Вчера для теста прописал в ядре:

        SetHealth(GetMaxHealth());
        SetPower(POWER_MANA, GetMaxPower(POWER_MANA));

Совпало, хотя кусок кода выше пропустил.

[virusav]

Или можно просто в Player::ResummonPetTemporaryUnSummonedIfAny() передавать
в Pet::LoadPetFromDB( Player* owner, uint32 petentry, uint32 petnumber, bool current ) в параметре current не true, а false.

[rsa]

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

Или можно просто в Player::ResummonPetTemporaryUnSummonedIfAny() передавать
в Pet::LoadPetFromDB( Player* owner, uint32 petentry, uint32 petnumber, bool current ) в параметре current не true, а false.

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

[virusav]

Код

    if (getPetType() == SUMMON_PET && !current)             //all (?) summon pets come with full health when called, but not when they are current
    {
        SetHealth(GetMaxHealth());
        SetPower(POWER_MANA, GetMaxPower(POWER_MANA));
    }
    else
    {
        SetHealth(savedhealth > GetMaxHealth() ? GetMaxHealth() : savedhealth);
        SetPower(POWER_MANA, savedmana > GetMaxPower(POWER_MANA) ? GetMaxPower(POWER_MANA) : savedmana);
    }

можно заменить на

    if (getPetType() == SUMMON_PET)             //all (?) summon pets come with full health when called, but not when they are current
    {
        SetHealth(GetMaxHealth());
        SetPower(POWER_MANA, GetMaxPower(POWER_MANA));
    }
    else
    {
        SetHealth(savedhealth > GetMaxHealth() ? GetMaxHealth() : savedhealth);
        SetPower(POWER_MANA, savedmana > GetMaxPower(POWER_MANA) ? GetMaxPower(POWER_MANA) : savedmana);
    }

Насколько это правильно?

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

[rsa]

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

Насколько это правильно?

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

Это правильно, но else в текущей реализации работать не будет никогда. Все прочие типы грузятся из базы только в моей реализации петов.

Единственный случай загрузки пета с неполной маной/ХП был как раз предусмотрен для временного ансаммона (залезание/слезание с маунта/вехикла), сейчас в соответствии с Вашими замечаниями этот режим ликвидирован. Еще есть case телепорта игрока с петом, но его в любом случае надо изучать и прописывать отдельно.

[virusav]

Если сделать так:

if (getPetType() == SUMMON_PET)             //all (?) summon pets come with full health when called

то у пета на примере моего перса 7303 жизней из 7313 при спешивании игрока.
Каждый раз разница в 10 единиц, та же картина с маной, т.е. где-то еще идет пересчет параметров пета.

[rsa]

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

Если сделать так:

if (getPetType() == SUMMON_PET)             //all (?) summon pets come with full health when called

то у пета на примере моего перса 7303 жизней из 7313 при спешивании игрока.
Каждый раз разница в 10 единиц, та же картина с маной, т.е. где-то еще идет пересчет параметров пета.

в чистом ядре на пета наворочено некоторое количество весьма неприятных хаков, если я правильно помню то и на ХП с маной тоже - в статсистеме зашиты. я у себя их все убрал (правда навесил других но менее кривых, надеюсь). в результате именно этого эффекта не имею точно.

PS все еще продолжается отображение резистов пета всегда по 1 каждого типа на чистом ядре? давненько уже не пробовал запустить...

[virusav]

Сообщение от Йоха

так и есть только что проверил, покоцаный страж скверны появляется со 100% хп если залезть и слезть с маунта
При чем заметил мелкий нюанс, после спешивания страж всегда появляется с очень малым количеством хп, а буквально через секунду у него резко становится 100%.

Если оставить тот блок при загрузке пета нетронутым и добавить

diff --git a/src/game/Player.cpp b/src/game/Player.cpp
index 27a83d8..59431a1 100644
--- a/src/game/Player.cpp
+++ b/src/game/Player.cpp
@@ -21900,6 +21900,11 @@ void Player::ResummonPetTemporaryUnSummonedIfAny()
     Pet* NewPet = new Pet;
     if(!NewPet->LoadPetFromDB(this, 0, m_temporaryUnsummonedPetNumber, true))
         delete NewPet;
+    else if (NewPet->getPetType() == SUMMON_PET)             //all (?) summon pets come with full health when called
+    {
+        NewPet->SetHealth(NewPet->GetMaxHealth());
+        NewPet->SetPower(POWER_MANA, NewPet->GetMaxPower(POWER_MANA));
+    }
 
     m_temporaryUnsummonedPetNumber = 0;
 }

то получим загрузку пета с предыдущими параметрами, после чего произойдет полное восстановление, как и написал Йоха.

[virusav]

Чтобы также работало для петов охотника:

diff --git a/src/game/Player.cpp b/src/game/Player.cpp
index 27a83d8..5700541 100644
--- a/src/game/Player.cpp
+++ b/src/game/Player.cpp
@@ -21900,6 +21900,11 @@ void Player::ResummonPetTemporaryUnSummonedIfAny()
     Pet* NewPet = new Pet;
     if(!NewPet->LoadPetFromDB(this, 0, m_temporaryUnsummonedPetNumber, true))
         delete NewPet;
+    else
+    {
+        NewPet->SetHealth(NewPet->GetMaxHealth());
+        NewPet->SetPower(NewPet->getPowerType(), NewPet->GetMaxPower(NewPet->getPowerType()));
+    }
 
     m_temporaryUnsummonedPetNumber = 0;
 }

Теперь такой вопрос: что будет на офе, если послать пета в бой, самому в него не вступать и через какое-то время сделать маунт/дисмаунт?
Будет ли пет чернокнижника или охотника полностью восстановлен или нет?

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

Чтобы также работало для петов охотника:

Теперь такой вопрос: что будет на офе, если послать пета в бой, самому в него не вступать и через какое-то время сделать маунт/дисмаунт?
Будет ли пет чернокнижника или охотника полностью восстановлен или нет?

на оффе вот как.
1. Охотник (Лок) посылает своего пета в атаку.
2. Пет дерётся, автоматом игрок тоже вступает в режим боя, даже не нанеся ни одного удара (тоесть сесть на маунта не сможет, пока не выйдет из режима боя)
3. Если моб убивает пета, то автоматом агро перекидывается на игрока (отстояться когда моб убил пета не получится).
4. Если это хант, то можно использовать спелл "Притвориться мёртвым" чтобы заивейдить моба (снять агро в 0), тогда моб убежит и восстановит своё здоровье в 100%. (есть исключения когда спелл у ханта не работает, это с гвардами. у них агро не сбрасывается и они просто тебя убивают, при атаке городов и прочей местности противоположной фракции)