JavaFX - Singletone, Abstract Factory, Dependency Injection

Question

Witam rozpoczynam przygodę z pewnym projektem w Javie. Uznałem że przyszła już pora w mojej przygodzie z programowaniem na zaznajomienie się z wzorcami projektowymi. Na celowniku są te wzorce które podałem w temacie. Po zapoznaniu się z całą wiedzą teoretyczną nie jestem pewny czy dobrze zrozumiałem założenia poszczególnych wzorców oraz sposób ich zastosowania, ale oczywiście zdając sobie sprawę z tego że czytanie teorii za dużo nie daje zwłaszcza w moim przypadku spróbowałem dokonać implementacji. To co się wykluło zamieszczę w kodzie poniżej, a przechodząc do tego o co mi właściwie chodzi.... mianowicie czy ta implementacja ma sens i czy jest zrobiona w miarę prawidłowo i czy to ma jakiś potencjał, a jeśli nie to co jest tutaj nie tak. Czekam na sugestie i konstruktywną krytykę. 

Nie użyłem tutaj jeszcze Dependency Injection ale to co mogę powiedzieć to do tego będę chciał użyć Google Guice żeby wstrzykiwać zależności z widoków do Kontrolerów które są tworzone przez "Fabryki". Na tą chwile nie zagłębiamy się w logikę biznesową no chyba że jest to istotne w tej sytuacji. 

Main.java
 

public class Main extends Application {
	@Override
	public void start(Stage stage){
		GuiFactory factory = GuiFactory.getControllerFactory(ScreenName.Welcome);
		Controller controller = factory.createController();
		controller.show(stage);
	}
	
	public static void main(String[] args) {
		launch(args);
	}
}

ScreenName.java

public enum ScreenName {
	Welcome
}

GuiFactory.java

public abstract class GuiFactory {

	public static GuiFactory getControllerFactory(ScreenName name) {
		switch (name) {
		  case Welcome:
			  return new WelcomeControllerFactory();
		  default:
			  return null;
		}
	}
	public abstract Controller createController();
}

WelcomeControllerFactory.java

public class WelcomeControllerFactory extends GuiFactory {
	Controller controller;
	public Controller createController()
	{
		return controller = WelcomeController.getInstance();
	}
}

Controller.java

public interface Controller {
	public Scene generateScene();
	public void show(Stage stage);
}

WelcomeController.java

public class WelcomeController implements Initializable, Controller {

	public Parent parent;
	public Scene scene;
	public Stage stage;
	
	private final static WelcomeController INSTANCE = new WelcomeController();
	
	public static WelcomeController getInstance(){
		return INSTANCE;
	}
	private WelcomeController(){}
	
	@Override
	public Scene generateScene() {
		FXMLLoader fxmlloader = new FXMLLoader(getClass().getResource("WelcomeScreen.fxml"));
    	fxmlloader.setController(this);
    	try {
			parent = (Parent) fxmlloader.load();
		} catch (IOException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}
		parent.getStylesheets().add("application.css");
		
		scene = new Scene(parent);
		
    	return scene;
	}

	@Override
	public void show(Stage stage) {
		this.stage = stage;
    	stage.setScene(generateScene());
        stage.setTitle("Welcome");
        stage.setResizable(false);
        stage.show();
	}

	@Override
	public void initialize(URL location, ResourceBundle resources) {
		// TODO Auto-generated method stub
	}

}

Pliku fxml nie zamieszczam nie wydaje mi się żeby miał on jakieś znaczenie.
 

Będę wdzięczny za jakiekolwiek mądre wypowiedzi w tym temacie.

Answer 1

Singleton nie jest najlepszą praktyką. Jest trudny w testowaniu, łamie eknapsulację.

http://www.tutorialspoint.com/design_pattern/abstract_factory_pattern.htm

Aczkolwiek można tam wywalić to "static"

Comments

Tzn sprawa wygląda tak że bardzo chciałem wykorzystać tego Singletona ze względu na to że chodzi mi głownie o poznanie wzorców projektowych więc nawet byłbym rad jeżeli z powodu tego były by później problemy, tak najłatwiej się nauczyć i zrozumieć, a co do linku który pan podał, rozumiem że skoro podał pan ten link to są błędy w implementacji wzorca fabryki, wiadomo pierwsza implementacja mogą być ale też mam pytanie czy ogólnie założenie że kontrolery będą tworzone przez fabryki jest dobre i druga sprawa kiedy po wrzuceniu posta dalej siedziałem nad kodem to pojawił się też problem z Dependency Injection, ponieważ z tego co poczytałem trochę w sieci to właśnie powinienem się zdecydować czy użyć fabryk czy dependency injection ponieważ trochę koliduje to ze sobą no chyba że myślałem też o rozwiązaniu takim że wstrzykiwanie zależności jest wtedy "kontrolowane" przez wzorzec fabryk . A wstrzykiwanie byłoby związane z kontrolerami. Niestety mój zamysł początkowy nie wyszedł za bardzo bo chciałem zrobić coś na zasadzie że kontrolery tworzone w fabrykach a do nich wstrzykiwane zależności z widoku tylko że chyba zależności i tak są wstrzykiwane przy adnotacji @FXML