Стартап Как обратиться к главному классу через другой класс

[Zlodey_duo]

Очень часто вижу сообщения в просьбе о помощи, как обратиться к главному классу дабы получить конфиг? (Обратиться к конфигу).

Сделать это очень просто

Вам нужно написать следующий код в главном классе:

    private static Main plugin;
    public static Main getInstance() { return plugin; }

    public void onEnable() {
            plugin = this;
}

Готово)

Теперь как обратиться к главному классу, в моём примере это Main.

Main.getInstance()

после instance() через точку мы можем к примеру получить конфиг

Main.getInstance().getConfig();

(!!!) Знающие люди, если как-то можно упростить код и т.п. Напишите, пусть начинающие учатся.
(!!!) Не судите строго

 

[SashaBrine]

public static Main instance() { return plugin; }

Чему ты учишь людей надо getInstance

 

[Zlodey_duo]

Чему ты учишь людей надо getInstance

А разница?

 

[SashaBrine]

А разница?

Загугли "Java конвенция"
И ищи там методы

 

[Zlodey_duo]

Загугли "Java конвенция"
И ищи там методы
Посмотреть вложение 62349

Спасибо за критику, исправил.

 

[Dymeth]

Это не лучший способ обращаться к объекту плагина. Использовать такой способ стоит только в крайнем случае.
В большинстве ситуаций достаточно передавать главный класс плагина в конструкторы менеджеров

 

[SashaBrine]

Это не лучший способ обращаться к объекту плагина. Использовать такой способ стоит только в крайнем случае.
В большинстве ситуаций достаточно передавать главный класс плагина в конструкторы менеджеров

Это второй способ но например к static методам ты что будешь передавать объект плагина?

 

[Dymeth]

Это второй способ но например к static методам ты что будешь передавать объект плагина?

Static методы тоже должны использоваться в исключительных случаях - фактически только для утилит. А инстанс плагина там, как правило, и не нужен. Случаи получения плагина статикой есть, но они довольно редки. И в целом нет проблемы в утилитарные методы передавать инстанс чего-либо.
Но вопрос - что мешает сделать такой метод нестатическим методом?

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

 

[SashaBrine]

Static методы тоже должны использоваться в исключительных случаях - фактически только для утилит. А инстанс плагина там, как правило, и не нужен. Случаи получения плагина статикой есть, но они довольно редки. И в целом нет проблемы в утилитарные методы передавать инстанс чего-либо.
Но вопрос - что мешает сделать такой метод нестатическим методом?

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

package com.company.mytopplugin;

public class LoadUtil {
    public static void loadCommands() {
       
    }
}

 

[Dymeth]

package com.company.mytopplugin;

public class LoadUtil {
    public static void loadCommands() {
     
    }
}

Для чего этот метод статический и вынесен в утилиты?

 

[SashaBrine]

Для чего этот метод статический и вынесен в утилиты?

Грузит команды и устанавливает их экзекьюторы

Вот пример:

public class LoadUtil {
    private static Main plugin;
    static {
        this.plugin = Main.getInstance();
    }
 
    public static void loadCommands {
        plugin.getCommand("maincommand").setExecutor(new MainCommand());
    }
}

 

[alexandrage]

Грузит команды и устанавливает их экзекьюторы

Вот пример:

public class LoadUtil {
    private static Main plugin;
    static {
        this.plugin = Main.getInstance();
    }
 
    public static void loadCommands {
        plugin.getCommand("maincommand").setExecutor(new MainCommand());
    }
}

Дичайший говногодинг.

 

[SashaBrine]

Дичайший говногодинг.

Что тебе не понравилось?)))

 

[MurlikMurlik]

Что тебе не понравилось?)))

Раз на то пошло, блок инициализации static можно вообще не использовать для инициализации статичной переменной, просто используй

private static Main plugin = Main.getInstance();

Это тоже самое и будет короче.
Кроме этого, это все не имеет смысл, так как проще получить инстанс сразу у Main класса и не городить лишние ненужные переменные.

Main.getInstance().getCommand("maincommand").setExecutor(new MainCommand());

 

[alexandrage]

Кроме этого, это все не имеет смысл, так как проще получить инстанс сразу у Main класса и не городить лишние ненужные переменные.

Или писать по человечески передав переменную в конструктор нужному классу.

 

[Reality_SC]

Раз на то пошло, блок инициализации static можно вообще не использовать для инициализации статичной переменной, просто используй

private static Main plugin = Main.getInstance();

Это тоже самое и будет короче.

Плюс скорее всего ещё и работать точно также не будет, если случайно загрузка класса дёрнется до onEnable плагина.

Самый корректный способ не умереть в лапше в будущем, как уже сказали выше, SPR и передача инстанса плагина в твой класс через конструктор.

 

[SashaBrine]

Плюс скорее всего ещё и работать точно также не будет, если случайно загрузка класса дёрнется до onEnable плагина.

Вообще то нет я собираюсь писать этот метод как раз в onEnable

public void onEnable() {
    plugin = this;
    LoadUtil.loadCommands();
}

 

[Sergeo]

Грузит команды и устанавливает их экзекьюторы

Вот пример:

public class LoadUtil {
    private static Main plugin;
    static {
        this.plugin = Main.getInstance();
    }
 
    public static void loadCommands {
        plugin.getCommand("maincommand").setExecutor(new MainCommand());
    }
}

Не лучший код, но все же

    public void registrationCmd(CommandExecutor[] executors, String... commands) {
        if(executors.length == 1) {
            for(String command : commands) {
                plugin.getCommand(command).setExecutor(executors[0]);
            }
        } else {
            for (int i = 0; i < executors.length; i++) {
                plugin.getCommand(commands[i]).setExecutor(executors[i]);
            }
        }
        Messages.logger("(COMMANDS) >> Registered commands: " + commands.length);
    }

 

[SashaBrine]

Не лучший код, но все же

    public void registrationCmd(CommandExecutor[] executors, String... commands) {
        if(executors.length == 1) {
            for(String command : commands) {
                plugin.getCommand(command).setExecutor(executors[0]);
            }
        } else {
            for (int i = 0; i < executors.length; i++) {
                plugin.getCommand(commands[i]).setExecutor(executors[i]);
            }
        }
        Messages.logger("(COMMANDS) >> Registered commands: " + commands.length);
    }

Так тоже можно

 

[Dymeth]

Так тоже можно

Не только можно, но и нужно. О проблемах статики сказано выше