hisahi / tiralabra-2019-des-aes Goto Github PK

Vertaisarvioinnin aloitusaika 11.4.2019. 20:00

Ennen tätä vertaisarviointia itselläni ei ollut mitään kokemusta tai käsitystä salausmenetelmistä, eikä koodikatselmoinin aikana täydellistä ymmärrystä sovelluksessa käytetyistä mekanismeista tullut, mutta hyvin kommentoidun koodin ja kattavan dokumentaation ansiosta minä sain aiheesta perusymmärryksen. Pidän tätä merkkinä siitä, että projekti on tehty hyvin.

En voi antaa selkeitä parannusehdotuksia koska minun kokemattomat silmät eivät huomanneet mitään moitettavaa.

Ainoa juttu josta nyt (ehkä) voin mainita on se, etten onnistunut käynnistämään sovelluksen heti käyttöohjeessa mainitun kommentin avulla (java -jar kryptoa.jar), en löytänyt kryptoa.jar tiedoston (tämä hyvinkin voi johtua minun huolimattomuudesta). Kun itse käänsin koodin, ohjelma lähti käyntiin ilman ongelmia ja on toiminut dokumentaatiossa mainitulla tavalla. Tämä pieni huomautus kuuluu dokumentaation, jota ei vertaisarvioinnissa ole määrä arvioida.

Kaiken kaikkiaan projekti on vakuuttava.

Koodi haettu 26.4 klo 16:21 commit

Projekti näyttäisi olevan jo valmis tai ainakin lähellä. Se ei kääntynyt jostain syystä pöytäkoneellani, mutta lähti pyörimään läppärilläni ilman ongelmia. Veikkaan että ongelma on openjdk-versiossa.
Lisäksi repossa oleva gradlew executable ei toiminut kummallakaan koneella. Sen voisi korjata sillä silloin projektin lataajalla on aina oikea versio gradlesta. Ongelma on kai että excludaat kaikki .jar tiedostot .gitignoressa jonka takia myös gradlew-wrapperi puuttuu reposta.

./gradlew build
Error: Could not find or load main class org.gradle.wrapper.GradleWrapperMain

Testasin ohjelmaa muutamalla eri algoritmilla enkä löytänyt mitään ongelmia. Testivipu oli kanssa siisti ominaisuus.

Itse koodi on selkeästi kirjoitettua, jaoteltua ja kommentoitua. Sitä oli välillä raskas seurata, mutta se taitaa olla täysin oman kokemattomuuteni syytä aiheen parissa.

Jonkinlaisia end-to-end testejä voisin ehdottaa, jos on pakko keksiä jotain rakentavaa palautetta/tekemistä.

Dokumentaatio oli runsasta ja pidin paljon kuvaajista, joita olit tehnyt.