YTDB

Вернуться   YTDB > Корзина / Trash > Архив MaNGOS (пока не было ru-mangos) > Патчи

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

Если кто-то хочет задать вопрос по патчу - лучше не на этом форуме вообще.

Ответ
 
Опции темы Поиск в этой теме Опции просмотра
Старый 15.11.2009, 18:54   #1
tempura
Not a member, is a noob
 
Аватар для tempura
 
Регистрация: 05.11.2009
Адрес: Азерот, Штормвинд
Сообщений: 530
Сказал(а) спасибо: 153
Поблагодарили 453 раз(а) в 170 сообщениях
tempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человек
По умолчанию GuildHouse патч валит сервер при раскупленых ГХ

Патч guildmaster.cpp на SD2.

Вот баг-репорт получил.
Цитата:
пишут как гх все раскупили при попытке купить -серв падает. Есть у тебя затычка?
Сам пока не имею возможности проверить и исправить. Кто-то имеет возможность помочь хотя бы в проверке?

Ревизии - все (он не меняется).

Тема в разделе патчей на ядро.
__________________
Совершенно безопасен для людей, обладающих хотя бы некоторыми минимальными зачатками интеллекта, и способными строить причинно-следственные цепочки.
tempura вне форума   Ответить с цитированием
Старый 02.12.2009, 12:00   #2
Mr.Grom
Not a Junior!
 
Аватар для Mr.Grom
 
Регистрация: 07.11.2009
Сообщений: 10
Сказал(а) спасибо: 4
Поблагодарили 45 раз(а) в 12 сообщениях
Mr.Grom Скоро придёт к известности
По умолчанию

Вот место которое отвечает за работу патча, если все гх куплены

PHP код:
        if (showFromId 0)
        {
            
//all guildhouses are occupied
            
_creature->MonsterWhisper(MSG_NOFREEGHplayer->GetGUID());
            
player->CLOSE_GOSSIP_MENU();
        } else
        {
            
//this condition occurs when COUNT(guildhouses) % GOSSIP_COUNT_MAX == 0
            //just show GHs from beginning
            
showBuyList(player_creature0);
        } 
