[Вопрос] MYSQL

MYSQL

PHP код:

CMD:offban(playerid, params[]) { if(!pInfo[playerid][pAdmin]) return 1; if(sscanf(params, "s[24]is[30]", params[0], params[1], params[2])) return SendClientMessage(playerid, COLOR_GREY, "[Ошибка] Используйте: /offban [имя игрока] [кол-во дней] [причина]"); if(params[1] < 1 || params[1] > 30) return SendClientMessage(playerid, COLOR_GREY, "[Ошибка] Количество дней может быть от 1 до 30"); new format_string[200]; format(format_string, 200, "SELECT * FROM `ban_list` WHERE `name` = '%s'", params[0]); return mysql_tquery(connectionHandle, format_string, "Offban", "iiss", playerid, params[1], params[0], params[2]); }

PHP код:

forward Offban(playerid, days, str[], tmp[]); public Offban(playerid, days, str[], tmp[]) { new rows; cache_get_row_count(rows); if(rows) return SendClientMessage(playerid, COLOR_GREY, "[Ошибка] Данный аккаунт уже заблокирован"); new format_string[200]; format(format_string, 140, "Администратор %s оффлайн забанил игрока %s на %i дней. Причина: %s", pInfo[playerid][pName], str, days, tmp); SendClientMessageToAll(COLOR_LIGHTRED, format_string); format(format_string, 200, "INSERT INTO `ban_list` (`name`, `date`, `reason`, `unbandate`, `aname`) VALUES ('%s', NOW(), '%s', %i, '%s')", str, tmp, gettime() + 86400 * days, pInfo[playerid][pName]); return mysql_tquery(connectionHandle, format_string, "", ""); }

В итоге смешивается str и tmp и в имени игрока получается что-то вроде *первая буква str пробел tmp*

К примеру:
// Введенные данные: params[0] = test, params[1] = 1, params[2] = Бан
// Вывод в SendClientMessageToAll -> "Администратор name забанил игрока t Бан на 1 дней. Причина: Бан
// Ну и соответственно в имя игрока в таблицу выводится t Бан

С чем это связано? Как решить эту проблему?

С таймером похожий баг был. Лучше передавай текст PVar-ом или переменной

//DeimoS: Да, проблема именно в таймере. Не умеет он нормально передавать несколько строк

//Seviel: Вот так ошибаются люди, я не прав.

Не слушай бред выше. Вроде уже много раз обсуждалось, что использование params[0], params[1], ничем не оправдывает себя, только запутывает код, да и доступ к одному массиву проигрывает в скорости. Так по поводу одного массива - как ты хочешь в одном массиве записать две текстовые переменные?

Код команды должен быть примерно таким:

PHP код:

CMD:offban(playerid, params[]) { if(!pInfo[playerid][pAdmin]) return 1; extract params -> new string:user[24+1], days, string:reason[144]; else return SendClientMessage(playerid, COLOR_GREY, !"[Ошибка] Используйте: /offban [имя игрока] [кол-во дней] [причина]"); if(days < 1 || days > 30) return SendClientMessage(playerid, COLOR_GREY, !"[Ошибка] Количество дней может быть от 1 до 30"); new format_string[42+24+1]; format(format_string, sizeof(format_string), "SELECT * FROM `ban_list` WHERE `name` = '%s'", user); return mysql_tquery(connectionHandle, format_string, "Offban", "iiss", playerid, days, user, reason); }

Цитата:

Сообщение от 123

Не слушай бред выше. Вроде уже много раз обсуждалось, что использование params[0], params[1], ничем не оправдывает себя, только запутывает код, да и доступ к одному массиву проигрывает в скорости. Так по поводу одного массива - как ты хочешь в одном массиве записать две текстовые переменные?

Код команды должен быть примерно таким:

PHP код:

CMD:offban(playerid, params[]) { if(!pInfo[playerid][pAdmin]) return 1; extract params -> new string:user[24+1], days, string:reason[144]; else return SendClientMessage(playerid, COLOR_GREY, !"[Ошибка] Используйте: /offban [имя игрока] [кол-во дней] [причина]"); if(days < 1 || days > 30) return SendClientMessage(playerid, COLOR_GREY, !"[Ошибка] Количество дней может быть от 1 до 30"); new format_string[42+24+1]; format(format_string, sizeof(format_string), "SELECT * FROM `ban_list` WHERE `name` = '%s'", user); return mysql_tquery(connectionHandle, format_string, "Offban", "iiss", playerid, days, user, reason); }

А exctract ни как не запутывает код?
И что в твоем варианте изменилось что у него заработает код, кроме как ты обращение сделал к объявленным массивам, а не params[0], params[1]?
Я бы тоже параметры лучше передавал PVar'ами

Научитесь пользоваться поисковыми системами и поиском на форуме.

Я не знал в чем корень проблемы, как я мог найти данную тему?
Разве форумы не созданы для того, чтобы люди могли указывать на корень проблемы и ее решение?

Спасибо за указанную тему, премного благодарен.

Цитата:

Сообщение от $continue$

Научитесь пользоваться поисковыми системами и поиском на форуме.

Мммде уж, прикол в том, что ничего не изменилось

Цитата:

Сообщение от 123

Не слушай бред выше. Вроде уже много раз обсуждалось, что использование params[0], params[1], ничем не оправдывает себя, только запутывает код, да и доступ к одному массиву проигрывает в скорости. Так по поводу одного массива - как ты хочешь в одном массиве записать две текстовые переменные?

Код команды должен быть примерно таким:

PHP код:

CMD:offban(playerid, params[]) { if(!pInfo[playerid][pAdmin]) return 1; extract params -> new string:user[24+1], days, string:reason[144]; else return SendClientMessage(playerid, COLOR_GREY, !"[Ошибка] Используйте: /offban [имя игрока] [кол-во дней] [причина]"); if(days < 1 || days > 30) return SendClientMessage(playerid, COLOR_GREY, !"[Ошибка] Количество дней может быть от 1 до 30"); new format_string[42+24+1]; format(format_string, sizeof(format_string), "SELECT * FROM `ban_list` WHERE `name` = '%s'", user); return mysql_tquery(connectionHandle, format_string, "Offban", "iiss", playerid, days, user, reason); }

Использование отдельных переменных для каждого из params позволило решить данную проблему. Спасибо за подсказку.

Тему можно закрывать.