[patch/dev] Deathbringer's Will

[Astoria]

Сильно не пинайте. Мой первый опыт программирования на мангосе, да и программирования вовсе.
Есть итем Deathbringer's Will со спелом Icecrown 25 Heroic Melee Trinket c дамми аурой. Написал следущее:

PHP код:

/*
600/700     Strength        71484/71561
1200/1400     Attack Power    71486/71558
600/700     Agility            71485/71556
600/700     Critical Strike    71491/71559
600/700     Haste Rating    71492/71560
*/
case 71562:
{
    uint32 spell_id = 0;
    switch(target->getClass())
    {
        case CLASS_WARRIOR:
        case CLASS_DEATH_KNIGHT:
        case CLASS_PALADIN:
            switch(urand(1,3))
            {case 1: spell_id = 71561; break;case 2: spell_id = 71559; break;case 3: spell_id = 71560; break;}            }
            break;
        case CLASS_ROGUE:
        case CLASS_SHAMAN:
            switch(urand(1,3))
            {case 1: spell_id = 71558; break;case 2: spell_id = 71556; break;case 3: spell_id = 71560; break;}
            break;
        case CLASS_HUNTER:
            switch(urand(1,3))
            {case 1: spell_id = 71558; break;case 2: spell_id = 71559; break;case 3: spell_id = 71556; break;}
            break;
        case CLASS_DRUID:
            switch(urand(1,3))
            {case 1: spell_id = 71561; break;case 2: spell_id = 71556; break;case 3: spell_id = 71560; break;}
            break;
    }
    target->CastSpell(target, spell_id, true);
    return;
} 


Но как понятно, она работает только при apply ауры. Как сделать так чтобы работала от прока спела?

[Amaru]

Эту штуку надо засунуть в HandleDummyAuraProc, а не в HandleAuraDummy, также по хорошему добавить запись на нее в таблицу spell_proc_event.

[rsa]

кроме того, там еще надо сильно мудрить с кулдауном а триггерные спеллы брать не по классам а по типам энергии... нормальную реализацию делал SeT, можно поискать в R2.

[Amaru]

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

[Astoria]

Я руководствовался этим http://www.wowwiki.com/Deathbringer's_Will

триггерные спеллы брать не по классам а по типам энергии

The transformations the trinket provides are random, last 30 seconds each, and vary between classes.

Тут рандом кажется более логичным, кроме одного случая (формы друидов).

нормальную реализацию делал SeT, можно поискать в R2.

Посмотрю если дело будет совсем плохо, очень уж хочется сделать самому.

[rsa]

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

Тут рандом кажется более логичным, кроме одного случая (формы друидов).

вот именно. на друидах-то все колом и становится.

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

Посмотрю если дело будет совсем плохо, очень уж хочется сделать самому.

вы бы бросили уже изобретенные велосипеды изобретать. действующих разработчиков (даже самого начального уровня) под мангос остались считанные единицы, а еще нерешенных (совсем или частями проблем) - немеряно, всем хватит по самые ушки.
попробуйте вон хоть один issue из R2 решить - вот то будет дело, а решать проблему которую решали уже раз 10 минимум (гугль дает минимум 10 разных кодов) - как-то тоскливо.

[Astoria]

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

Читайте второе предложение #1. Мне сразу лезть прикручивать велосипеды? Или придумать как обходить трабл с опкодами на кате?

попробуйте вон хоть один issue из R2 решить - вот то будет дело

Ваш форк меня мало интересует. И это не его форум.

[rsa]

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

Читайте второе предложение #1. Мне сразу лезть прикручивать велосипеды? Или придумать как обходить трабл с опкодами на кате?

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

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

Ваш форк меня мало интересует. И это не его форум.

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

[Astoria]

идите на гетмангос

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

[zergtmn]

Опять пиар mangosR2

[virusav]

Отсутствие активности на чистом мангосе провоцирует подобные высказывания (о сторонних репозитариях).

[rsa]

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

Опять пиар mangosR2

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

Добавлено через 14 минут

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

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

По факту оно так и есть. Суточный траффик комментариев на гитхабе (не буду говорить где) уже давно превышает месячный траффик на этом форуме. Правда на 6 языках
Все, кому еще интересен вопрос развития мангоса (именно мангоса, а не вопрос поднятия ЧСВ отдельных членов mangos team) давно уже перешли на другие площадки, и хорошо если не на ТС.

[Mayss]

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

[Astoria]

Конечный вариант, проверил - работает.

diff --git a/src/game/UnitAuraProcHandler.cpp b/src/game/UnitAuraProcHandler.cpp
index f56885e..052bfaa 100644
--- a/src/game/UnitAuraProcHandler.cpp
+++ b/src/game/UnitAuraProcHandler.cpp
@@ -1004,6 +1004,45 @@ SpellAuraProcResult Unit::HandleDummyAuraProc(Unit *pVictim, uint32 damage, Aura
                     }
                     break;
                 }
+				case 71519:
+				case 71562:
+				{
+					if(((Player*)this)->HasSpellCooldown(dummySpell->Id))
+						return SPELL_AURA_PROC_FAILED;
+
+					switch(getClass())
+					{
+						case CLASS_WARRIOR:
+						case CLASS_DEATH_KNIGHT:
+						case CLASS_PALADIN:
+						{
+							uint32 proc_spells[] = {71491, 71492, 71484, 71559, 71560, 71561};
+							triggered_spell_id = (dummySpell->Id) == 71519 ? proc_spells[urand(0,2)] : proc_spells[urand(3,5)];
+							break;
+						}
+						case CLASS_ROGUE:
+						case CLASS_SHAMAN:
+						{
+							uint32 proc_spells[] = {71485, 71486, 71492, 71556, 71558, 71560};
+							triggered_spell_id = (dummySpell->Id == 71519) ? proc_spells[urand(0,2)] : proc_spells[urand(3,5)];
+							break;
+						}
+						case CLASS_HUNTER:
+						{
+							uint32 proc_spells[] = {71485, 71486, 71491, 71556, 71558, 71559};
+							triggered_spell_id = (dummySpell->Id) == 71519 ? proc_spells[urand(0,2)] : proc_spells[urand(3,5)];
+							break;
+						}
+						case CLASS_DRUID:
+						{
+							uint32 proc_spells[] = {71485, 71492, 71484, 71556, 71560, 71561};
+							triggered_spell_id = (dummySpell->Id) == 71519 ? proc_spells[urand(0,2)] : proc_spells[urand(3,5)];
+							break;
+						}						
+					}
+					((Player*)this)->AddSpellCooldown(dummySpell->Id, 0, time(NULL) + cooldown);
+					break;
+				}
             }
             break;
         }

[Amaru]

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

[rsa]

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

[Astoria]

А откуда данные о том, как выбираются спеллы в зависимости от спека? Я сомневаюсь что на офе кто то бы отдал волю кастерам, чтобы проверить то, что прокать будет только хаст. Дайте пруф.