Ocena prostej gry osoby początkującej (Java)

Question

Witam. Proszę o ocenę kodu gry. Może być fala krytyki, ale pod warunkiem, że będzie konstruktywna. Jeżeli nie chcecie poświęcać na to większej ilości czasu to zwróćcie uwagę tylko na dwie rzeczy:

- zastosowanie wzorca MVC (czy poprawne)

- podział na metody (czy nie jest zbyt chaotycznie np. w metodzie actionPerformed w Controllerze)

Jestem pewna, że jakieś błędy od razu rzucą wam się w oczy, więc proszę chociaż o parę słów, które bywają na wagę złota ;)

https://github.com/Carmen-Sandiego/MemoryGame

Answer 1

1. Commity, staraj się jakoś w prosty sposób nazywać commity inaczej niż: Update […]

2. Odpal swój projekt w InteliJ (Serio, IDE do javy :> ) i zobacz co Ci powie. Jest tam kilka nieużywanych fragmentów kodu, kilka opcji refactoru → zmienne. Podrzucam zipa z wynikami inspekcji:  link

3. Nie jestem zwolennikiem tworzenia widoku w kodzie. Może jakiś template engine? Albo FXML?

4.  Widok → nie robi czasem aby za dużo (patrząc po tych getterach/setterach ? 

5. Po uruchomieniu nie wiem co robić, ilość klików nie zwiększa się, ciężko stwierdzić czy dobrze klikam :D

 

Comments

2. Oo, dzięki za tę inspekcję :D Tylko czy te wszystkie rzeczy powinno się poprawiać, czy też niektóre zmiany są bardziej opcjonalne (np. zmiana pól na finalne)?

4. Gdzie w takim razie powinny się znaleźć metody zmieniające tekst? Skoro inna klasa miałaby to robić, to potrzebowałaby prywatnego pola widoku - czyli w widoku powinnam stworzyć tylko getter do tego pola?

5. No to nie wiem, u mnie działa ;) Generalnie intuicyjność jest pewnie do poprawy, ale skupiłam się na samym kodzie.

Answer 2

Jak dla mnie (niekoniecznie znawcy Javy) jest całkiem ok, coś co bardzo rzuca się w oczy to highlightButton w:

https://github.com/Carmen-Sandiego/MemoryGame/blob/master/src/MemoryGameView.java

Gdzie aż narzuca się użycie tablicy, która byłaby chociaż wskaźnikiem na red/blue/...button oraz osobnej na lightRed/Blue...

Ogólnie to część kodu można zrobić po prostu "sprytniej", poza tym raczej +

Comments

Dzięki za opinię i wskazówkę :) Tutaj poprawiłam używając tablic:

https://github.com/Carmen-Sandiego/MemoryGame/commit/58c6bf75e2194ad0ee24c3d3c57fd9050c7d8a52