Vertaisarviointi #2
Description
Moi,
mielenkiintoinen aihe. Valitettavasti joudun lomamatkan vuoksi arvioimaan jo tässä vaiheessa viikkoa, joten suuri osa tämän viikon työstä jää minulta näkemättä. Erityisesti BigInteger-luokan toteutus olisi ollut kiinnostava, koska se on niin olennainen osa tätä projektia. Projekti buildasi hyvin ja käyttöohjeet ovat esimerkilliset. Homma toimi hyvin ohjeen mukaan. En ole aikaisemmin tutustunut näihin alogritmeihin syvällisemmin, joten sainpahan mahdollisuuden lukea Wikipediaa.
Projektirakenne on hyvä ja koodi itsessään on selkeää ja hyvin ymmärrettävää, joten parannusehdotuksia joutuu lähinnä keksimällä keksimään. JavaDoceja puuttuu paikoittain ja itse varmaan kommentoisin myös algoritmien toimitaa metodien sisällä vähän tarkemmin, mikäli joku muukin osaamaton lukija sattuu paikalle. Uskon, että nämä ovat jo muutenkin työlistalla. Käyttöliittymässä voisi olla mahdollisuus tehdä AES+RSA suoraan PGP-tyylisesti, vaikka käyttöliittymä ei varsinaisesti olekaan tämän kurssin ydinsisältöä.
TiraRandom toimii satunnaisesti, kuten pitääkin.