MVC w javafx oraz prośba ogólne porady co do pisania aplikacji

Witam!
Napisałem prosty kalkulator w javafx ponieważ chciałem się zapoznać z podstawowym wzorcem MVC. Początek kodu wziąłem z tutoriala lecz ponad połowę robiłem samodzielnie i chciałbym się was zapytać czy dobrze zaimplementowałem logikę MVC. Chciałbym jeszcze oprócz tego poznać opinię na temat formatowania mojego kodu oraz komentarzy. Jestem dość początkujący i liczę na rady odnośnie tej kwestii. Czy bardzo widać po kodzie nieprofesjonalizm? Tutaj link do repozytorium na github: https://github.com/damianwnukowski/Kalkulator-MVC . Co do mvc to nie jestem do końca pewien czy tworzenie obiektu model w klasie controller to jest dobry pomysł. Może powinienem korzystać w tym przypadku z metod statycznych? Jeśli ktoś by chciał zerknąć na ten mały projekcik i coś dodać od siebie byłbym wdzięczny :).

Ogólnie formatowanie wygląda ok oprócz paru niedociągnięć

// CalculatorModel.java 

private float calculationValue = 0; // nie żałuj spacji

case '+':{ // niepotrzebna klamra
	addTwoNumbers(firstNumber, secondNumber);
	break;
}

// CalculatorController.java

//Creating model object to make calculations	// niepotrzebny komentarz - komentuj tylko niejasny kod, tutaj widać co robisz
private CalculatorModel model = new CalculatorModel(); 

if (operation == '=') { //spacje!

Okej dzięki! Muszę sobie jakiś nawyk wyrobić co do formatowania żeby być bardziej konsekwentny czasem dam spacje czasem nie itp. W switch też myślałem że klamer nie trzeba ale nie byłem pewien a jak działało to nie ruszałem :D.

Nie widzę w tym przykładzie nic złego w tworzeniu modelu w kontrolerze, najlepiej gdybyś go wstrzykiwał, ale póki co jest ok. Co do modelu - nie potrzebujesz tam pola calculationValue, twoje metody mogły być funkcjami - czyli zwracasz z nich wynik niezależnie od stanu obiektu. Drugą rzeczą, którą bym poprawił to twoje metody w kontrolerze, np processOperator ma 50 linii. Mógłbyś ja rozbić na mniejsze metody co by z pewnością ułatwiło czytanie, plus usunięcie pustych linii, które nic nie wnoszą. Często tak jest, że jak masz bloki kodu oddzielone pustymi liniami w metodzie to podpowiedź, że to prawdopodobnie mogłaby być osobna metoda.