rsaalgorithm's People
rsaalgorithm's Issues
Vertaisarviointi
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!
Vertaisarviointi RsaAlgorithm-projektista
Vertaisarvointi https://github.com/jonthus/RsaAlgorithm -projektista
Dokumentaatio
- README.md hyvännäköinne.
- Dokumentaatio on ajantasalla.
Primes.py
- generoitavien lukujen suuruus voisi olla muuttuja.
- 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
- generateKeys, encryption ja decryption voisivat olla liitettyinä luokkaan?
ui/gui.py
- Dokumentaatio-stringiä voisi laittaa mukaan metoodeihin.
- 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
- Toteutettu kattavasti ja hyvin.
Yleisellä tasolla
- RSA -salaus on huomattavan hidas algoritmi. Asiaan voisi auttaa että RSA:lla salataan vain esim. AES -salaisuuden, ja antaa AES:n tehdä enkryptauksen/dekryptauksen.
- 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.
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.