В разработке оптимизация кода

[Аминчик]

 

[MrMagaChannel]

Зачем использовать cmd.getName().equalsIgnoreCase("don")?
Если можно сразу через label.equalsIgnoreCase("don")
Все это лучше пихай в конфиг, как List, и через for выводи.

p.sendMessage("§e=====Помощь=====");
           p.sendMessage("§e/don check [ник] - §cУзнать привилегию игрока");
           p.sendMessage("§e/don set [ник] [привилегия] - §cВыдать привилегию игроку");
           p.sendMessage("§e/don remove [ник] [причина] - §cЗабрать привилегию у игрока");
           p.sendMessage("§e/don prefix [ник] [префикс] - §cИзменить префикс игроку");
           return true;

И еще, зачем привязывать плагин к PEX, когда он давно сдох? Динозавров любите?

 

[_MasterCapeXD_]

File cfg = new File(getDataFolder() + File.separator + "config.yml"); if (!cfg.exists()); { getConfig().options().copyDefaults(true); saveDefaultConfig(); }

this.saveDefaultConfig(); все это сделает.

Можешь переменные в поле вывести, текстовые например и сделать их константами с final (особенно те, что повторяются)

 

[Типа админ:D]

Зачем использовать cmd.getName().equalsIgnoreCase("don")?
Если можно сразу через label.equalsIgnoreCase("don")
Все это лучше пихай в конфиг, как List, и через for выводи.

p.sendMessage("§e=====Помощь=====");
           p.sendMessage("§e/don check [ник] - §cУзнать привилегию игрока");
           p.sendMessage("§e/don set [ник] [привилегия] - §cВыдать привилегию игроку");
           p.sendMessage("§e/don remove [ник] [причина] - §cЗабрать привилегию у игрока");
           p.sendMessage("§e/don prefix [ник] [префикс] - §cИзменить префикс игроку");
           return true;

И еще, зачем привязывать плагин к PEX, когда он давно сдох? Динозавров любите?

В туториалах я всегда видел cmd.getName(), соответственно, он либо по туториалу делал, либо запомнил это из туториала, который смотрел давно.
С каких пор PEX входит в список динозавров?

 

[TheZefirrkka]

С каких пор PEX входит в список динозавров?

С тех, как его перестали обновлять.

 

[Типа админ:D]

С тех, как его перестали обновлять.

Оу... Кажется, я многое упустил...

 

[InterWall]

Оу... Кажется, я многое упустил...

Ага. Пекс давно подыхает, сейчас luckperms в моде.

 

[Nikolai_Faint]

String remove_player = this.getConfig().getString("remove_player").replace("&", "§").replace("{name}", name);

Ту маин
    public static void message(Player p, String msg){ 
        p.sendMessage(ChatColor.translateAlternateColorCodes('&', msg)); 
   } 
Ту семеде
public static Main m = Main.m;
    String s = m.getConfig().getString("lol");
    Main.message(p, s);

 

[_MasterCapeXD_]

public static Main m = Main.m;

getInstance() уже не в моде?

 

[_MasterCapeXD_]

import ru.tehkode.permissions.PermissionManager; import ru.tehkode.permissions.PermissionUser; import ru.tehkode.permissions.bukkit.PermissionsEx;

Юзай vault, тогда, при смене менеджера прав переписывать код не надо будет.

 

[Nikolai_Faint]

getInstance() уже не в моде?

Неа.

 

[TheZefirrkka]

getInstance() уже не в моде?

-

 

[hyndorik]

Зачем использовать cmd.getName().equalsIgnoreCase("don")?
Если можно сразу через label.equalsIgnoreCase("don")
Все это лучше пихай в конфиг, как List, и через for выводи.

p.sendMessage("§e=====Помощь=====");
           p.sendMessage("§e/don check [ник] - §cУзнать привилегию игрока");
           p.sendMessage("§e/don set [ник] [привилегия] - §cВыдать привилегию игроку");
           p.sendMessage("§e/don remove [ник] [причина] - §cЗабрать привилегию у игрока");
           p.sendMessage("§e/don prefix [ник] [префикс] - §cИзменить префикс игроку");
           return true;

И еще, зачем привязывать плагин к PEX, когда он давно сдох? Динозавров любите?

Ну так label возвращает использованный альяс, ты опять хрень советуешь, при использовании label альянсы у команд будут недоступны, круто саму себе в ногу стрелять.

Ну а на счет самого кода автора, то тут как бы оптимизировать нечего с таким примитивным кодом. Код если рассматривать с архитектурной точки зрения конечно говно полное, если автор хочет улучшить его в этом направлении, то надо бы получше разобраться в ооп для начала, потом можно изучить solid принципы

 

[Аминчик]

Всем спасибо!