Возможно должно быть
PHP код:
if (showFromId == NULL
или
PHP код:
 if (!showFromId

Последний раз редактировалось Mr.Grom; 02.12.2009 в 13:35.
Mr.Grom вне форума   Ответить с цитированием
Старый 02.12.2009, 12:27   #3
tempura
Not a member, is a noob
 
Аватар для tempura
 
Регистрация: 05.11.2009
Адрес: Азерот, Штормвинд
Сообщений: 530
Сказал(а) спасибо: 153
Поблагодарили 453 раз(а) в 170 сообщениях
tempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человек
По умолчанию

кто бы протестил. А то мне сейчас тупо негде и не на чем.
__________________
Совершенно безопасен для людей, обладающих хотя бы некоторыми минимальными зачатками интеллекта, и способными строить причинно-следственные цепочки.
tempura вне форума   Ответить с цитированием
Старый 02.12.2009, 13:08   #4
real
Guest
 
Сообщений: n/a
Exclamation

скорее всего 1 вариант
PHP код:
if (showFromId == 0)     
        { 
  Ответить с цитированием
Старый 02.12.2009, 13:36   #5
Mr.Grom
Not a Junior!
 
Аватар для Mr.Grom
 
Регистрация: 07.11.2009
Сообщений: 10
Сказал(а) спасибо: 4
Поблагодарили 45 раз(а) в 12 сообщениях
Mr.Grom Скоро придёт к известности
По умолчанию

Тогда уж
PHP код:
if (showFromId == NULL
но красивее.
PHP код:
if (!showFromId
Mr.Grom вне форума   Ответить с цитированием
Старый 02.12.2009, 18:24   #6
tempura
Not a member, is a noob
 
Аватар для tempura
 
Регистрация: 05.11.2009
Адрес: Азерот, Штормвинд
Сообщений: 530
Сказал(а) спасибо: 153
Поблагодарили 453 раз(а) в 170 сообщениях
tempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человек
По умолчанию

собственно предмет


уже правлено.
__________________
Совершенно безопасен для людей, обладающих хотя бы некоторыми минимальными зачатками интеллекта, и способными строить причинно-следственные цепочки.
tempura вне форума   Ответить с цитированием
Старый 07.12.2009, 01:21   #7
Mr.Grom
Not a Junior!
 
Аватар для Mr.Grom
 
Регистрация: 07.11.2009
Сообщений: 10
Сказал(а) спасибо: 4
Поблагодарили 45 раз(а) в 12 сообщениях
Mr.Grom Скоро придёт к известности
По умолчанию

Если ГиМ меняет( по незнанию прописывает его сам в другом городе) дом то ГХ для гильды недоступен.

Последний раз редактировалось Mr.Grom; 07.12.2009 в 11:46.
Mr.Grom вне форума   Ответить с цитированием
Старый 07.12.2009, 11:38   #8
tempura
Not a member, is a noob
 
Аватар для tempura
 
Регистрация: 05.11.2009
Адрес: Азерот, Штормвинд
Сообщений: 530
Сказал(а) спасибо: 153
Поблагодарили 453 раз(а) в 170 сообщениях
tempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человек
По умолчанию

"меняет", в смысле - продает один, и покупает другой?


Кажется скоро придется все нафиг переписывать под Gossip System...
__________________
Совершенно безопасен для людей, обладающих хотя бы некоторыми минимальными зачатками интеллекта, и способными строить причинно-следственные цепочки.
tempura вне форума   Ответить с цитированием
Старый 07.02.2010, 18:25   #9
Smak
Guest
 
Сообщений: n/a
По умолчанию

Так не роняет:
Код:
if (!showFromId)
  Ответить с цитированием
Пользователь сказал cпасибо:
tempura (07.02.2010)
Старый 07.02.2010, 21:13   #10
Chesterfield
Guest
 
Сообщений: n/a
По умолчанию

Цитата:
if (showFromId == NULL)
NULL это макрос того же самого нуля, который был указан.
  Ответить с цитированием
Старый 07.02.2010, 23:06   #11
tempura
Not a member, is a noob
 
Аватар для tempura
 
Регистрация: 05.11.2009
Адрес: Азерот, Штормвинд
Сообщений: 530
Сказал(а) спасибо: 153
Поблагодарили 453 раз(а) в 170 сообщениях
tempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человек
По умолчанию

Ну в оригинале-то там вообще
Код:
if (showFromId = 0)
стоит.
__________________
Совершенно безопасен для людей, обладающих хотя бы некоторыми минимальными зачатками интеллекта, и способными строить причинно-следственные цепочки.
tempura вне форума   Ответить с цитированием
Старый 07.02.2010, 23:15   #12
Konctantin
RuDB Dev
 
Аватар для Konctantin
 
Регистрация: 06.11.2009
Сообщений: 677
Сказал(а) спасибо: 592
Поблагодарили 690 раз(а) в 289 сообщениях
Konctantin Великолепный пример подражанияKonctantin Великолепный пример подражанияKonctantin Великолепный пример подражанияKonctantin Великолепный пример подражанияKonctantin Великолепный пример подражанияKonctantin Великолепный пример подражания
По умолчанию

if (showFromId = 0) - означает, что переменной showFromId присваивается значение 0
А когда == или !showFromId означает что идет проверка на значение.
Konctantin вне форума   Ответить с цитированием
Старый 07.02.2010, 23:47   #13
tempura
Not a member, is a noob
 
Аватар для tempura
 
Регистрация: 05.11.2009
Адрес: Азерот, Штормвинд
Сообщений: 530
Сказал(а) спасибо: 153
Поблагодарили 453 раз(а) в 170 сообщениях
tempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человек
По умолчанию

вау.... завидую.

Давайте я вам баллистику СЦ-130ВПС расскажу. Обещаю, что вы в этом тоже ничего не поймете.


Как я понимаю - безразлично что писать:
Код:
if (showFromId = 0) // присвоение значения
if (showFromId == NULL) // проверка на значение
if (!showFromId) // проверка на значение
то есть все три строки - равнозначны? Или же все-таки какая-то из них содержит ошибку? При первой пишут, что "валит сервер при раскупленых ГХ", при последней пишут что не валит. Не понимаю.
__________________
Совершенно безопасен для людей, обладающих хотя бы некоторыми минимальными зачатками интеллекта, и способными строить причинно-следственные цепочки.
tempura вне форума   Ответить с цитированием
Старый 07.02.2010, 23:51   #14
timmit
YTDB dev, histori
 
Аватар для timmit
 
Регистрация: 05.11.2009
Сообщений: 1,597
Сказал(а) спасибо: 1
Поблагодарили 1,093 раз(а) в 971 сообщениях
timmit За него можно гордитьсяtimmit За него можно гордитьсяtimmit За него можно гордитьсяtimmit За него можно гордитьсяtimmit За него можно гордитьсяtimmit За него можно гордитьсяtimmit За него можно гордитьсяtimmit За него можно гордиться
По умолчанию

Первая дает ошибку
Код:
if (showFromId = 0) // присвоение значения
Идентичны:
Код:
if (showFromId == NULL) // проверка на значение
if (!showFromId)
__________________
ars longa vita brevis
timmit вне форума   Ответить с цитированием
Старый 07.02.2010, 23:54   #15
tempura
Not a member, is a noob
 
Аватар для tempura
 
Регистрация: 05.11.2009
Адрес: Азерот, Штормвинд
Сообщений: 530
Сказал(а) спасибо: 153
Поблагодарили 453 раз(а) в 170 сообщениях
tempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человек
По умолчанию

А, вот теперь понял. Тогда последняя строка и правда красивее смотрится. Везде поправил.
__________________
Совершенно безопасен для людей, обладающих хотя бы некоторыми минимальными зачатками интеллекта, и способными строить причинно-следственные цепочки.
tempura вне форума   Ответить с цитированием
Старый 07.02.2010, 23:57   #16
Konctantin
RuDB Dev
 
Аватар для Konctantin
 
Регистрация: 06.11.2009
Сообщений: 677
Сказал(а) спасибо: 592
Поблагодарили 690 раз(а) в 289 сообщениях
Konctantin Великолепный пример подражанияKonctantin Великолепный пример подражанияKonctantin Великолепный пример подражанияKonctantin Великолепный пример подражанияKonctantin Великолепный пример подражанияKonctantin Великолепный пример подражания
По умолчанию

Код:
int i = 0;
int value = 5;
i = value; // теперь i равно 5

if (i = value)  // это ошибка так нельзя писать

if (i == value)
	return true;
	
if (!value)
	return false;
	
if (value == null)
	return false;
Konctantin вне форума   Ответить с цитированием
Старый 08.02.2010, 00:23   #17
tempura
Not a member, is a noob
 
Аватар для tempura
 
Регистрация: 05.11.2009
Адрес: Азерот, Штормвинд
Сообщений: 530
Сказал(а) спасибо: 153
Поблагодарили 453 раз(а) в 170 сообщениях
tempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человекtempura Реально хороший человек
По умолчанию

Ну тогда я про баллистику не буду, но популярные сведения предоставлю.


Ага, вот это я кажется понял:
Код:
i = value; // теперь i равно 5

if (i = value)  // это ошибка так нельзя писать
Тут value и так присваивается в предыдущей строке, поэтому запрос "если i = value" - некорректный.
__________________
Совершенно безопасен для людей, обладающих хотя бы некоторыми минимальными зачатками интеллекта, и способными строить причинно-следственные цепочки.
tempura вне форума   Ответить с цитированием
Старый 08.02.2010, 06:53   #18
griffonheart
Guest
 
Сообщений: n/a
По умолчанию

Цитата:
Сообщение от tempura Посмотреть сообщение
Код:
if (i = value)  // это ошибка так нельзя писать
Это семантическая ошибка, но не синтаксическая. Так писать можно, но в данном контексте не правильно.
if(i = value) - означает следующее:
1) Присваиваем переменной i значение переменной value
2) Если результат присваивания - истина - условии выполняется, если ложь - не выполняется. А рельзтат может быть ложью, если допустим значение переменной value ложно (равно нулю, например)
PS: Если что, извиняюсь за боян...
  Ответить с цитированием
Ответ

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

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

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

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


Похожие темы
Тема Автор Раздел Ответов Последнее сообщение
[патч] Релоад локалес госсип меню NeatElves Принятые патчи / Accepted patches 1 25.01.2010 11:06
[патч] Релоад госсип скрипт NeatElves Принятые патчи / Accepted patches 5 24.01.2010 09:03
Вешает сервер каст босса Джаналай tempura Ядро 3 05.12.2009 10:22


Текущее время: 20:50. Часовой пояс GMT +4.


YTDB - MaNGOS DataBase

Русский форум Мангос - официальный форум RMDC

Главная цель проекта MaNGOS - обучающая, поэтому разрешается использовать исходный код и собранную программу только для образовательных целей.
Вы не можете использовать MaNGOS в коммерческих целях, а также не разрешается устанавливать публичные серверы на базе MaNGOS.
Любое копирование материалов, информации в любом виде без указания источника - форума YTDB будет считаться нарушением авторских прав и нарушением Уголовного Кодекса РФ, ст. 146 ст. 147.
Перевод vBulletin: zCarot
Punk's On-Air - Punk, Folk, Blues, Ska Bascinet - реконструкция denw IL-2 Training Книготорговая Компания «А-5» - художественная, учебная литература, скидки, опт и мелкий опт, доставка, дешево, низкие цены издательства