Ru-MaNGOS  

Вернуться   Ru-MaNGOS > Ядро > Патчи

Важная информация

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

Если кто-то хочет задать вопрос по патчу - велкам.

Ответ
 
Опции темы Поиск в этой теме Опции просмотра
Старый 30.11.2010, 22:04   #1
KiriX
Умный
Старожил
 
Аватар для KiriX
 
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 431 раз(а) в 181 сообщениях
Записей в дневнике: 4
KiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человек
По умолчанию 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, а зачёт каста реализуется иным образом...
Надеюсь, общая идея ясна. Владимир, хотелось бы услышать ваши комментарии по данному поводу.
KiriX вне форума   Ответить с цитированием
Старый 01.12.2010, 15:56   #2
ANTOMA
Новичок
 
Регистрация: 07.03.2010
Сообщений: 11
Сказал(а) спасибо: 126
Поблагодарили 0 раз(а) в 0 сообщениях
ANTOMA На верном пути
По умолчанию

и зачем бежать впереди поезда?
в уже работающую схему, придумывать велосипед...
как по мне, реализуются оставшиеся квесты, килл поля и забыли.
ANTOMA вне форума   Ответить с цитированием
Старый 01.12.2010, 20:49   #3
KiriX
Умный
Старожил
 
Аватар для KiriX
 
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 431 раз(а) в 181 сообщениях
Записей в дневнике: 4
KiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человек
По умолчанию

Цитата:
Сообщение от ANTOMA Посмотреть сообщение
и зачем бежать впереди поезда?
в уже работающую схему, придумывать велосипед...
как по мне, реализуются оставшиеся квесты, килл поля и забыли.
Эмс... Что-то мне подсказывает, что они (эти оставшиеся квесты) реализуются правильным путём горааааздо быстрее, если поля килл сейчас...
Зачем оставлять место для хаков, когда с этим и в коде, и в базе всегда ведётся яростная борьба?

Господа модераторы, нехорошо редактировать чужие сообщения без малейшего комментария...
KiriX вне форума   Ответить с цитированием
Старый 03.12.2010, 14:59   #4
ANTOMA
Новичок
 
Регистрация: 07.03.2010
Сообщений: 11
Сказал(а) спасибо: 126
Поблагодарили 0 раз(а) в 0 сообщениях
ANTOMA На верном пути
По умолчанию

аааа, так и напиши, хочу реализовать вариант..... пока не пнешь не полетит .
В таком случаи абсолютно за, более того минус затраты на чтение дополнительных полей
ANTOMA вне форума   Ответить с цитированием
Старый 21.12.2010, 13:30   #5
KiriX
Умный
Старожил
 
Аватар для KiriX
 
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 431 раз(а) в 181 сообщениях
Записей в дневнике: 4
KiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человек
По умолчанию

Владимир, можно всё-таки услышать ваш комментарий по этому поводу?
KiriX вне форума   Ответить с цитированием
Старый 18.05.2011, 23:36   #6
Humort
Новичок
 
Регистрация: 24.03.2010
Сообщений: 5
Сказал(а) спасибо: 3
Поблагодарили 1 раз в 1 сообщении
Humort На верном пути
По умолчанию

Да, нужно удалить эту таблицу, но перед этим, также, нужно реализовать, наконец-то, грамотно, все скрипты, чтобы действительно, не пришлось никому возвращаться к ней, а то, многие используют её, для того, чтобы реализовать квесты, к примеру, такие как http://ru.wowhead.com/quest=9685. Не смотря на то, что это является хаком.

То есть, как было сказано в посте №2, зачем торопиться? Думаю, надо сначала приложить усилия, чтобы освободить таблицу, а потом и её удалять.
Humort вне форума   Ответить с цитированием
Старый 19.05.2011, 07:06   #7
KiriX
Умный
Старожил
 
Аватар для KiriX
 
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 431 раз(а) в 181 сообщениях
Записей в дневнике: 4
KiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человек
По умолчанию

1) Приведённый в пример вами квест никак не должен быть - у него есть скрипт в СД2
2) В YTDB (596) к настоящему времени всего 58 кв. использующих это поле. И я вовсе не уверен, что данные кв работают правильно или работают вообще.
Проект так не любит хаки и оставляет такую дыру...
58 квестов. Если сейчас они реализованы через это глупое поле, значит через ЭвентАИ или скрипты их реализовать не составит труда. Но нахрен никто не пошевелится что-то менять, пока есть прекрасная возможность сделать это хаковым способом...
KiriX вне форума   Ответить с цитированием
Старый 20.05.2011, 00:32   #8
Humort
Новичок
 
