Koodi ladattu, testtattu ja arvioitu 1.12.2021 klo: 15.05

Asennus ja komentorivikomennot

Asennus onnistui ohjeilla moitteettomasti ja komentorivikomennot toimivat pääsentöisesti niinkuin pitikin. poetry run invoke pylint komento ei toiminut etkä sellaista komentoa ole tasks.py tiedostoon tallentanut.

Dokumentaatio ja koodin kommentointi

Dokumentaatio on pääsääntöisesti kattavaa ja koodia kommentoitu laajasti sekä sitä on helppo lukea. Testi/toteutusdokumentti aika vaiheessa, mutta mielestäni se sallittakoon tässä vaiheessa projektia. Määrittelydokumentti jopa hieman liian pitkä, osa sisällöstä saattaisi kuulua toteutusdokumenttiin. Viikkoraportit voisivat olla selkeämmin kirjoitettuja (esim. väliotsikoiden käyttö).

Koodi

Projekti selkeästi vaiheessa, mutta kirjoitettu koodi selkeää ja helppo lukuista. Primes luokan lowPrimeList listan generoitiin saattaisi olla optimaalisempikin tapa (ks. Sieve of Eratosthenes). Myöskin testaillessani millerRabin metodia 1024 bittisillä luvuilla, sain sen jumiin ikuiseen silmukkaan, eli algoritmi ei ole aivan valmis. Miller-Rabin testin suorittaminen parametrillä k=6 ei myöskään ole ideaalia, koska todennäköisyys että hyväksytty luku ei ole alkuluku on 0.00024414062 (vertailun vuoksi käytän itse omassa Miller-Rabin testissä parametriä k=40, jolloin todennäköisyys että hyväksytty luku ei ole alkuluku on noin 8.27 * 10^-25).

Testaus

Testikattavuus on hyvällä tasolla. Mielestäni sinulla on iso ongelma testeissäsi. Testaat nyt että onko luku, jonka generoit omilla metodeillasi, alkuluku myös omilla metodoillasi. Eli mitä tarkoitan, jos sinulla on virhe vaikka alkulukun oikeellisuutta testaavissa metodeissa, miten voit olla varma että omilla metodeilla generoitu luku on alkuluku. Koska itse ohjelmasi jo tekee sen kertaalleen, niin tottakai testit menevät myöskin läpi. Pitäisi esimerkiksi testata tunnetulla alkuluvulla tai tunnetulla yhdistetyllä luvulla miten metodisi toimivat.

Hyvää duunia ja onnea jatkoon!

Vertaisarvointi https://github.com/jonthus/RsaAlgorithm -projektista

Dokumentaatio

1. README.md hyvännäköinne.
2. Dokumentaatio on ajantasalla.

Primes.py

1. generoitavien lukujen suuruus voisi olla muuttuja.
2. Primes.millerRabin, Primes.lowPrimes ja Primes.checkPrimality voisivat olla prefixattuja __ -merkkijonolla joka tarkoittaa että metoodi on C++:n private/protected -mallinen sisäisesti kutsuttava.

RSA.py

1. generateKeys, encryption ja decryption voisivat olla liitettyinä luokkaan?

ui/gui.py

1. Dokumentaatio-stringiä voisi laittaa mukaan metoodeihin.
2. sys.path.insert(0.... vähän hankalasti ymmärettävä. Parempi olisi pitää koodit yhdessä hakemistossa tai laittaa sys.path:iin projektin juurihakemisto, jotta voisi tehdä esim.: import RsaAlgorithm.src.RSA, tai jotain vastaavaa.

Unit testaus

1. Toteutettu kattavasti ja hyvin.

Yleisellä tasolla

1. RSA -salaus on huomattavan hidas algoritmi. Asiaan voisi auttaa että RSA:lla salataan vain esim. AES -salaisuuden, ja antaa AES:n tehdä enkryptauksen/dekryptauksen.
2. Tkinterin käyttö estää tehokkaasti testaamisen shellistä.

Johtopäätös

Erittäin hyvin tehty projekti josta on hankala sanoa mitään vikoja kun niitä ei näytä olevan.