EeroAnt
commented
1 year ago Tosi hyvää palautetta kaikin puolin, kiitos! Jännittävää, että tuo 'poetry install' ei asentanu suoraan kaikkia riippuvuuksia. Pitänee tarkastella sitä. Kaikki huomiot oikein valideja ja pitää ottaa tutkailuun. Noita negatiivisiä avaimia olin itsekin kohdannut ja miettinyt, että pitääköhän puuttua. Onneksi sen korjannee kohtuu kivuttomasti. Kiitokset vielä!
Moi!
RSA-koodarit on tosiaan paritettu ainakin tällä ensimmäisellä kierroksella, joten tässä palautetta.
Jouduin painiskelemaan hetkisen verran oman ympäristöni kanssa, sillä laitoksen fuksiläppärin python/poetry-versiot olivat jonkinlaisessa solmussa. En ole ottanut sudoja käyttöön fuksikoneella, joten siirryin omalle Windows-läppärilleni ja sain pienen taistelun jälkeen homman rullaamaan. Nämä ongelmat siis täysin omasta ympäristöstäni johtuvia, ja repon README:ssä olevat komennot toimivat alkukonffausten jälkeen erinomaisesti.
Avainten generointi oli nopeaa ns. järkevillä syötteillä, ja suurempien bittimäärien ja haastokierrosten yhdistelmät luonnollisesti jumittivat käyttöliittymän. Huomasin näppäillessäni liian vauhdikkaasti, että tyhjä komento (pelkkä enter) tai kirjaimen syöttäminen kaataa käyttöliittymän, sillä se odottaa numeroa:
Avainten tallettaminen tiedostoihin toimii hienosti. Testasin myös ääkkösiä sisältäviä nimiä sekä pelkkää alaviivaa (_), ja molempien käyttö tiedostonimenä onnistui. Ohjelmassa voisi ehkä olla jokin ohjeistus siihen, ettei käyttäjältä itseltään vaadita tiedostopäätteen määrittelyä, sillä onnistuin vahingossa saamaan aikaiseksi kelvottoman avaintiedoston antamalla tiedostonimeksi "reetta.txt":
Yksityinen avain saattaa muodostua negatiiviseksi, mistä aiheutuu käsittääkseni ongelmia siinä vaiheessa, kun avaimia aletaan oikeasti käyttää salaamiseen ja salauksen purkamiseen. Tuo kannattaa pitää mielessä, jos seinä tulee vastaan myöhemmillä viikoilla!
Koodi on mielestäni selkeää ja loogisesti jäsenneltyä. Voisiko avaingeneraattori.py:stä ehkä siirtää apufunktioita muualle, jotta kovempi matematiikka pysyisi yhdessä paikassa ja generaattoritoiminnot toisessa? Tämä kommentti saattaa kyllä kummuta siitä, että olen tottunut oman ohjelmani rakenteeseen ja kaikki muu tuntuu vieraalta.
Koodin kommentit ovat mielestäni hyvin kirjoitettuja ja helposti ymmärrettäviä. En ole itse kovinkaan matemaattisesti suuntautunut, joten joudun aina keskittymään huolellisesti lukiessani laskenta-algoritmeja, ja välillä tuli mieleen, että jokin rivikommentti voisi olla avuksi. Toisaalta olet onnistunut pitämään funktiot ihanan tiiviinä ja lyhyinä, eikä sinne väliin varmaan mahdu enempää kommentteja, jos jäljen haluaa pitää selkeänä.
Yritin ajaa testejä kylmiltään, mutta minulta puuttui pytest. Sen ladattuani sain virhettä sympyn puuttumisesta:
Latasin senkin, minkä jälkeen sain ajettua testit ja ne menivätkin läpi.
Kaiken kaikkiaan ohjelma näyttää minun näkökulmastani olevan hyvällä mallilla! Kiva huomata, että olemme päätyneet aika lailla samoihin ratkaisuihin RSA-härveleidemme kanssa. Hyvää työtä!