Оцените код плагина

[AlphaCH1337]

Решил попробовать программирование на Java, нашел интересный для себя проект.
Можете оценить код? Например, я не использую PermissonsEx.
Описание в репозитории
https://github.com/k-egor-smirnov/detour

 

[_MediaNik_]

Плохой код. В Main классе все функции, максимум строк - 100, Main класс для того создан, чтобы в нем регистрировать плагин и переменные, создавай классы, каждый из которых отвечает за свое.
Вот: https://github.com/MediaNik5/BossWars/tree/master/BossWars/src/ru/BossWars
P.S. В Main классе вообще ничего не должно быть, кроме onEnable, onDisable, onLoad, и переменных, которые используются только в Main или только в нем создаются.

 

[slavik123123123]

P.S. В Main классе вообще ничего не должно быть, кроме onEnable, onDisable, onLoad, и переменных, которые используются только в Main или только в нем создаются.

https://github.com/Dimatert9/AntiRelog/blob/master/src/ru/xtime/antirelog/Main.java
Аок

 

[_MediaNik_]

https://github.com/Dimatert9/AntiRelog/blob/master/src/ru/xtime/antirelog/Main.java
Аок

Ну я имел ввиду в его плагине...

 

[Code]

это ад

public void onQuit(PlayerQuitEvent event) {
    boolean isFound = false;
    if (findDoNotDelete(event.getPlayer().getName())) {
        isFound = true;
        return;
    }
    if (!isFound) {
        delete(event.getPlayer().getName());
        return;
    }
}

должно быть просто

public void onQuit(PlayerQuitEvent event) {
    if (!findDoNotDelete(event.getPlayer().getName())) {
        delete(event.getPlayer().getName());
    }
}

вместо вот этого говна:

public boolean findDoNotDelete(String player) {
    for (int i = 0; i < doNotDelete.size(); i++) {
        if (doNotDelete.get(i) == player) {
            return true;
        }
    }
    return false;
}

есть

public boolean findDoNotDelete(String player) {
    return doNotDelete.contains(player);
}

 

[slavik123123123]

Импорт замени, иначе кроме bungee твой плагин нигде работать не будет.

Нет. Плагин будет работать только на Spigot. На CraftBukkit не будет.

 

[AlphaCH1337]

Импорт замени, иначе кроме bungee твой плагин нигде работать не будет.

Это импортировалось автоматически, в классе settings у меня импорт org.bukkit.ChatColor;

 

[slavik123123123]

https://github.com/k-egor-smirnov/detour/pull/1
https://github.com/k-egor-smirnov/detour/pull/2
))

 

[AlphaCH1337]

Вопрос не по теме.
Почему не запускается проект в Eclipse?

 

[slavik123123123]

Вопрос не по теме.
Почему не запускается проект в Eclipse?

создаешь тупо проект, и переносишь всё нужное в папку src мышкой.

 

[AlphaCH1337]

https://github.com/k-egor-smirnov/detour/pull/1
https://github.com/k-egor-smirnov/detour/pull/2
))

java.lang.ClassNotFoundException: ru.alphach1337.detour.Main

Хотя он есть

 

[slavik123123123]

Хотя он есть

 

[AlphaCH1337]

Хотя он есть

Оказывается, не тот спигот прикрутил к проекту
Спасибо за помощь! Тема закрыта