niinam / otm-harjoitustyo Goto Github PK

Latasin projektin torstaina 26.4. kello 16:32. Tarkastelemani versio lienee vielä melko keskeneräinen, ainakin vaativuusmäärittelyssä listattuihin ominaisuuksiin verrattuna.

Ohjelman koodi on rakenteeltaan selkeä, eikä ole vaikeaa ymmärtää miten ohjelma toimii. Testejäkin näyttää olevan sopivasti. Kaikki menivät läpi, ja ne testaavat oleellisia ohjelman ominaisuuksia.

Koodin ulkonäön suhteen ainoa kritiikkini on se, että Closet-luokassa jotkin metodien nimet ovat hiukan pitkiä ja vaikeaselkoisia. Jos tulevissa versioissa metodeja kutsutaan koodissa usein, voi olla luettavuuden kannalta hyödyllistä käyttää lyhyempiä nimiä.

Ainakin tässä versiossa Main-luokka käyttää kaikkia domain-pakkauksen luokkia. Jos aiot myös jatkossa käyttää kaikkia domainin luokkia, ne saa importattua kaikki kätevästi yhdellä rivillä kun korvaa kaikki importit yksittäisellä importilla import com.mycompany.vaatekaapinhallintajarjestelma.domain.*;