spell1..spell4

[zergtmn]

В настоящий момент (10526) они используются для (1) generic_creature AI и для (2) экшн-баров контролируемых существ.
Для (2) случая 4-х слотов не достаточно.

Пример (possesed):


Случай без команды "Атаковать":


Пример (vehicle):




Уникальный случай с командой "Атаковать":


Вижу 2 выхода:
1. Отказаться от устаревшего generic_creature в пользу ACID, удалить spell1..spell4 из creature_template, завести новую таблицу для хранения экшн баров.
2. Добавить столбцы spell5..spellN, где в 99% случаев будут нули.

Если 1 вариант устраивает, могу подготовить патч.

Update: spell1 еще для тотемов используется, куда бы его пристроить...

[ghostpast]

С точки зрения теории нормализации БД второй вариант не подходит

[rsa]

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

[YuruY]

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

С точки зрения теории нормализации БД второй вариант не подходит

Зато не плодит таблиц (ну и ответ выше).

[virusav]

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

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

[zergtmn]

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

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

Уж прям в разы О_о
Отключи вывод в консоль, быстрее будет)

[rsa]

Если точнее то раз в 8 при использовании конката таблиц, и процентов на 20 при использовании отдельной карты std::vector-ов спеллов, но во втором случае надо сильно перекраивать существующие объекты и работу с ними.

PS у меня консоли нету совсем... на /dev/null назначена.

[virusav]

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

Если кеш по нпц спеллы приходят в 4 полях, то скорее всего их никто никуда не денет.

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

[rsa]

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

[rsa]

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

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

это при сохранении полной совместимости с текущим форматом. во всех "старых" ячейках были забиты поиски в новой таблице.

[Konctantin]

Если кеш по нпц спеллы приходят в 4 полях, то скорее всего их никто никуда не денет.

В кеше нет данных по спелам:

    class CreatureTemplate
    {
        public int      Entry;
        public string   Name;
        public string   SubName;
        public string   IconName;
        public uint     TypeFlag;
        public uint     Type;
        public int      Family;
        public uint     Rank;
        public uint[]   KillCredit  = new uint[2];
        public uint[]   ModelId     = new uint[4];
        public float    HealthModifier;
        public float    PowerModifier;
        public sbyte    RacialLeader;
        public uint[]   QuestItem   = new uint[6];
        public uint     MovementId;
    }

[virusav]

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

это при сохранении полной совместимости с текущим форматом. во всех "старых" ячейках были забиты поиски в новой таблице.

Не знаю, каким образом выбирались спеллы из другой таблицы.

Я бы на парсере сделал так:
1. Выбираем одним запросом всех нпц и все спеллы по ним. В итоге получаем на каждого нпц одну строку, в которой в отдельном поле были бы коды спеллов через запятую.
2. Из поля с кодами спеллов брал бы значение и получал бы все спеллы нпц.

Насчет сишника не могу сказать, как лучше будет сделать.

[rsa]

Судя по всему ждать решения проблемы от тима не приходится, предлагаю свое -
https://github.com/mangosR2/mangos/c...c4e12378cfa6c6
работает нормально. поля из creature_template пока не удалял, потом прибью.

[zergtmn]

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

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

Они для тотемов нужны и для generic_creature

[rsa]

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

Они для тотемов нужны и для generic_creature

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

[zergtmn]

У меня была мысль сделать таблицу:

entry actionbar_index spell1 spell2 spell3 spell4 spell5 spell6

для поддержки вехиклов с несколькими наборами спеллов на экшн баре, например http://www.wowhead.com/npc=30275. В ядре нужно делать античитерскую проверку при касте на вехикле, что спелл показывается в текущий момент на экшн баре.

[rsa]

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

У меня была мысль сделать таблицу:

в этой реализации оно может быть сделано гораздо проще - поле activeState оставлено именно для подобных замен. по умолчанию оно 0 и все идет из основного набора. можно дать наборам 1, 2, 3 соотв. state и подменять их прямо на лету... фантазия не ограничена.

[zergtmn]

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

в этой реализации оно может быть сделано гораздо проще - поле activeState оставлено именно для подобных замен. по умолчанию оно 0 и все идет из основного набора. можно дать наборам 1, 2, 3 соотв. state и подменять их прямо на лету... фантазия не ограничена.

Не то. В вашей реализации activeState меняется для всех кричеров с одним entry.

[rsa]

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

Не то. В вашей реализации activeState меняется для всех кричеров с одним entry.

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