miiaramo / yatsy Goto Github PK

projekti ladattu githubista 20.9. klo 15.00

Koodissa pistää silmään ensiksi kommenttien vähäisyys, mikä vaikeuttaa ulkopuolisen koodin lukua. (Sama ongelma omassakin projektissa) Tosin hetken aikaa projektiin tutustuttuaan ei niitä enää kaipaakaan. Luokat ja metodit (sekä muuttujat) on kuitenkin nimetty selkeästi ja intuitiivisesti, joten on helppo ymmärtää luokan tai metodin tehtävä. Luokat oli myös jaettu pakkauksiin järkevästi. Ainoaksi kysymysmerkiksi jäi Kasi-luokka, joka lienee joko jäänne aikaisemmasta rakenteesta tai uusi muutos, jonka toteutus on vielä kesken.

Metodit ovat myös useimmiten niin ytimekkäitä ettei niiden toimintaa tarvitse koodia lukiessa jäädä ihmettelemään. Poikkeuksena vosi mainita Pelaaja-luokan asetaPisteet-metodin, joka nykymuodossaan sekä asettaa päivitettävään ruutuun uuden pistemäärän että päivittää väli- ja loppusummat. Single responsibility -hengessä summien päivittämiset voisi erottaa omiksi metodeikseen.

Käyttöliittymän alku näytti hyvältä lähtökohdalta. En tiedä, onko ongelma tiedossa, mutta ainakaan omalla koneellani ääkköset eivät näkyneet oikein.

Testejä ei vielä hirveästi ollut, mutta se ainoan testatun luokan testit oli toteutettu jokseenkin samaan tapaan kuin olisin itse tehnyt, ei siis huomaustettavaa testien laadusta.

Ei siis mitään suurempia huomauteltavia, aika helppolukuista koodia. Harmi ettei vielä päässyt pelailemaan.