Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Koodikatselmointi #1

Open
Vekkumasa opened this issue Apr 27, 2020 · 0 comments
Open

Koodikatselmointi #1

Vekkumasa opened this issue Apr 27, 2020 · 0 comments

Comments

@Vekkumasa
Copy link

Projekti ladattu 27.4 12.58

Koodi on helposti luettavaa ja selkeää. Jokaisella luokalla ja metodilla on selkeästi oma roolinsa ja pakkausrakenne on selkeä.

Checkstyle virheitä oli vain muutama, joista kaikki viittasivat metodien pituuteen (toisaalta tietokantaa käsittelevät metodit tuppaavat olemaan aika pitkiä varsinkin jos jokaista saraketta käsittelee eli asialle ei oikeastaan voi mitään.)
Testikattavuus todella laaja ja testit ovat järkeviä (ja kaikki menevät läpi.)

Sovellus validoi käyttäjän syötteet hienosti, esim uutta havaintoa luodessa on mahdolliset arvot ilmoitettu selvästi ja mikäli siltikin käyttäjä yrittää laittaa väärää arvoa kenttään luodaan siitä asiallinen popup-window mikä listaa kaikki virheet lomakkeella.

Todella hyvää työtä, en valitettavasti keksi projektiisi parannusehdotuksia.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant