CMD SHOW V6.0

Автор: Seroff
Версия: 6.0

Автор позволил :)


Описание:


Файлы:

Список кваров:


Список изменений:

deadkiller,
Что данный плагин делает?

Например у человека чит и он спамит обычно команда типа такого примера: gg_spam on стоит прописать в Containi.ini часть команды типа так - _spam и если он выполнит эту команду получит бан, так же за другие команды которые прописаны в ini файлах

deadkiller,
Спасибо! Крутой плагин буду использовать его у себя.

Лучше бы автор позволил опубликовать описание Передайте господину, что для хранения команд прекрасно подойдёт Trie, что снизит 'нагрузку' от плагина на порядки. Иммунитет-то зачем?

Согласен без описания ни как

Trie не дай бог, я видел плагин маздана не хотел бы такой плагин иметь, там эти Trie Trie Trie и падение сервера ну в баню

deadkiller, ты не прав насчёт Trie. Маасдам не показатель: он не зря котов в мешке выпускает

Шутка-минутка: Мастаман добавляет в плагины бэкдоры вот и вся его суть перерабатывания плагинов

Я писал что сервер спамит и крашится из-за Trie
Trie Alias Detector

deadkiller, во-первых, давай не будем кивать на маздана. Его код ‒ не образец для подражания. Во-вторых, довольно глупое заявление без пруфов, недалеко ушло от 'крашится из-за amxx', не верю. Давай не будем холиварить, моё дело предложить.

Я уже лично избегаю Trie плагины!

Мне без разницы, ты просто передай автору мои слова

Он сказал, хорошая идея, если найдется что еще добавить обновит с этой функцией.

deadkiller, Trie в плагине и Tried в том логе совершенно разные вещи, никак между собой не связаны, гореэксперт по крашам

Плагин вообще ни о чем.
По пунктам
1. Че за хардкод путей


2. Ничего что write_file каждый вызов открывает и закрывает файл. Плюс к времени смены карты
А также не обрабатывается ошибка записи в файл (банально нет прав)

3. оформление кода просто ужасное, пришлось вручную расставить отступы, чтобы хоть как-то разобраться в коде

4. Тупое недопонимание венгерской нотации. Либо использовать либо нет. i_Immunity, sBuffer. И в то же время g_ArrayBadCont, isFiledB. При этом sBuffer не может быть ибо по этой же нотации оно должно быть szBuffer так как zero-terminated string как у переменной szFileComment.
Туда же отнести переменные i_logs, i_bad, i_bantype. В одних приставка g_ (global) в других их отсутствие. Создает впечатление что плагин состоит из склейки кода из разных примеров или плагинов

5. for(i=0;i<ArraySize(g_ArrayNotShow);i++) - на каждую итерацию дергаем получение количества элементов в массиве? И это в часто вызываемом  client_command? И таких циклов несколько?

6. for(new client=1; client < 33; client++){ - хардкод. При том что макс количество не обязательно должно быть 32. MaxClietns или get_maxplayers не не слышал

7.
Серьёзно? А проверять на коннект игрока кто будет? Очередное недопонимание как нужно делать

8. public user_punish(id,const szReason[],any:...){ - зачем тут public?

9. message_begin(MSG_ONE, 8, {0,0,0}, id); - рили? Сложно получить ИД мессагы с помощью get_user_msgid?

Еще в придачу вот такой вот кусок кода

Мы и так проверили is_user_admin(id). Поэтому get_user_flags(id) & szFlag ? "Admin" : "Player" будет равно всегда Admin. Тут скорей всего опечатка.


Еще одно необдуманное решение. На каждый вызов client_command мы по сути открываем файл для записи, пишем туда строку, закрываем файл. И так для всех игроков в пределах одного кадра. И это учесть что это для всех команд от игрока без какого-либо фильтра. Очень дорогостоящая операция открытия файла так часто.

esterio,у вас там реально на гей-каэсе нубьё перевелось, что нашлось время поучить местное? Не-не, я не против, просто интересуюсь, для общего развития. Одна просьба: перед ответом ознакамливайся с написанным плз. За Array с циклами я уже сказал, просто по-другому: предложил Trie. Автор, ожидаемо, не придал большого значения, раз решил ждать оказии, и его не убедило магическое 'на порядки'. Всё, это клиника ‒ быдлокодер по жизни (знания и опыт не помогут). Вот GM-X стОит усилий, а это самовыражение через конструктивную критику ‒ не факт...

Автор предложил, если изменишь как будет лучше в плагине, вообще все на свое усмотрение, то добавить в авторство и будет для него познавательно

Он добавил:

Вопрос: