Koodikatselmointi #1
Description
koodi ladattu 26.11.2014 noin klo 22:00
Koodi on erinomaisesti kommentoitu ja Java-doc on kattava. Jo nopealla vilkaisulla saa ajatuksen mitä metodeissa tapahtuu. Luokat ja niiden sisältö on jaoteltu selkeästi ja sama linja jatkuu metodien ja muuttujien nimeämisessäkin. Myös testit on tehty samoja periaatteita noudattaen. Visuaalisesti koodi on myös selkeää luettavaa ja se on sisennetty hienosti. Koodista on karsittu pois myös turha toisto.
Testejä on tehty jokaiselle luokalle ja myös testit on tehty kattavasti. Koitin ajaa testejä ja vain kaksi yhdestätoista meni läpi. Mahdollinen ongelma on Node-luokassa "Could not initialize class com.mycompany.tiralabra_maven.Node" sillä en saanut luokan ainoaa importia toimimaan com.sun.istack.internal.NotNull; Tähän olisin kaivannut jotain ohjeistusta vaikka vika onkin minussa :). Toinen asia mikä silmään pisti oli Kartta-luokka, sillä kaikki muu sisältö on englanniksi. Lisäksi näin ulkopuoliselle olisi kiva löytyä jotain visuaalista ohjelmaa ajaessa, nyt tuo "Hello world" jäi ainoaksi tapahtumaksi itse pääluokkaa suoritettaessa. Esimerkiksi lopullinen reitti voisi olla kiva nähdä, kuten testausdokumentissa olikin kuvia...
Kaiken kaikkiaan koodia oli ilo lukea, mikään ei ollut huonosti :)