Koodikatselmointi

Ladattu 8.6.2018 klo 21.00

Käyttöönotto ja yleisilme

Selkeä käyttöohje, kaikki toimi ensi yrittämällä halutulla tavalla! Erittäin helposti omaksuttava käyttöliittymä, ainakin jos vähääkään ymmärtää terminaalin päälle. Jopa testiraportin generoiminen kansiolle toimi aivan kuten tarkoitettu. Yleisilme ja käytettävyys siis hyvällä mallilla.

Testit

Gradlen testien ajaminen toimi moitteettomasti ja virheettä. Jacocon testikattavuus myös erittäin korkea, paikoin 100 %. Omalle testikansiolle ajettu testiraportti antoi parhaimmillaan compress ration arvoksi 4 % ja enintään 33 %. Myös tältäkin osaa ohjeistus oli tarkoituksenmukaista ja hyvin laadittu. Enää tarvitsee laajentaa itse testausdokumenttia vähän enemmän. (Mikä nähtävästi on jo suunnitteilla: “Documentation: testing document shall be written and implementation document shall be enhanced.”)

Koodin laatu

Koodi erittäin helppolukuista, johdonmukaisesti ja kevyesti kirjoitettua. Paikoitellen hyvinkin nokkelia optimointeja käytetty, mitkä kielivät kokemuksesta ohjelmoinnin parissa. Samaten javadoc kertoo ansiokkaasti metodien toiminnasta. Checkstyle tuottaa ainoastaan yhden virheen: luokassa PerformanceTester on rivi 168 on yli 100 merkkiä pitkä.

Dokumentaatio

Dokumentaatio ja etenkin käyttöohje on, kuten yllä mainittu, erinomainen. Pidän etenkin monista kaavioista ja julkaistuista suorituskykytestien tuloksista.

Lopuksi

Kaiken kaikkiaan vakuuttavaa työtä! Aiheeseen vähemmän perehtyneelle voi olla vaikeaa antaa itse toiminnallisuudesta paljoakaan teknisiä neuvoja, paitsi ehkä, että projekti näyttää monelta osin olevan jo valmis! Enää pieniä viilauksia työhön ja se on siinä!