tspaanan
commented
1 year ago Kiitos vertaisarvionnista! Selkeytin sen perusteella vielä dokumentaatiota, noiden välilyöntien ja vastaavien kanssa rajoituksena on siis shell-tulkki (varmaan poikkeuksetta), jotka lukevat esim. välilyönnin aina parametrin loppumisena. Ongelma "kierretään" käyttämällä lainausmerkkejä, joskin jotkut erikoismerkit kuten ! on varattu muuhun käyttöön eikä niitä voi lainkaan antaa komentoriviltä (tässä ohjelmassa --message parametrin jälkeen). ! pitäisi antaa muodossa !, mutta ohjelmassa ei ole toiminnallisuutta sellaiseen kirjanpitoon, että mitkä kenoviivat ovat escape-merkkejä ja mitkä oikeita viestin osia :-) Menee monimutkaiseksi...
--input parametrilla voi tässä ohjelmassa kuitenkin antaa salattavaksi tiedoston, johon esim. heippa!Salataan voi olla kirjoitettuna: silloin kierretään shell-tulkin rajoitteet, käytännössä Python hoitaa koko matkan alusta alkaen ja lukee tiedoston sisällön ennen sen salaamista eikä ongelmaa synny.
Kiitos siis vielä, tämä vertaisarviointi pisti todella miettimään noita shell-tulkkien rajoja!
susannakinnunen
Projekti kloonattu 23.2. klo 9:53
Hei! Todella hienoa työtä.
Kirjalliset dokumentaatiot olivat huolellisesti tehtyjä. Määrittely- ja toteutusdokumentti antoivat hyvän pohjatiedon itse koodin tarkasteluun ja saivat lukijan kiinnostumaan aiheesta.
Testausdokumentissa kohtaa: "Tällä hetkellä ohjelman hyödyntämiä algoritmeja testataan muutamilla erilaisilla testisyötteillä." voisi hieman avata. Miksi juuri kyseiset muutamat testisyötteet on valittu? Valitettavasti en saanut ajettua testejä annetuilla ohjeilla.
Koodi oli selkeää, eri toiminnot oli jaoteltu moduuleihin ja funktioiden nimet olivat sopivan pituisia. Koodi oli kommentoitua erityisesti niissä kohdissa, joissa funktion tai muuttujan nimi ei jo itsessään kertonut sen toimintoa. Ehdotan, että create_keys.py-moduulissa sijaitsevaan gcd-muuttujan nimeä avataan enemmän.
Ohjelma toimi pääosin erittäin hyvin. Joidenkin viestien salaaminen ei onnistunut. Tässä viestit, jotka johtivat virheviestiin. _heippa! Salataan heippa? Salataan heippa. Salataan heippa!Salataan_
Seuraavaat syöteet taas toimivat:
heippa.Salataan heippa?Salataan
Voiko tällä olla mahdollisesti jotain tekemistä välilyöntien ja muiden kuin kirjainmerkkien kanssa? Endtoend-testauksessa viesti luodaan kirjaimista, ehdotan, että ohjelmaa testataan myös muunlaisilla merkeillä.
Kiitos vielä huolellisesti tehdystä projektista, minkä dokumentaatio ja koodi toimivat hyvänä opetusmateriaalina RSA-algoritmia aiemmin tuntemattomalle.