ReqSpellCast [quest_template]
Давно уже думаю об этом, и вот, кажется, настала пора написать это.
Эти поля не нужны. База YTDB уже давно перешла на почти полностью безхаковый режим работы (насколько это вообще возможно, конечно) и все данные берутся из кэшей клиента. Как результат: сейчас в базе всего 73 квеста, где задействованы данные поля... 73 из 9456. И эти 73 квеста ни что иное как просто ещё не попавшиеся в кэшах. Следовательно, любое использование данного поля - хак в базе. Раз любое - значит подобные поля попросту не нужны. Да и силами EventAI сейчас всё очень неплохо решается, оставшееся - силами SD2. 0,77% квестов использующих данные поля - по-моему достаточно свидетельство об их ненадобности... Ну и напоследок пример: квест http://ru.wowhead.com/?quest=8346 Раньше как решались подобные задачи? ID маназмея в ReqCreatureOrGOId1, ID спелла в ReqSpellCast1. Но по данным с кэша в ReqCreatureOrGOId1 стоит http://ru.wowhead.com/npc=15468, а зачёт каста реализуется иным образом... Надеюсь, общая идея ясна. Владимир, хотелось бы услышать ваши комментарии по данному поводу. |
и зачем бежать впереди поезда?
в уже работающую схему, придумывать велосипед... как по мне, реализуются оставшиеся квесты, килл поля и забыли. |
Цитата:
Зачем оставлять место для хаков, когда с этим и в коде, и в базе всегда ведётся яростная борьба? Господа модераторы, нехорошо редактировать чужие сообщения без малейшего комментария... |
аааа, так и напиши, хочу реализовать вариант..... пока не пнешь не полетит :).
В таком случаи абсолютно за, более того минус затраты на чтение дополнительных полей |
Владимир, можно всё-таки услышать ваш комментарий по этому поводу?
|
Да, нужно удалить эту таблицу, но перед этим, также, нужно реализовать, наконец-то, грамотно, все скрипты, чтобы действительно, не пришлось никому возвращаться к ней, а то, многие используют её, для того, чтобы реализовать квесты, к примеру, такие как http://ru.wowhead.com/quest=9685. Не смотря на то, что это является хаком.
То есть, как было сказано в посте №2, зачем торопиться? Думаю, надо сначала приложить усилия, чтобы освободить таблицу, а потом и её удалять. |
1) Приведённый в пример вами квест никак не должен быть - у него есть скрипт в СД2
2) В YTDB (596) к настоящему времени всего 58 кв. использующих это поле. И я вовсе не уверен, что данные кв работают правильно или работают вообще. Проект так не любит хаки и оставляет такую дыру... 58 квестов. Если сейчас они реализованы через это глупое поле, значит через ЭвентАИ или скрипты их реализовать не составит труда. Но нахрен никто не пошевелится что-то менять, пока есть прекрасная возможность сделать это хаковым способом... |
Абсолютно с вами солидарен, даже на счёт того, что тот приведённый мною пример есть в Sd2, притом, настолько корявый, что все палы орды, устали уже изощряться в ругательствах на этот счёт....
Как я заметил, всем лень, и никто не хочет ничего делать, из того, что действительно важно.... Каждый делает лишь то, чего им так хочется делать... Ну неужели так трудно, целенаправленно перелопатить эти 58 квестов? А также, по дороге прихватить то, что до сих пор, не могут осилить Sd2? P.S. Не буду упоминать целую стопку других, не менее важных проблем, которыми никто не хочет заниматься..... |
|
Vinolentus, убрать лишние поля из базы и их обработчик в коде - до минут 15. Я мог бы это сделать. Но сначала я хочу убедиться, что это будет принято. Если это нафиг никому не надо и всё прекрасно и так - я не стану тратить свои даже 15 минут...
|
В UDB используется в 150-ти. Как не будут использоваться или мало останется так и удалим. Борьба с хаками это хорошо, но без экстремизма. Пример с SD2 - да могу дропнуть хоть сейчас GetGUID() - и что - девелоперы SD2 просто уписаются от счастья поставленные перед фактомм наличия нескольких тысяч ошибок сборки.
И сколько времени после этого не будет собьираться SD2, а может девелоперы просто плюнут и уйдут. |
Однако где гарантии, что сейчас девелоперы UDB стараются сделать эти кв правильно? А вот если бы изменения были - им пришлось бы отрыть информацию, что же действительно должно быть в квесте.
Я уже и не хотел упоминать, что это поле используется в кв где, к примеру, нужно излечить 10 кого-то там. Да, с этим полем это легко реализовать. И сделать потом тоже легко 10 раз на одного НПС кастануть. Да и кто будет задумываться как правильно надо, если жалоб разрабам о этих кв не поступает - они ведь работают! =) В общем печально, что разработчики ядра избрали именно такую позицию =( |
Ожидать что кто будет исправлять 150 квестов за раз не реально. И я думаю новых использований давно не добавлялось. Просто надо время чтобы исправить оставшиеся - я лично устраивать нездоровые сенсации и ставить девелоперов UDB пред фактом гемороя не собираюсь. Само по себе наличие полей не мешает ничему. Небудут нужны - удалим.
|
Vladimir
А иногда полезно пнуть. Сколько времени можно ждать (больше года) дроп полей в "creature_movement" т.к. в "creature_movement_scripts" они дублируются? Там делов на час по "перегнать", по базе (могу помочь перегнать). |
Цитата:
|
Shmoo, what you think about drop field ReqSpellCast???
|
udb still has a bunch of quests with it.
But yes, this should be dropped :) Edit: However it seems many of them can be dropped directly |
|
shmoo, and stop to look at UDB - they really slowly!!! =)))
|
Текущее время: 23:55. Часовой пояс GMT +3. |
ru-mangos.ru - Русское сообщество MaNGOS