[BugTraq] команда deal -1 кидает сделку левому игроку

BugTraq: обнаруженные ошибки и обсуждение их устранения.

Модератор: 4epT

Аватара пользователя
4epT
macro-маньячина
Сообщения: 2792
Зарегистрирован: Чт дек 21, 2006 1:23 pm
Сервер RO:: 4game
Discord: ya4ept#8494
Контактная информация:

[BugTraq] команда deal -1 кидает сделку левому игроку

Сообщение 4epT »

в общем если написать в консоле "deal -1" то бот кидает сделку какому-либо игроку, а по идее не должен никому кидать, т.к. игроки у нас начинаются с нуля :x

пример макроса:

Код: Выделить всё

macro test {
	log @player(test)
	do deal @player(test)
}
если по близости нету игрока "test" то @player(test) даёт нам "-1" и получается мы кидаем сделку левому игроку!

В общем придумал такую заплатку:
src\Commands.pm:

Код: Выделить всё

+} elsif ($arg[0] =~ /\d+/ && !$playersID[$arg[0]] || $arg[0] eq "-1") {
-} elsif ($arg[0] =~ /\d+/ && !$playersID[$arg[0]]) {
	error TF("Error in function 'deal' (Deal a Player)\n" .
		"Player %s does not exist\n", $arg[0]);
не уверен что это оптимальное решение..
надо бы добавить в SVN
Быстро и качественно напишу конфиг (макрос)! Стучи!
¤ Свежий бот ¤ Config checker ¤ Manual ¤
Изображение
Изображение
Аватара пользователя
piroJOKE
Модератор
Сообщения: 8205
Зарегистрирован: Сб ноя 04, 2006 2:20 am
Сервер RO:: localhost
Откуда: Molvania

Re: [BugTraq] команда deal -1 кидает сделку левому игроку

Сообщение piroJOKE »

Хм.... А может оставить это как есть - типо, не баг это, а фича, которая может когда-то пригодиццо? ;) ;) ;)

ЗЫ: А он часом не последнему игроку в списке кидает, ы?
Use brain against brain, ai against ai... · как правильно задавать вопросы · faq · download
Аватара пользователя
4epT
macro-маньячина
Сообщения: 2792
Зарегистрирован: Чт дек 21, 2006 1:23 pm
Сервер RO:: 4game
Discord: ya4ept#8494
Контактная информация:

Re: [BugTraq] команда deal -1 кидает сделку левому игроку

Сообщение 4epT »

ну я в толпе не проверял, из 2-х игроков она второму точно кидает :)

Добавлено спустя 47 секунд:
а как оно нам может пригодиться?
Быстро и качественно напишу конфиг (макрос)! Стучи!
¤ Свежий бот ¤ Config checker ¤ Manual ¤
Изображение
Изображение
Аватара пользователя
kLabMouse
Профессионал
Сообщения: 4776
Зарегистрирован: Вс ноя 05, 2006 4:32 pm

Re: [BugTraq] команда deal -1 кидает сделку левому игроку

Сообщение kLabMouse »

Заплатка не Ахти.
Думаем.... Лучше превратить строку в число, и приверить больше ли она нуля.
Ответить