Koodikatselmointi

[hunnak]

Koodikatselmointi
https://github.com/iPegii/ot-harjoitustyo

Latasin projektin 27.4 noin kello 21.31.

Ohjelmakoodille on tehty selvät paketit dao, domain, mongodb ja ui. Paketeilla on selkeät nimet ja ne on kirjoitettu pienillä kirjaimilla. Pakettien sisällöt ovat järkevät, ne sisältävät aina yhden selvän kokonaisuuden.
Paketissa on monta eriteltyä luokkaa, joista jokaisella on jokin oma selkeä tehtävä. Luokkien, muuttujien ja metodien nimentä noudattaa yleisiä suosituksia. Kaikissa luokissa kielenä on käytetty vaadittua kieltä eli Englantia. CamelCase on selvästi läsnä muuttujien, metodien ja parametrien nimennässä.
Metodit ovat sopivan pituisia. Koodissa ei ole mitään häiritsevää toisteisuutta.
Tärkeitä testejä on tehty ja testaavat ohjelman kannalta järkeviä ja olennaisia asioita. :)

Kokeilin ajaa ohjelmaa ohjeiden mukaisesti:

Graafinen ikkuna ilmestyy ohjelman käynnistyessä.
Ohjelmaa ajaessa ohjeiden mukaisesti ja monella kertaa ja useimmalla kertaa yrittäen kaatui ohjelma aina samaan virheeseen,kun yritti toimintoa(nappia) create tai login.
Olin lisännyt tarvittavan keys.properties tiedoston ohjeen mukaisesti. Virhe toistui vaikka tein asiat alusta uudestaa, ja yritin käyttää toimintoja monella tapaa. Valitettavasti aina sama virhe koodi

Error: Something unexpected happened: null
java.lang.NullPointerException
at financetrackerapp.mongodb.UserService.getAll(UserService.java:72)

Ilmeisesti tietokannan käyttämisessä ongelma tai lisää ohjeita mitä tehdä ennen ohjelman ajamista.

Koodi näyttää erinomaisen laadukkaalta ja näyttää siltä, että jonkun toisen olisi helppo jatkaa ohjelman kehittämistä tästä eteenpäin, kunhan ohjelmanjatkajaa perehdytetään tietokantatiedostojen lukuun ja toimintaan. Kannattaa tarkistaa vielä kertaalleen ohjeesi muille ohjelman käyttäjille.

Hyvä ja selkeä vaatimusmäärittely, sekä arkkitehtuurikuvaus. :)