Vertaisarvio

[koedi]

Checkout time 22.04.2021 21:35

Yleistä

Koodi on selkeää ja helposti luettavaa. Metodit ovat sopivan pituisia ja niiden kommentit (JavaDoc) ovat asiallisia. Muuttujien, metodien ja luokkien nimeäminen on selkeää jaj johdonmukaista

Joihinkin oleellisiin kohtiin koodissa, esim. Solver, voisi lisätä lyhyen kommentin mitä siinä tapahtuu. Tällöin olisi helpompi seurata koodia. Varsinkin tässä kun kyseessä on hieman erikoisempi algoritmi niin pienimuotoinen lisäkommentointi auttaisi.

Implementaatiosta

Satunnaislukugeneraattorista olisi voinut luoda oman version. Nyt käytössä on Javan Random.

GeneratorUI::start(). Tätä pystyisi hieman siistimään siirtämällä osan toiminnoista omaan metodiin. Nyt kun UI:n luonti on melko lyhyt niin nykyinen toteutus on ihan ok. UI-elementtien lisääntyessä metodin seurattavuus ja pituus lähtee ns. lapasesta :)

Sovellus

Sovellus toimii jouhevasti ja UI on selkeä ja helppo käyttää.

Virheviestit on varmaan osa implmentaation seuraamista, mutta käyttäjälle epämääräisiä

Propagation error, retry round 48
Propagation error, retry round 49
Propagation error, retry round 50
Propagation error, retry round 51
Propagation error, retry round 52
Propagation error, retry round 53

Sokkeloon jää irrallisia osia, mutta nämä varmaan poistuu lopulliseen versioon mennessä. Vai onko ne algoritmiin liittyviä artefakteja?

Sokkelon graafinen esitys 5/5. Oikein hyvä ja selkeä sokkelo.

Yhteenveto

Hyvää ja siistiä koodia. Valittu algoritmi on varsin mielenkiintoinen ja erikoinen (ainakin minulle). UI on selkeä ja yksinkertainen. Tästä on hyvä jatkaa lopulliseen versioon.