Kokonaisuutena koodiasi on mukavaa lukea. Ne paikat jotka olet jo ehtinyt dokumentoida, olet dokumentoinut selkeästi. Lisäksi logiikkaluokkien metodit ovat lyhyitä, ja helppoja lukea. Attribuutit ja metodit on nimetty hyvin.
Peli on ideana hauska, ja on hyvä että on erilaisia pelimuotoja. Mikäli aikaa riittää voisit lisätä tulostaulut esimerkiksi alkuvalikkoon?
Pakkaukset ovat hieman epäselvästi nimetty, miten olisi kayttoliittyma ja sovelluslogiikka pakkauksien nimeksi?
Luokan TiedostonLukija.java voisi sijoittaa sovelluslogiikka pakkaukseen
Yleisesti luokat nimetään CamelCasella, Laivanupotus ja Piilosana voisi refactoroida. Ei iso vaiva netbeansilla.
Käyttöliittymäluokassa on lisäksi aika paljon rivejä. Pitäisikö eri peleille tehdä ehkä omat luokat?
Käyttöliittymäluokassa lisätään myös todella suuri määrä nappeja ja niiden kuuntelijoita. Kyseiset operaatiot voisi mahdollisesti suorittaa listoilla ja loopeilla?
Testit on ilmeisesti vielä keskeneräisiä. Olet käyttänyt melkein jokaisessa testiluokassa hiukan eri tyyliä setUpin tekemiseksi. Suosisin kuitenkin itse PelaajaTest luokan tapaa. Vähiten rivejä ja helpoin ymmärtää. :)
Hei!
Latasin projektisi 11.6 kello 1205.
-mknn