Проверка возможности каста

[virusav]

Есть спелл http://ru.wowhead.com/?spell=53067, который в идеале требует реагенты.
При вызове

pPlayer->CastSpell(pPlayer,53067,true);

каст происходит без всякой проверки на наличие итемов.
После каста итемы не исчезают из инвентаря.

1. Есть ли какая-нибудь функция проверки возможности каста?
2. Должна ли быть проверка в ядре на требование реагентов по дбц?

[MaS0n]

1. Да, Spell.cpp-CheckCast. Основа - различные серверные проверки для предотвращения читерства + спец-ограничения

2. Есть. Ф-ция TakeReagents() вызывается в cast после того, как проходят проверки CheckCast, CheckItems, CheckPower.

В данном случае это не работает из-за triggered = true.

Триггерные спелы не требуют реагентов или энергии, каст инстант и без учета ЛОС

[virusav]

CheckCast находил, но в конференции сказали, что это не то.

В CheckCast есть вызов CheckItems.

1. Является ли данный спелл триггерным?
2. Достаточно ли будет вызова CheckCast, а после него TakeReagents?
3. Если у меня изначально есть объект Player* pPlayer, то можно ли из него вызвать

if (const SpellEntry* pSpellInfo = GetSpellStore()->LookupEntry(53067))

и CheckCast для проверки или надо еще что-то добавлять?

[MaS0n]

Достаточно

pPlayer->CastSpell(pPlayer, 53067, false);

И спелл будет забирать реагенты

[virusav]

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

[MaS0n]

Нет, триггерным он изначально не является.
Проверки будут работать в любом случае.
Триггерность задается с кастом, CastSpell/CastCustomSpell, bool triggered в описании

[virusav]

Встречал разные определения триггерного спелла, но четкого так и не нашел.

Ситуация: ареатриггер срабатывает при нахождении игрока в его области, игрок должен кастануть спелл, если есть реагенты.

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

[MaS0n]

Думаю, нагрузка минимальна и доп. проверки не надо, если реагентов нет - просто не будет каста и игрок врятли это заметит вообще

[virusav]

Большое спасибо, обманули меня в конференции, засранцы.

[virusav]

Снова возник вопрос о проверке возможности каста.

В данном случае ситуация немного другая: надо вернуть true, только если каст спелла успешно произведен, иначе false.

Дано:

const SpellEntry* pSpellInfo = GetSpellStore()->LookupEntry(ххх)
Player* pPlayer
Item* pItem
const SpellCastTargets &pTargets

По идее, если бы CastSpell возвращал true/false, то проблема бы решалась просто, но такого нет.

Как, имея указанные выше данные, проверить возможность каста и правильно вернуть true/false?

[Vladimir]

потому что каст происходит позднее из Update. Соответствеено неизвестно будет ли он реально произведен.
Конечно можно возвращать true как "возможно будет скастован если что-либо не случится"

Базовые проверки производятся для каста мобами в CreatureAI::CanCast

[virusav]

Кастует игрок.
Как в таком случае проверять?

[Vladimir]

А смысл? Вы не получите 100% достоверного результата в точке где каст выполняется из кода, так как это только регистрация события... Как я уже написла _можно_ добавит возвращение результата предварительных проверок, но в таком случае будет 100% достоверным только false-результат, а true-результат только вероятносным что может быть все будет хорошо.

[virusav]

Ситуация такова: в СД2 игрок должен скастовать спелл, требующий реагенты.

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

Этого как-то можно избежать?

[virusav]

Есть какое-нибудь решение?
Если нет, то лучше после каста возвращать true с кривой анимацией или false с ложной ошибкой?

[Vladimir]

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

[virusav]

Проверка нужна полная.

На текущий момент получается, что надо сделать кучу проверок, потом вызвать каст спелла с той же кучей проверок или с triggered = true.

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

[Inn04ka]

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