Statyczne metody w niestatycznym obiekcie

Witam.
Czy taka praktyka jest właściwa?

public class UserManager {
    
    private final HashMap<String, User> users;
    
    public UserManager() {
        this.users = new HashMap<>();
    }
    
    public void init() {
        //fill users hashmap
    }
    
    private User getUser(String user) {
        return users.get(user);
    }
    
    public static User getUser(String user) {
        return Main.getInstance().getUserManager().getUser(user);
    }
}

W klasie "Main" obiekt UserManager jest initializowany po starcie apki

Ops. W metodzie "getUser" powinno być public

Po co ta statyczna metoda zatem?

Patryk27 napisał(a):

Po co ta statyczna metoda zatem?

Właśnie się pytam czy takie coś ma sens.
Pisanie Main.getInstance().getUserManager().getUser("nazwa") z palca ciągle to trochę brzydko. I chciałem sobię "upięknić" kod takim pomysłem.
Jest to błąd czy nie?

Ładowanie statyczne hashmapy wydaje mi się, że nie jest dobrą praktyką samo w sobie.

Poczytaj o stosowaniu dependency injection.

Tak, wiem, znam i stosuję (chociaż nie w tym przypadku bo chciałem akurat trochę poeksperymentować :o)

No, ale dobrze. Dzięki za pomoc ;)

Uczynny Kot napisał(a):

Witam.
Czy taka praktyka jest właściwa?

Mniej więcej tak jak praktyka srania z balkonu, czyli - to zależy, ale moim zdaniem niewłaściwa. Samo stosowanie singletonów (getInstance()) już stanowi pewien tzw. code smell, a ta praktyka jeszcze pogarsza sytuację, "zarażając" singletoństwem Bogu ducha winny model.

Takie rozwiązanie zaszywa w klasie User szczegółową wiedzę na temat implementacji systemu (że jest jakiś Main, że jest jakiś user manager...), co łamie całe mnóstwo dobrych zasad: choćby zasadę pojedynczej odpowiedzialności, prawo Demeter, open/closed principle, separacji zagadnień itd.

Wprowadza przy tym cyrkularną zależność, bo UserManager potrzebuje klasy User, a klasa User potrzebuje (teraz) klasy UserManager.

Tak, wiem, znam i stosuję (chociaż nie w tym przypadku bo chciałem akurat trochę poeksperymentować :o)

W duchu takiej motywacji, patrz punkt pierwszy ;)