ptr5000 / dummycoin Goto Github PK

Lähdekoodi ladattu 9.2. klo 16:04

Projektisi aihe on mielenkiintoinen ja määrittelydokumentissa tehty tehtäväpriorisointi selkeä.

Suurin osa projektin luokista ja metodeista on dokumentoimattomia. Toiminnallisuus on tosin jaoteltu loogisiin osa-alueisiin tiedostoittain ja luokittain, mutta esimerkiksi RSA:n toiminta jää melko lailla mysteeriksi, jos lukija ei erikseen selvitä sitä muualta. Kriittisempää dokumentoinnin kannalta olisi kuitenkin itse blockchainiin (blockchain.py, wallet.py ja transaction.py) liittyvien luokkien ja metodien dokumentointi. Tämä helpottaisi kokonaiskuvan hahmottamista etenkin pythonin kaltaista tyypittämätöntä kieltä käytettäessä.

Testit ovat myös melko epäkuvaavia. Mainitsit määrittelydokumentissasi toteuttavasi suuren osan testauksesta integraatiotesteillä, mikä entisestään lisää tarvetta testien dokumentaatiolle, sillä testattavat kokonaisuudet ovat suuria verrattuna yksikkötesteihin ja yksittäisissä testeissä näyttäisi olevan useita assertioita.

Itse koodi vaikuttaisi olevan idiomaattista Pythonia, osa muuttujista on tosin hieman kryptisesti nimettyjä (mikä on utxo?) Lisäksi esimerkiksi Transfer-luokan tarve on vähän kyseenalaista sillä send-metodi ei muokkaa olion sisäistä tilaa ja se on luokan ainoa metodi. Luokka olisi siis ehkä mahdollista muuttaa pelkäksi funktioksi.

Joku suoritettava tiedosto (esimerkiksi komentoriviltä), joka esimerkiksi louhii yhden lohkon (?), lisää sen ketjuun ja tulostaa ketjussa olevat hajautusarvot (jos ymmärsin blockchainin arkkitehtuurin oikein) olisi hyvin havainnollistava ennen kuin web-käyttöliittymä on toteutettu.

Lähdekoodi ladattu 16.2 22:58

Aihe vaikutti mielenkiintoiselta. Itselleni se osin on varsin vieras aihe, mikä vaikeuttaa ymmärtämistä ja arviointia. Harmi ettet ollut vielä tehnyt Demo-UI:ta.

Osa koodista vaikutti ihan hyvin kommentoidulta (esim. Blockchain), osassa koodia taas kommentteja olisi voinut olla enemmän. Muuten koodi vaikutti olevan laadukasta.