Skip to content

Vertaisarviointi #2

@uberballo

Description

@uberballo

13.40

Projekti näyttää olevan hyvässä kunnossa! Itse vinkkejä ei tullut mieleen, mutta enemmän pieniä muistutuksia. Dokumentaatiota, että itse koodissa on vain erittäin pieniä "korjattavia" kohtia.

Dokumentaatiossa käyttöohjeessa mainittiin release mitä ei ollut, jonkinlainen luokkakaavio olisi hyödyksi. Muuten ei mitään erikoista. Lyhyet sekä simppelit. Riippuu itse lukijasta kuinka laajasta dokumentaatiosta välittää.

Testeistä voi poistaa turhia metodeja, esim public static void tearDownClass(), mutta tämäkin on erittäin pieni asia. Arvostan kyllä laajoja testejä.

Itse koodi myös näyttää hyvältä. En huomannut mitään korjattavaa tai parannettavaa. Parit kommentit ja kaikki kunnossa!

Tämä projekti on jo valmis. Ei ole suurempia parannuksia tai korjattavia itse koodissa. Edellä mainitut korjaukset olivat pieniä, sekä mitkä varmaan korjaat jo viimeiseen palautukseen mennessä. Ei muuta kuin kehuja! Hyvin tehty

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions