Fishing plus 1.3

[Kelsi]

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

Fishing plus - Обновленная система рыбалки

О плагине:
Привет!
Fishing Plus- плагин изменяющий систему рыбалки в Майнкрафт


Список рыб:
- треска
- лосось
- рыба фугу
- тропическая рыба

Особенности:
• Вместо предмета рыбы будет ловиться моб
• Есть шанс, что рыба сорвётся с крючка

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

 

[BlackBaroness]

Тебе стоит:
- Использовать gitignore и прикладывать pom.xml к репозиторию
- Изучить нейминг. В частности, классы всегда называются с большой буквы
- Исправлять опечатки. Слово succes повсюду
- Убрать эти странные методы конфига в начале и заменить простым saveDefaultConfig
- Добавить побольше инфы. Добыча рыб ввиде мобов звучит интересно, но без какого нибудь скриншота или видео не каждый станет качать и пробовать у себя

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

 

[BuseSo]

Я бы за такое побил.

Спойлер: ААААА!!!!

Вам необходимо зарегистрироваться для просмотра изображений-вложений

Вам необходимо зарегистрироваться для просмотра изображений-вложений

1) Зачем глобальные переменные в главном классе? Избавься от них.
2) Зачем скопление if, else if, else if, else if с непонятными цифрами, если можно сделать switch от нуля до какой-то там цифры? Я бы вообще сделал шансы, которые можно было бы настроить через конфиг.
3) А зачем там вообще саут на втором скрине в конце метода fish_randomfunc? Удали, будет консоль захламлять.
4) Не используй _ в методах, а название классов пиши с большой буквы.
5) Офигел от жизни: EntityType.valueOf(String.valueOf(fish))
Мне это напомнило миниатюру типа

boolean bool = false;
if (String.valueOf(false).equals(String.valueOf(bool)) == false) {}

Переделай на нормально.
6) А почему цвета нельзя редактировать? Вернее, можно, но это придётся использовать §, вместо привычного &, плак. Добавь и убери ChatColor.ЦВЕТ.
7)

You must be registered for see links

у тебя уже есть player, зачем p? Убери одно из них.

 

[BlackBaroness]

4) Не используй _ в методах, а название классов пиши с большой буквы.

Да и в полях редко разрешается, разве что в константах

 

[Kelsi]

Я бы за такое побил.

Спойлер: ААААА!!!!

Вам необходимо зарегистрироваться для просмотра изображений-вложений

Вам необходимо зарегистрироваться для просмотра изображений-вложений

1) Зачем глобальные переменные в главном классе? Избавься от них.
2) Зачем скопление if, else if, else if, else if с непонятными цифрами, если можно сделать switch от нуля до какой-то там цифры? Я бы вообще сделал шансы, которые можно было бы настроить через конфиг.
3) А зачем там вообще саут на втором скрине в конце метода fish_randomfunc? Удали, будет консоль захламлять.
4) Не используй _ в методах, а название классов пиши с большой буквы.
5) Офигел от жизни: EntityType.valueOf(String.valueOf(fish))
Мне это напомнило миниатюру типа

boolean bool = false;
if (String.valueOf(false).equals(String.valueOf(bool)) == false) {}

Переделай на нормально.
6) А почему цвета нельзя редактировать? Вернее, можно, но это придётся использовать §, вместо привычного &, плак. Добавь и убери ChatColor.ЦВЕТ.
7)

You must be registered for see links

у тебя уже есть player, зачем p? Убери одно из них.

Уф... Сколько костылей и ошибок я сделал... Могу оправдаться, что это первый плагин. Постараюсь в ближайшем будущем исправить свои ошибки, благодарю за критику