Witajcie, niedawno skończyłem swój pierwszy projekt w Javie. Chciałbym się możliwie jak najwięcej z niego nauczyć, dlatego proszę Was o ocenę mojego kodu. Napiszcie mi jakie błędy popełniłem i jakie usprawnienia mógłbym wprowadzić. Zastanawiam się też nad rozbiciem kilku klas na osobne paczki, żeby każda klasa zawierała jeszcze mniej metod. Czy uważacie, że to dobry pomysł? Mam na myśli głównie klasy opisujące poszczególne sceny GUI.
Repozytorium na GitHubie//goo.gl/T6wRXn
0
1
Czemu z 90% metod jest statyczna? Bardzo zły pomysł
1
Odnośnie komentarza
to mój pierwszy projekt napisany obiektowo, nie wiedziałem, że jest to błąd - super, uczysz się po to właśnie żeby to wiedzieć :)
jak mogę go wyeliminować ? Muszę zmieniać cały sposób pisania poszczególnych klas, Myślałem, że utworzenie kilku zmiennych statycznych i definiowanie ich następnie przez różne metody będzie najbardziej przejrzystym rozwiązaniem? - tak, powiedziałbym, że powinieneś (to nie jest tak, że zrobiłeś coś bardzo źle - bardziej chodzi o to, że mimo swej przejrzystości ten kod będzie ciężko zmienić a w realnej pracy to jest niemal pewne, że będzie zmienione).
Statyczne metody przydają się do jakichś metod utilsowych, nietrzymających stanów (a i tu niekoniecznie zawsze są dobrym wyborem)
Generalnie rozchodzi się o to, że statyczne metody oznaczają kod mocno powiązany (strongly coupled), lepiej jest implementować jakieś funkcjolaności via interfejsy, abstrakcje. Poczytaj sobie o interfejsach, dziedziczeniu i polimorfizmie. Generalna zasada jest taka, że im mniej obiekty o sobie wiedzą tym lepiej. Co więcej to sprawi, że kod będzie być może mniej przejrzysty ale łatwo zmienialny, testowalny - w skrócie, prędzej czy później Ci się zwróci.