Tiralabra vertaisarviointi

Projekti ladattu 26.11.2020 klo 10:00

Koodikatsaus

Jos tiedostoa/luokkaa ei ole mainittu, en keksinyt kommentoitavaa ja se on hyvä.

Kompleksi.java

Hyvältä näyttää. Tämä on oma mielipide - ehkä voisi nimetä a ja b -muuttujat kuvaavammin, kuin yhdellä kirjaimella (esim. muuttuja a voisi olla nimellä reaali tms). Tämä saattaa helpottaa koodin lukua ulkopuolisena ja vähentää riskiä käyttää vahingossa samaa muuttujanimeä myöhemmin uudestaan esim. jossain luokan metodissa.

Toisaalta, onhan muuttujat selitettty dokumentaatiossa.

Kompleksiluvun laskuoperaatiot on tehty fiksusti.

Tallenne.java

Hyvä kattava virheenhallinta.

Tyylillisesti voisi olla tyhjiä rivejä eri kokonaisuuksien välissä lukemisen helpottamiseksi (vrt. tavallisen tekstin kappalejaot). Mutta tämä on taas näitä omia mielipiteitä ja onhan kyseessä on kohtuu lyhyt koodipätkä.

Projektin rakenne

Varsin hyvä omasta mielestä tämän laajuiseen projektiin, jossa kuitenkin algoritmin suunnittelu ja toteutus on pääosassa.

Jos projekti olisi laajempi, esimerkiksi käyttöliittymä voisi olla omassa paketissaan, tiedostojen lukeminen omassa paketissaan jne.

Testit

Yksikkötestejä on tehty laajasti ja testikattavuus on hyvä. Testikattavuuden lisäksi testejä tehdessä on mietitty myös ohjelman toiminnan oikeellisuuden testausta.

Fourierille on tehty myös suorituskykytesti.

Testausdokumentin mukaan lisää testejä on luvassa.

Testaus on myös automatisoitu hyödyntäen Github Actionsia, mistä plussaa.

Toiminnan testailu

Python-skriptillä generoidut taajuudet toimi hyvin.

Tein vielä kokeeksi Audacityllä (generate->tone) siniaaltoa 2093Hz taajuudella ja vain 0.01 amplitudilla ja sovellus osasi napata taajuuden täydellisesti. Näkyi toimivan myös square ja sawtooth -waveformeilla.

Kaiken kaikkiaan hyvältä näyttää.

vertaisarviointi 2

En valitettavasti saanut kummallakaan ohjeella jaria ajettua: no main manifest attribute, in app.jar. (Repositorio kloonattu 4.12.2020)

Kaikki oli todella hyvin dokumentoitua ja muuttujat yms. oli selitetty selkeästi.

Pakkausrakennetta olisi voinut jakaa hieman enemmän.

Testikattavuusraportin katsoin artefakteista ja vihreältä näyttää eli testikatttavuus näyttää hyvältä. Testit on jaettu moneen luokkaan sen mukaan mitä testaavat. Myös tosi hyvä idea laittaa repoon valmiiksi testinäytteitä. Harmi etten saanut ajettua ohjelmaa.