mc.plarmi.Core.java:5
Java:
import net.milkbowl.vault.economy.Economy;
Если Vault будет отсутствовать, то в консолине будет исключение об ошибке загрузки нужного класса. Не есть хорошо. Лучше в onEnable() проверять наличие плагина, и если он есть, то из другого класса подгружать его.
mc.plarmi.Core.java:? (unknown)
Java:
public void onEnable() {
PluginManager pm = this.getServer().getPluginManager();
pm.registerEvents(this, this);
//...
this.getServer().getPluginManager().registerEvents(this, this);
Хоть мы и живём во времена, где всем пофигу на занятость лишних килобайтов, но всё же делать ссылку для одного действия... Как то мажорно.
mc.plarmi.Core.java:? (unknown)
Java:
public void onEnable() {
//...
configFile = new File(this.getDataFolder(), "config.yml");
config = new YamlConfiguration();
this.config= this.getConfig();
Разумеется, config вытаскиваем из static. Ибо static тебе вовсе не нужен - у тебя есть и статическая ссылка на самого себя (т.е. Core.java), и местами ты передаёшь ссылку на Core.java в другие классы. Так что config у тебя и так и так доступен.
mc.plarmi.Core.java:? (unknown)
Java:
public void onEnable() {
//...
try {
Commands.load(this);
} catch (Exception var3) {
var3.printStackTrace();
}
Зачем "var3.printStackTrace();" когда можно* "log.log(Level.WARNING, "Ошибка при загрузке команд", var3);" ?
* - а порой и нужно, ибо в стеках из ниоткуда надоело топиться.
Надобность в saveYamls() и loadYamls() отпадёт сама собой.
Про setupEconomy() уже писал - в другом классе.
mc.plarmi.Core.java:? (unknown)
Java:
public boolean onCommand(CommandSender sender, Command command, String commandLabel, String[] args) {
if(sender instanceof Player) {
Player player = (Player)sender;
if(commandLabel.equals("payday")) {
return Commands.onCommand(args, player);
}
}
return false;
}
Ладно, я уже привык видеть onCommand() в основном классе. Хрен с вами. Но чтобы реализовывать это с интерфейсом... Да и ещё в таком маленьком плагине.... Мде.
И только у меня декомпилировалось так?
mc.plarmi.Core.java:? (unknown)
Java:
public void paydays() {
Bukkit.getScheduler().scheduleSyncRepeatingTask(this, new Runnable() {
int count;
{
this.count = Core.config.getInt("Plugin.Delay") * 60;
}
//...
