Gra w Kółko i Krzyżyk napisana w Javie

Question

Witam, jestem początkującym programistą w języku Java i właśnie stworzyłem swoją pierwszą aplikacje okienkową przy pomocy pakietu Swing - grę w Kółko i Krzyżyk. Chciałbym poprosić bardziej doświadczonych programistów o krytyczny komentarz dotyczący wszystkich aspektów tego programu - co zostało zrobione źle, co mógłbym zrobić lepiej / poprawić, czy kod jest czytelny i opisany właściwymi komentarzami, ew. co zrobiłem dobrze. Załączam link do kodu zamieszczonego na GitHubie. Z góry dziękuję za pomoc.

https://github.com/baqazan/TicTacToe

Answer 1

1) Masz trzy: if, else if, else if, a w nich dokładnie to samo: 

String winner = "Winner:   " + checkWinner();
JOptionPane.showMessageDialog(null, winner);
System.exit(0);

2) 

Zamiast:

checkWinner()=="X"

Powinieneś albo przekształcić swój kod żeby działał na char'ach albo sprawdzać w ten sposób:

checkWinner().equals("X")

3) 
Niepotrzebnie wywołujesz metodę checkWinner() najpierw w warunku if, a później wewnątrz if'a drugi raz. Zamiast tego przed if'em powinieneś przypisać to co checkWinner() zwróciło do jakiejś zmiennej

4) 

if(fields[0].getValue()!=null && fields[0].getValue()==fields[1].getValue() && fields[1].getValue()==fields[2].getValue())

Brzydkie to. Zauważ, że robisz to samo sprawdzenie 9 razy tylko dla różnych indeksów tablicy, zamiast tego mógłbyś stworzyć metodę, która zwarca boolean, a jako argumenty przyjmuje te indeksy tablicy - poprawia czytelność i kod jest krótszy.

5)
Zamiast oznaczać pola, w których żaden z graczy nie udzielił jeszcze odpowiedzi jako null mógłbyś użyć Optional<String> z Javy 8. Ewentualnie pójść o krok dalej i zamiast Stringa stworzyć jakiegość enum'a, który ma w sobie "X" i "O".

6) 
Wiem, że to głównie kwestia gustu i preferencji, ale w Javie przyjęło się, że klamry piszemy tak:

public void method() { 
    // whatever
}

Pewnie bardziej podoba Ci się ten styl, którego używasz, ale jeśli kiedyś będziesz wspólnie pracować nad jakimś projektem z innymi programistami, lub po prostu pracować w firmie jako Java Dev to Cię zjadą za te klamry :) 
 

Comments

Dzięki za udzielone rady, przerobiłem według nich kod i faktycznie wygląda dużo czytelniej :)

Answer 2

• Klasy umieszczaj w pakietach o pewnych nazwach, a nie w pakiecie domyślnym. To najważniejsza rzecz do zapamiętania! W żadnym profesjonalnie napisanym programie w tym języku nie znajdziesz czegoś takiego.
• Oddzielaj warstwę prezentacji od logiki, co pomoże pisać tobie większe programy.

Answer 3

Fajnie, że pracujesz i się rozwijasz. Na pewno głębiej wypowie się ktoś mądrzejszy ode mnie, ale ja też mam kilka uwag. Po pierwsze, dlaczego nie wystawiłeś całego projektu na github. Poczytaj sobie o tym, bo każde IDE pozwala na automatyczne dodanie całego projektu do githuba.

Druga sprawa, to komentarze. Fajnie, że komentujesz swój kod, ale jeszcze lepiej by było, gdybyś nauczył się dokumentować go narzędziem JavaDoc.

Comments

Druga sprawa, to komentarze. Fajnie, że komentujesz swój kod, ale jeszcze lepiej by było, gdybyś nauczył się dokumentować go narzędziem JavaDoc.

Spoko, ale on nie tworzy publicznego API z którego każdy może korzystać.  Jak tworzy własny projekt to wystarczy mu planowanie i dostarczanie commitów o konkretnej treści. Raczej w takim projekcie by się nie zgubił.

Podsumowując, komentarze JavaDoc są zbędne w małych projektach do nauki. Wyjątkiem jest tu nauka JavaDoc.