zirinna / HimmunHedelmaPeli

ohjelmoinnin harkkatyö

0 stars 0 forks source link

Koodikatselmointi 1 #1

Closed Walther closed 8 years ago

Walther commented 8 years ago

Code Review: Himmun Hedelmäpeli

Plussat

Pirteät piirrokset! :D
Hyvä jaottelu paketteihin ja luokkiin
Hyvät kommentit!
Testiluokat luotu jokaiselle luokalle
Varsin hyvä testikattavuus
Selkeät nimet luokilla, metodeilla, testeillä
Kehitysehdotukset
Dependenssiongelma: package javafx does not exist. Kerro lisäriippuvuudesta dokumentaatiossa, ja ohjeista asentamaan.
Main-luokka saattaisi hyötyä refaktoroinnista: ehkä GUI toiseen luokkaan mahdollistaisi "uusi peli?" -promptin?

zirinna commented 8 years ago

Tosiaan joo JavaFX:n takia pelini vaatii Java8:n (jossa JavaFX pitäisi olla vakiona mukana), lisäsin tiedon tästä README.hen :)