Pomoc przy refaktoryzacji kodu aplikacji desktopowej

Question

Aktualnie piszę aplikację okienkową z użyciem AWT/Swing pod środowiskiem Eclipse. Ma to być prosty symulator piłkarski (mecz w roli trenera, seria rzutów karnych, zarządzanie drużynami, edycja zawodników, odczyt i zapis do pliku). Podkreślone funkcje zostały praktycznie ukończone.

Chciałbym poprosić o rady dotyczące tego co w kodzie mogę poprawić, co należało by zmienić, o jakich rzeczach typowych dla Javy powinienem się dowiedzieć, aby kod był bardziej solidny. Nie mam tak naprawdę kogo zapytać, stąd kieruję się na forum. Dotychczas programowałem tylko w C, C++ i Matlabie, co wynika z kierunku studiów i zdaję sobie sprawę, że kod/repozytorium może wyglądać bardzo nieprofesjonalnie, jednak proszę o spojrzenie życzliwym okiem i pomoc w wypunktowaniu co powinienem zmodyfikować, aby nie popełniać tych samych błędów przy rozwoju aplikacji ; )

Link do repozytorium z projektem: Github - Java - Beniaminek

Answer 1

Patrząc na jedną funkcję:

https://github.com/PiotrAGH/java.beniaminek/blob/master/Menu.java#L246

Oh wow.

for(...) {
    Club club = league.get(i);
    for(...) {
        Player player = club.getTeam().get(j);
    }
}

Lepiej, nie?

https://github.com/PiotrAGH/java.beniaminek/blob/master/Menu.java#L260

Co Ci szkodzi nowa linia na końcu pliku? Pliki powinny kończyć się nową linią: http://stackoverflow.com/questions/729692/why-should-text-files-end-with-a-newline

https://github.com/PiotrAGH/java.beniaminek/blob/master/Menu.java#L264

https://docs.oracle.com/javase/tutorial/essential/exceptions/tryResourceClose.html

Comments

Dziękuję : )
Te długie nazwy rzeczywiście bez potrzeby wydłużały kod... Chyba niepotrzebnie broniłem się przed tworzeniem nowych obiektów.
Z nową linią na końcu pliku miałem problem, był konflikt pomiędzy odczytem (bez nowej linii - co wynika z pętli), a zapisem (chodziło o niezgodność formatowania lub właśnie niepotrzebnie dodawana linijka). Wniosek: Do poprawy ; )