oaarne/otm-harjoitustyo

otm-harjoitustyo's Issues

Koodikatselmointi

Koodi ladattu 3.5 ~klo 17.25.

Koodin pakkausrakenne on erittäin selvä ja toimiva.

Koodi on hyvää, näin aloittelevana kotlinkoodarina törmäsin jopa täysin uusiin asioihin.
Luokat sisältävät jonkin verran käyttämättömiä muuttujia ja funktiota, sekä paljon kommentoitua koodia. Vaikka kehityksen aikana kommentoitu koodi voikin helpottaa muistamista, se myös tekee koodin lukemisesta ulkopuoliselle hidasta ja vaikeaa. Asiaa vaikeuttaa, että kommentit saattavat olla eri tasoissa, ja joissain tapauksissa rivinvaihtoja on liian vähän (esim MainView-luokka).

Testeihin liittyvät asiat jättäisin kokonaan testitiedostoihin (JsonSourceListDaon sample-funktiot).

Isompia hassuuksia en löydä, aiken kaikkiaan hieno ohjelma. Käyttöliittymä on erittäin siisti!

PS. en saanut ohjelmaa käynnistettyä gradlew:n avulla. Error: Could not find or load main class org.gradle.wrapper.GradleWrapperMain. Ideasta gradlen avulla käynnistäminen kuitenkin onnistui.