Snake w swing'u, prośba o ocenę kodu

Question

Cześć,

Chciałbym podzielić się swoim projektem, którego po trochu robiłem przez ostatni czas w wolnych chwilach. Linki do kodu oraz samej gry:

https://megawrzuta.pl/download/6361a1518ed3aa40d29819de34daa353.html

Sterowanie tylko przez klawiaturę, także w menu oraz możliwość zapisania uzyskanego wyniku. Wszelka krytyka mile widziana

Answer 1

Już dawno nie robiłem komuś code review, ale podrzucę ci co wypadało by zmienić.

1. nazwy katalogów zaczynaj z małej litery. przykład z twojego kodu (View i Menus) - "package View.Menus;"

2. "public abstract class AMenu" - Tworzenie prefiksu A przed klasą abstrakcyjną jest już przestarzałe i nie powinno się tego już używać. To samo z I dla Interfejsów. Czasami jeszcze dodaje się sufix impl by implementacja z interfejsem się tak samo nie nazywały ale to wyjątek

3."int selectedMenuItemIndex;" - Używaj modyfikatorów dostępu, chyba że naprawdę potrzebujesz by zmienna była widziana per pakiet.

4. public static String[][] readPlayerScoresFromFile(String fileName) - Masz język obiektowy więc rozdzielaj każdą operacje do innej metody. W tej metodzie nie odczytujesz z pliku tylko otwierasz go, odczytujesz, dzielisz na playerName i score, i jeszcze coś dodatkowo w pętli robisz, postaraj się wyrzucić to do metod prywatnych. Może i tu się trochę za bardzo czepiam ale przeczytaj sobię książkę clean code. Dobra jest.

5. public class ArrowAction extends AbstractAction - Nie używaj klas wewnętrznych wydaje mi się że lepiej to zrobić w odddzielnym pliku. Wyjątkiem mogą być jakieś patterny jak naprzykład builder.

 

Droga przez code review jest dobrą drogą by stać się dobrym programistą, szacunek za to bo nikt z nas nie lubi być krytykowanym za swój kod. Wypisałem ci co mi się na początku rzuciło w oczy, kodu nie uruchamiałem i tylko wzrokiem przeleciałem co jest nie tak. Sam mam dopiero 1,5 roku doświadczenia w tym i nie jestem jakimś specem. Pozdro