Ru-MaNGOS

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

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

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

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

Ответ
 
Опции темы Поиск в этой теме Опции просмотра
Старый 30.11.2010, 22:04   #1
KiriX
Умный
Старожил
 
Аватар для KiriX
 
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 433 раз(а) в 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
Поблагодарили 433 раз(а) в 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
Поблагодарили 433 раз(а) в 181 сообщениях
Записей в дневнике: 4
KiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человек
По умолчанию

Владимир, можно всё-таки услышать ваш комментарий по этому поводу?
KiriX вне форума   Ответить с цитированием
Старый 18.05.2011, 23:36   #6
Humort
Гость
 
Сообщений: n/a
По умолчанию

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

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

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

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

P.S. Не буду упоминать целую стопку других, не менее важных проблем, которыми никто не хочет заниматься.....
  Ответить с цитированием
Старый 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
Поблагодарили 433 раз(а) в 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
Поблагодарили 438 раз(а) в 181 сообщениях
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
Поблагодарили 433 раз(а) в 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
Поблагодарили 438 раз(а) в 181 сообщениях
Vladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небес
По умолчанию

Ожидать что кто будет исправлять 150 квестов за раз не реально. И я думаю новых использований давно не добавлялось. Просто надо время чтобы исправить оставшиеся - я лично устраивать нездоровые сенсации и ставить девелоперов UDB пред фактом гемороя не собираюсь. Само по себе наличие полей не мешает ничему. Небудут нужны - удалим.
__________________
Так как устал объяснять знайте ICQ не пользуюсь
Vladimir вне форума   Ответить с цитированием
Старый 21.05.2011, 21:58   #14
YuruY
YTDB Dev
 
Аватар для YuruY
 
Регистрация: 01.02.2010
Сообщений: 288
Сказал(а) спасибо: 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), 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
Поблагодарили 433 раз(а) в 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
Сообщений: 259
Сказал(а) спасибо: 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
Поблагодарили 433 раз(а) в 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


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


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