Koodikatselmointi #17
Description
Projekti noudettu 3.5. kello 12:00
Yleistä
Projekti noudattaa hyvää ohjelmointityyliä. Ohjelman toiminnallisuus on jaettu selkeästi eri luokkiin niin, että sen rakenne on helposti hahmotettavissa. Koodin tyyliseikat ovat kunnossa.
Testit ovat kattavia, ja niitä on tehty useille luokille.
Joitain havaintoja
Main-luokka ei tee muuta kuin aloittaa uuden ScentUpGui:n suorituksen. Main-metodi voisikin olla suoraan ScentUpGui-luokassa.
Luokan ScentUpService metodi createUserScent sisältää aika paljon parametreja. Kaksi ensimmäistä parametria voisivat riittää, koska dateNow:in arvon voi määrittää luomisen yhteydessä, ja kaikille uusille UserScent-olioille asetetut muuttujat pref ja act taitavat olla samat.
ScentUpService:n metodi createUser tarkistaa käyttäjänimen olevan vapaa ennen käyttäjän luomista, mutta nimien pituuksien tarkistus toteutetaan käyttöliittymässä, mikä on mielestäni hieman hämmentävää.
ScentDao-luokalla on kaksi delete- ja findOne-metodia. Niiden koodi on pääasiassa samaa, mutta niillä on eri hakuehdot. Jos kyseisestä ratkaisusta on hyötyä, voisi metodeille yhteisen sisällön kuitenkin siirtää yhteen metodiin.
Koodiin on jäänyt hieman suomenkielisiä muuttujia luokkiin ScentDao ja UserDao (poistettava, stmtriippuvuudet), sekä luokkaan Database (lista).