Помогите Java, как лучше

[baksovic111]

Ребята, без лишних шуток пожалуйста!
Подскажите как лучше.
Можете пояснить в кратце.




Второй вариант

 

[hyndorik]

P.s это основа основ. Изучается на первой страничке, "конструкторы".

Так дело не в конструкторе, а в том чтобы Main получать не через статик метод, а инжектить его через конструктор

 

[Exception_Prototype]

Так дело не в конструкторе, а в том чтобы Main получать не через статик метод, а инжектить его через конструктор

а, так это оно и есть? Я думал априори так положено.

 

[hyndorik]

а, так это оно и есть? Я думал априори так положено.

Как положено?

 

[Exception_Prototype]

Как положено?

как показал @alexandrage http://rubukkit.org/threads/java-kak-luchshe.155529/#post-1521608

 

[alexandrage]

Так дело не в конструкторе, а в том чтобы Main получать не через статик метод, а инжектить его через конструктор

Ну так, передача переменных конструктору.

 

[Reality_SC]

DI всё-таки больше в том, что есть окружение, которое само разрешит твои зависимости и передаст его тебе, через конструктор или любым другим способом.
DI, конечно, хорошая парадигма, во многих местах без неё никуда, но здесь ею только детей пугать.

P.S. Я за передачу через конструкторы. Если хочется именно синглтон, есть много способов способов сделать это.

 

[hyndorik]

DI всё-таки больше в том, что есть окружение, которое само разрешит твои зависимости и передаст его тебе, через конструктор или любым другим способом.
DI, конечно, хорошая парадигма, во многих местах без неё никуда, но здесь ею только детей пугать.

P.S. Я за передачу через конструкторы. Если хочется именно синглтон, есть много способов способов сделать это.

Ну так джуйс для ленивых есть

 

[baksovic111]

Очень хорошая, информативная информация.
Всем огромное спасибо. Если у кого есть еще что сказать, буду рад прочитать 🤓

 

[baksovic111]

Это можно назвать более грамотным кодом?

 

[AtomicScience]

Это можно назвать более грамотным кодом?

Да, но все-таки лучше не передавать конфиг в конструкторе, а сделать паблик метод getConfig() в Main

 

[aceJKE]

1. Зачем тебе переменная plugin в Main?
2. Переменные в config выбиваются из конвенции по наименованию в Java.
3. Название Events - ужасно. Из названия не понятно что это за Events и почему Events, если это Listener.
4. Название Utils - ужасно. Из названия не понятно что это за Utils.

Прочитай следующие книги и не будешь задавать подобных вопросов.
http://www.ozon.ru/context/detail/id/2457392/
http://www.ozon.ru/context/detail/id/5011068/
http://www.ozon.ru/context/detail/id/29345868/

 

[Dereku]

Это можно назвать более грамотным кодом?

Посмотреть вложение 49755
Посмотреть вложение 49756

import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.PlayerDeathEvent;
import org.bukkit.plugin.java.JavaPlugin;

public class PluginName extends JavaPlugin implements Listener {
    private Config config;

    @Override
    public void onEnable() {
        this.config = new Config();
        this.getServer()/*register events*/;
    }

    @EventHandler
    public void onPlayerDeath(PlayerDeathEvent event) {
        Player player = event.getEntity();
        if (ScrollUtils.checkScroll(player)) {
            event.setKeepInventory(true);
            event.setKeepLevel(true);
            if (!config.scrollUsed.equalsIgnoreCase("none")) {
                player.sendMessage(config.scrollUsed);
            }
        } else {
            if (!config.scrollNotUsed.equalsIgnoreCase("none")) {
                player.sendMessage(config.scrollNotUsed);
            }
        }
    }
}

 

[alexandrage]

import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.PlayerDeathEvent;
import org.bukkit.plugin.java.JavaPlugin;

public class PluginName extends JavaPlugin implements Listener {
    private Config config;

    @Override
    public void onEnable() {
        this.config = new Config();
        this.getServer()/*register events*/;
    }

    @EventHandler
    public void onPlayerDeath(PlayerDeathEvent event) {
        Player player = event.getEntity();
        if (ScrollUtils.checkScroll(player)) {
            event.setKeepInventory(true);
            event.setKeepLevel(true);
            if (!config.scrollUsed.equalsIgnoreCase("none")) {
                player.sendMessage(config.scrollUsed);
            }
        } else {
            if (!config.scrollNotUsed.equalsIgnoreCase("none")) {
                player.sendMessage(config.scrollNotUsed);
            }
        }
    }
}

Топчик. Каша все в одном называется.

 

[lyooxa]

private static final Utils util = new Utils();
public static Utils getUtils() {
return util; }