Poll 1.1

[SonG]

SonG добавил(а) новый ресурс:

Poll - Плагин опроса игроков сервера

Плагин создан для опроса игроков сервера на интересующие вас вопросы (например стоит ли убрать у випов те или иные функции и т.п.) Результаты голосования заносятся в БД.
Так же присутствует проверка на время, которое игрок провёл на сервере (не каждый владелец сервера готов принимать голос школьника который зашел на сервер случайно на 5 мин)

Плагин был создан на заказ – ТЗ

Узнать больше об этом ресурсе...

 

[Sonyx]

Работа с БД тихий ужас - при нестабильном соединении сервер не будет выходить из фризов. Плюс ко всему на 1 действие сразу 2 селекта??? Еще и один с вложенным запросом? Это из разряда "Как усложнить то что максимально просто"?

 

[SergeyShorokhov]

Sonyx, в добавок критике нужно дать полезных советов.
Nunfy Принимаешь советы по коду? Я бы тоже подсказал несколько более верных решений.

[POLL_0] <-- почему тут сразу не писать наименование вопроса?
Как дела? <-- вместо этого
Отлично
Нормально
Не очень
Плохо

 

[Nunf]

wopox1337, конечно

 

[Denzer]

// ctrl + f "table_name", выбераем сразу всё(или поочередно) заменяем этот текст на название свой таблицы в mysql

почему нельзя вынести название таблицы в константу?

 

[fl0wer]

Хотел написать что-то по поводу кода и даже не знаю к чему придраться. К тройному массиву? К работе с бд? К логики кода? Тихий ужас... Зачем такое писать я нипанимаю

 

[SergeyShorokhov]

fl0wer, пиши по делу, по существу. Конкретные предложения, а не бессмысленные слова

 

[fl0wer]

wopox1337, если по существу, то вот: mysql_turple = SQL_MakeDbTuple

 

[SonG]

Ох, а ведь это еще fantom не высказался)))

 

[fantom]

register_menu("\r", 1023, "main_menu_handler");

Хендлер меню сработает на все меню которые будут иметь \r в тексте. Смотреть тут

Работа с базой в синхкронном режиме, возможны фризы.

В 1.8.3 уже есть инструмент для парсинга INI файлов

Загрузка игрока идет на client_connect который может быть вызван двавжды и в моент которого получение steamid не гарантировано.

SQL_MoreResults не к чему там. Лучше проверять SQL_NumResults.

В запросах лучше установить LIMIT 1 дабы не продолжать поиск дальше

Имя карты может быть изменено плагином mode2x2 и ему подобных. Получим конфуз

Хукы PRE а это не гарантирует их исполнение (хотя в большинстве случаев все же будут вызваны)

else if(get_user_gametime(id) < MIN_USER_GAME_TIME * 60)

Не понятно откуда тут id взялся если аргументом хука this. Скорей всего опечатка

 

[gil5587]

When I try to compile, I get this error:
// F:\addons\amxmodx\scripting\poll.sma(89) : error 017: undefined symbol "get_user_gametime"

 

[Алексеич]

gil5587, https://dev-cs.ru/forums/compile-errors/post-thread

 

[SonG]

Буквально вчера дошли руки для теста плагина на сервере.
fantom ты говорил о проблеме с меню. Сегодня поставил плагин и игроки стали жаловаться, что именно после смерти открывается вип меню которое невозможно закрыть. Это связано ?
UPD
Да, связано. Т.к. после отключения плагина, меню больше не выпадает и закрывается корректно.

 

[SonG]

Ресурс Poll был обновлен:

Обновление до версии 1.1

Нажмите на эту ссылку, чтобы узнать подробности обновления

 

[Knife]

Ребятки есть скриншотик ?!

 

[Hunter2017]

[ 91] unknown                 unknown     unknown           poll.amxx        bad load

added debug -> no errors showing. Compiled with 1.8.3 Compiler. Plugin dont work. Server is using REHLDS.

[AMXX] Plugin "poll.amxx" failed to load: Module/Library "csstatsx_sql" required for plugin.  Check modules.ini.

 

[EnemyDown]

Судя по ТЗ, по которому делался плагин.

Голосование происходит только когда игрок мёртв. Если начинается раунд и игрок не успел проголосовать, то меню с выбором ответа закрывается автоматически до след. смерти игрока и так пока не проголосует.

Проводить опросы на csdm сервере не получится? Или как то можно?