laakeri / tiralabra Goto Github PK

Moi, kävin koodisi läpi. Latasin sen 2.10.2016 noin klo 21.

Koodi seuraa aika hyvin c++ koodauskäytäntöjä ja koodi on jaettu erikseen header fileihin mikä on hyvä. Hieno Makefile myös niin oli helppo kääntää koodi ja ajaa testit bash scriptillä. Ne toimivat myös mun koneella.

Omissa tietorakenteissa on kivasti desctructorit jne niin että on clean code ja vältetään memory leakejä. Metodit ja muuttujat nimetty hyvin että ymmärtää vähän että mistä on kyse.

Koodin kommentointi olisi kyllä paikallaan, varsinkin kun on kyse näin vaikeasta algosta. Välillä oli vaikea seurata, että mitä joissain kohdissa tarkalleen tehdään. Ihan vaan joku yksi lause per metodikin auttaisi. Dokumentaatiota voisi muutenkin parantaa.

Toivottavasti tulevaisuudessa tulee myös lisää suorituskykytestausta. Joku missä näytetään, että kauan algoilla meni jne ja syötteiden kokojen ja tyyppien dokumentointi olisi varmaan hyvä.

Overall hyvää koodia!