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

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

 

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

 

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

 

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

 

это ад

Код:

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);
}

 

Если понадобится получать группу игрока, префикс - используй Vault.

Код:

import net.md_5.bungee.api.ChatColor;

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

Код:

Bukkit.getPlayer(sender.getName()).getLocation())

NPE словишь, если вызовешь команду из консоли.

Код:

if(sender instanceof Player) {
Player player = (Player)sender; // команда вызвана игроком
} else {
// команда вызвана из консоли, командным блоком или ещё чем-нибудь
}

 

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

 

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

 

https://github.com/k-egor-smirnov/detour/pull/1[DOUBLEPOST=1470500377,1470500050][/DOUBLEPOST]https://github.com/k-egor-smirnov/detour/pull/2
))

 

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

 

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

 

Хотя он есть

 

Оказывается, не тот спигот прикрутил к проекту[DOUBLEPOST=1470501878,1470501763][/DOUBLEPOST]Спасибо за помощь! Тема закрыта