Vertaisarviointi 30.8

Klo 21

Yleipalaute

Aihevalinta on kiinnostava ja oli mielenkiintoista tarkastella projektia.

Projekti on hyvällä mallilla ja vaadittu dokumentaation sekä toiminnallisuus on totetutettu ja
testit ovat kattavia.

'Toiminnallisesti' koodi on hyvälaatuista. Metodit ja muuttujat on nimetty selkeästi.

Kommenteista ja koodin laadusta

Koodin on epäselvää lukea sillä se on täytetty kommenteilla. Kannattaa mieluummin kuvata metodi selkeästi javadocissa ja jättää metodin sisälle mahdollisimman vähän kommentteja. Kun luettavassa koodissa on monta riviä kommenttia se muuttuu epäselväksi.

Myös itsestääänselvät metodit kuten equals() ja stringToByteArray() eivät tarvitse tarkkaa selostusta.

Suosittelen myös CheckStyleä tai muuta vastaavaa sillä koodi en muutenkin hieman epäselvää luettavaa.

Tietorakenteista

Projekti käyttää vielä joitain valmiita tietorakenteita (Map, StringBuilder) ja nämä tulisi toteuttaa itse.

Loppupalaute

Projekti on hyvällä mallilla ja toimiva, mutta kannattaa keskittyä enemmän projektin selkeyteen.