Ocena kodu - Notepad w JavaFX.

Question

Witam!

Postanowiłem wrzucić swój kod na forum. Jeszcze tego nie robiłem, ale pomyślałem, że fala konstruktywnej krytyki lub porad będzie przydatna. Co więcej chciałbym się dowiedzieć czy mój kod jest czytelny, "czysty" przez innych programistów, oraz czy moje commity są poprawnie opisywane. 

Notepad'a napisałem wykorzystując JaveFX, oraz korzystając z wzorców projektowych takich jak:

• MVC
• Facade
• Strategy
• Singleton

MVC - typowo jeśli chodzi o aplikacje z GUI, czyli po prostu oddzieliłem sobie kod, wszystko jest w odpowiednich paczkach. Co więcej Controller pełni rolę fasady (delegując zadania do wykonania), dodałem też serwisy odciążając logikę biznesową z modelu.

Strategy - ostatnio natrafiłem na ten wzorzec projektowy i chciałem się go nauczyć. Dlatego postanowiłem praktycznie go zastosować w jakimś projekcie. Zaimplementowałem wzorzec do funkcjonalności operacji na plikach(otwieranie, tworzenie nowego). 
Wygląda to tak:
Po otwarciu jakiegoś pliku, gdy coś do niego dopiszemy i w momencie gdy chcemy otworzyć nowy, pojawia nam się nowe okno (message.fxml). Decydujemy czy zapisać lub porzucić zmiany. I w tym momencie miałem problem z funkcjonalnością kodu. Musiałem decydować na ifach co ma dalej się dziać. Po zastosowaniu wzorca kod jest bardziej elastyczny i nie wymaga warunków.

Singleton - jakże kontrowersyjny wzorzec lub antywzorzec. Postanowiłem się go nauczyć i wykorzystać w projekcie, tylko nie jestem pewien kiedy do końca się powinno go stosować i tutaj byłbym wdzięczny za wyjaśnienie. Implementacja dotyczy klas: NotepadModel i FileModel. 

Co do funkcjonalności samego programu, zaimplementowałem tylko najważniejsze operacje (tworzenie, otwieranie, zapisywanie, zapisywanie jako..)

Celem tego projektu było przetestowanie wzorców projektowych.

Kod:
https://github.com/K0XM4N/NotepadJavaFX

Pozdro ;)