1. Koodikatselmointi

[Kiillottaja]

Projektin ZIP-tiedosto ladattu 3.4.2015 klo 9:30

Moikka, tässä koodikatsemus projektistasi.

Koodisi oli kokonaisuudessaan selkeää luettavaa. Aihemäärittelystä sai hyvin kuvan minkälainen ohjelmastasi on tulossa. En ollut ennen tutustunut ko. pasianssin muotoon, mutta nyt sekin on tuttu. Kävin koodiasi läpi Aihemäärittely mielessä ja olit hyvin onnistunut pelin logiikan toteutuksessa.

Luokat olivat selkeissä paketeissa ja sopivan kokoisia. Jokaisella metodilla oli selkeästi omat tehtvänsä, ihan niin kuin pitääkin. Mielestäni oli hyvä että Pelisaannot oli omassa luokassaa, eikä niitä oltu sotkettu Pelilogiikkaan.

Testejä olit kirjoitellut kattavasti projektin edetessä ja kaiken kaikkiaan ne testaavat hyvin luokkien metodeja, ja lisäksi lausekattavuus on erittäin hyvä. Pelilogiikan puolella testejä oli hieman vähäisesti. Mielestäni olisi hyvä kirjoittaa niitä enemmän mahdollisimman pian, koska ko. luokalla on keskeinen rooli ohjelmassasi.

Olit kirjoitellut hyvin kommentteja metodien toiminnasta. Niihin kun vielä lisäät (generoit) JavaDoc:n avainsanat (@param yms.), niin on tuokin puoli dokumentoinnista aloitettu.

Jäi hieman epäselväksi oletko tekemässä ohjelmaasi graafista käyttöliittymää, mutta mitä ilmeisemmin olet (?). Luokkakaavioon oli merkitty Käyttölliittymä-luokka, mutta Ilmeisesti tämä oli suunnitteilla / koodaamatta vielä? Joka tapauksessa käyttiksen toteuttaminen olisi hyvä aloittaa mahdollisimman pian, koska se vie ihan varmasti oman aikansa. (Side note dokumentoinnista: Kannattaa korjata ääkköset pois luokkakaaviosta niin vastaa korrektisti varsinaista toteutusta.)

Harmi, että vielä ei päässyt testaamaan toiminnallisuutta edes vähän pelin merkeissä, mutta koodi oli kokonaisuudessaan siistiä. Pidä sama linja jatkossakin! 👌