Мне вот интересно, кто-нибудь скачал уже этот плагин и проверил его в работе?
Я только посмотрел исходники.
1. Если команда введена неправильно - игрок, написавший команду, увидит надпись из config.yml, которая ему ничем не поможет. Потому что файл config.yml не дописан.
2. Игроки могут голосовать за мут самих себя. Забавно.
3. Если никто не проголосует (счет 0 - 0), плагин пишет: "Голоса распределились пор
авну". В принципе, не врёт. Кстати, логично было бы голос инициатора голосования добавить в счетчик голосов "за".
4. Если написать команду /vote yes(no), когда голосование не запущено, ответа не будет вообще.
5. Зачем в main классе строка:
if(event.getMessage().charAt(0) != '/') , если мы и так отслеживаем событие чата, а не команды?
6. Фраза "Вы не можете писать в чат!" недостаточно информативна. Непонятно, почему не можем писать. Что? Исправить через инкласстранслятор? А в конфиг добавить три (или четыре?) фразы этого плагина трудно?
7. Если никто на сервере не напишет /vote no (ну заняты были или лень, или тупо по барабану), то можно выдать мут игроку 2-3 голосами - то есть, буквально пары друзей хватит, чтобы выдать мут кому угодно. Эксплойт?
8. Можно выдать мут админам и даже опам. Это сильно.
9. Если сервер перезагрузить, то все муты обнулятся, так как информация о них содержится в памяти, а не в файле. Так что можете не ставить время мута особо большим, если ваш сервер периодически перезагружается.
10. Снова о фразах. Почему не выписать их в конфиг? Почему не добавить к ним цвета? Неужели это так трудно?
Это всё - только с точки зрения моего неопытного глаза.
В целом код, по моим меркам, добротный. Сравниваю со своими - мои ни в какое сравнение не идут. Пока "читал", почерпнул для себя пару неизвестных мне "хитростей".
Только непонятно, почему человек с подписью "Девелопер" не стыдится выкидывать сюда недоплагины, написанные на 50%. При этом в дефолт-конфиг вставляет строчки:
Код:
# спасибки сюда:
# R261034216073
# Z221643925630
Некрасиво.