Регистрация: 24.03.2010
Сообщений: 5
Сказал(а) спасибо: 3
Поблагодарили 1 раз в 1 сообщении
Humort На верном пути
По умолчанию

Абсолютно с вами солидарен, даже на счёт того, что тот приведённый мною пример есть в Sd2, притом, настолько корявый, что все палы орды, устали уже изощряться в ругательствах на этот счёт....
Как я заметил, всем лень, и никто не хочет ничего делать, из того, что действительно важно....
Каждый делает лишь то, чего им так хочется делать...
Ну неужели так трудно, целенаправленно перелопатить эти 58 квестов? А также, по дороге прихватить то, что до сих пор, не могут осилить Sd2?

P.S. Не буду упоминать целую стопку других, не менее важных проблем, которыми никто не хочет заниматься.....
Humort вне форума   Ответить с цитированием
Старый 20.05.2011, 04:17   #9
Vinolentus
Пользователь
 
Регистрация: 23.03.2010
Сообщений: 51
Сказал(а) спасибо: 14
Поблагодарили 32 раз(а) в 15 сообщениях
Vinolentus На верном пути
По умолчанию

Vinolentus вне форума   Ответить с цитированием
Старый 20.05.2011, 07:47   #10
KiriX
Умный
Старожил
 
Аватар для KiriX
 
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 431 раз(а) в 181 сообщениях
Записей в дневнике: 4
KiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человек
По умолчанию

Vinolentus, убрать лишние поля из базы и их обработчик в коде - до минут 15. Я мог бы это сделать. Но сначала я хочу убедиться, что это будет принято. Если это нафиг никому не надо и всё прекрасно и так - я не стану тратить свои даже 15 минут...
KiriX вне форума   Ответить с цитированием
Старый 20.05.2011, 22:55   #11
Vladimir
MaNGOS Dev
 
Регистрация: 09.02.2010
Сообщений: 594
Сказал(а) спасибо: 315
Поблагодарили 440 раз(а) в 183 сообщениях
Vladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небес
По умолчанию

В UDB используется в 150-ти. Как не будут использоваться или мало останется так и удалим. Борьба с хаками это хорошо, но без экстремизма. Пример с SD2 - да могу дропнуть хоть сейчас GetGUID() - и что - девелоперы SD2 просто уписаются от счастья поставленные перед фактомм наличия нескольких тысяч ошибок сборки.
И сколько времени после этого не будет собьираться SD2, а может девелоперы просто плюнут и уйдут.
__________________
Так как устал объяснять знайте ICQ не пользуюсь
Vladimir вне форума   Ответить с цитированием
Старый 21.05.2011, 08:22   #12
KiriX
Умный
Старожил
 
Аватар для KiriX
 
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 431 раз(а) в 181 сообщениях
Записей в дневнике: 4
KiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человек
По умолчанию

Однако где гарантии, что сейчас девелоперы UDB стараются сделать эти кв правильно? А вот если бы изменения были - им пришлось бы отрыть информацию, что же действительно должно быть в квесте.
Я уже и не хотел упоминать, что это поле используется в кв где, к примеру, нужно излечить 10 кого-то там. Да, с этим полем это легко реализовать. И сделать потом тоже легко 10 раз на одного НПС кастануть.
Да и кто будет задумываться как правильно надо, если жалоб разрабам о этих кв не поступает - они ведь работают!
В общем печально, что разработчики ядра избрали именно такую позицию
KiriX вне форума   Ответить с цитированием
Старый 21.05.2011, 08:35   #13
Vladimir
MaNGOS Dev
 
Регистрация: 09.02.2010
Сообщений: 594
Сказал(а) спасибо: 315
Поблагодарили 440 раз(а) в 183 сообщениях
Vladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небес
По умолчанию

Ожидать что кто будет исправлять 150 квестов за раз не реально. И я думаю новых использований давно не добавлялось. Просто надо время чтобы исправить оставшиеся - я лично устраивать нездоровые сенсации и ставить девелоперов UDB пред фактом гемороя не собираюсь. Само по себе наличие полей не мешает ничему. Небудут нужны - удалим.
__________________
Так как устал объяснять знайте ICQ не пользуюсь
Vladimir вне форума   Ответить с цитированием
Старый 21.05.2011, 21:58   #14
YuruY
YTDB Dev
 
Аватар для YuruY
 
Регистрация: 01.02.2010
Сообщений: 286
Сказал(а) спасибо: 125
Поблагодарили 97 раз(а) в 53 сообщениях
YuruY Скоро придёт к известностиYuruY Скоро придёт к известности
По умолчанию

Vladimir
А иногда полезно пнуть. Сколько времени можно ждать (больше года) дроп полей в "creature_movement" т.к. в "creature_movement_scripts" они дублируются? Там делов на час по "перегнать", по базе (могу помочь перегнать).
__________________
http://ytdb.ru/
http://svn2.assembla.com/svn/ytdbase/
YuruY вне форума   Ответить с цитированием
5 пользователя(ей) сказали cпасибо:
Dragon (23.05.2011), Fedia22 (22.05.2011), Humort (23.05.2011), KiriX (21.05.2011), Lordronn (23.05.2011)
Старый 23.05.2011, 14:21   #15
Dragon
Новичок
 
Регистрация: 07.03.2010
Сообщений: 22
Сказал(а) спасибо: 33
Поблагодарили 2 раз(а) в 2 сообщениях
Dragon На верном пути
По умолчанию

Цитата:
Сообщение от Vladimir Посмотреть сообщение
я лично устраивать нездоровые сенсации и ставить девелоперов UDB пред фактом гемороя не собираюсь.
Можно предупредить сейчас и через некоторое время удалить, вариант?
Dragon вне форума   Ответить с цитированием
Старый 11.09.2012, 11:57   #16
KiriX
Умный
Старожил
 
Аватар для KiriX
 
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 431 раз(а) в 181 сообщениях
Записей в дневнике: 4
KiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человек
По умолчанию

Shmoo, what you think about drop field ReqSpellCast???
KiriX вне форума   Ответить с цитированием
Пользователь сказал cпасибо:
YuruY (11.09.2012)
Старый 11.09.2012, 13:00   #17
schmoozerd
MaNGOS Dev
 
Регистрация: 17.11.2011
Сообщений: 99
Сказал(а) спасибо: 35
Поблагодарили 80 раз(а) в 26 сообщениях
schmoozerd Скоро придёт к известности
По умолчанию

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

Последний раз редактировалось schmoozerd; 11.09.2012 в 13:13.
schmoozerd вне форума   Ответить с цитированием
Пользователь сказал cпасибо:
KiriX (11.09.2012)
Старый 11.09.2012, 14:18   #18
NeatElves
YTDB Dev
 
Аватар для NeatElves
 
Регистрация: 06.03.2010
Сообщений: 258
Сказал(а) спасибо: 28
Поблагодарили 280 раз(а) в 136 сообщениях
NeatElves Как самоцвет среди гранитаNeatElves Как самоцвет среди гранитаNeatElves Как самоцвет среди гранита
По умолчанию

+ http://www.ru-mangos.ru/showthread.php?t=5267 ))
NeatElves вне форума   Ответить с цитированием
Пользователь сказал cпасибо:
KiriX (11.09.2012)
Старый 11.09.2012, 14:48   #19
KiriX
Умный
Старожил
 
Аватар для KiriX
 
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 431 раз(а) в 181 сообщениях
Записей в дневнике: 4
KiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человек
По умолчанию

shmoo, and stop to look at UDB - they really slowly!!!
KiriX вне форума   Ответить с цитированием
Ответ

Опции темы Поиск в этой теме
Поиск в этой теме:

Расширенный поиск
Опции просмотра

Ваши права в разделе
Вы не можете создавать новые темы
Вы не можете отвечать в темах
Вы не можете прикреплять вложения
Вы не можете редактировать свои сообщения

BB коды Вкл.
Смайлы Вкл.
[IMG] код Вкл.
HTML код Выкл.


Похожие темы
Тема Автор Раздел Ответов Последнее сообщение
[9312] Implement DB storage of new quest_template field RewHonorMultiplier KiriX CMaNGOS Commits 18 31.10.2010 15:48
[10657] Separate quest_template.QuestFlags from SpecialFlags newsbot CMaNGOS Commits 0 30.10.2010 02:17
[10621] Add new field RequiredClasses for quest_template newsbot CMaNGOS Commits 0 19.10.2010 01:42
[9852] Disable forced alteration of specialFlags in quest_template at startup. newsbot CMaNGOS Commits 3 11.05.2010 08:40


Текущее время: 07:48. Часовой пояс GMT +3.


ru-mangos.ru - Русское сообщество MaNGOS
Главная цель проекта MaNGOS - обучающая, поэтому разрешается использовать исходный код и собранную программу только для образовательных целей.
Вы не можете использовать MaNGOS в коммерческих целях, а также не разрешается устанавливать публичные серверы на базе MaNGOS.
Любое копирование материалов, информации в любом виде без указания источника - форума Ru-MaNGOS будет считаться нарушением авторских прав и нарушением Уголовного Кодекса РФ, ст. 146 ст. 147.
Перевод vBulletin: zCarot