Gra kółko i krzyżyk. Java Swing. Liczę na słowa krytyki.

pytanie zadane 15 marca 2017 w Nasze projekty przez kojonek2 Nowicjusz (120 p.)

Witam. Piszę ten post aby zaczerpnąć waszej opinii co mogłem zrobić lepiej lub inaczej. Zaznaczam, że projekt jest robiony wyłącznie w celach edukacyjnych. Stworzyłem grę kółko i krzyżyk w której można grać na planszy dowolnej wielkości. Gra stworzona w java swing przy pomocy window buildera. Jest to mój pierwszy projekt swing. Interfejs nie jest zbyt ładny, ale planuję dodać animacje do gry. Z góry dziękuje za odpowiedzi.

Kod źródłowy

2 odpowiedzi

odpowiedź 15 marca 2017 przez Raymond.Z Obywatel (1,800 p.)

Tak tylko na dzień dobry, a właściwie już wieczór :-)

1. Nie używaj w nazewnictwie klas, metod czy zmiennych "my"

2. Nie używaj wykrzyknika do negowania wartości logicznej, jest strasznie nie zgrabny, polecam porównanie do fałszu.

3. Staraj się nazywać trochę bardziej konkretnie funkcje, a nie tak ogólnie np "checkExtension", lepiej byłoby dać jakąś nazwę, która by wskazywała na to, że ta funkcja sprawdza czy plik jest z rozszerzeniem jsona.

4. Dlaczego widoki mają w sobie praktycznie całą funkcjonalność tej gry? Rozdziel to, klasy odpowiedzialne za widok niech będą odpowiedzialne tylko za ten widok.

komentarz 16 marca 2017 przez kojonek2 Nowicjusz (120 p.)

Bardzo dziękuję za odpowiedź. W weekend postaram się po poprawiać. Co do wykrzyknika to sam się łapałem czasem, że go nie zauważałem, ale nie wiedziałem czy przyrównywanie do fałszu jest akceptowalne.