Paatti-projektin toinen lähdekoodikatselmointi

Aika: Tiistai 15.5.2012 klo 15.10 - 16.13
Paikka: Sovellusprojektien kokoustila AgC226.1

Muistio:
- laadittu: 16.5.2012

Läsnä:

projektiryhmä:
- Tapio Keränen
- Toni Salminen
- Jari Salokangas
- Lauri Satokangas, sihteeri

ohjaajat:
- Jukka-Pekka Santanen
- Outa Valkama

----+----+----+----+----+----+----+----+----+----+----+----+----+----+----+----+

Valkama esitti seuraavat huomiot katselmoimistaan lähdekoodeista:
+ Ohjelmakoodi oli pääosin hyvin tehtyä
+ Ryhmä oli korjannut hämärästi nimetyt muuttujat ja komponentit
+ Ohjelmakoodissa olleet tyhjät rivit lisäsivät koodin luettavuutta
- Kommentteja puuttui muutamasta paikasta
- Kokonaiset nimet tekijöihin ja copyrightit mukaan kommentteihin
- Pari suomalaista kommenttia ja TODO:ta oli vielä ohjelmakoodissa
- Yksi huonosti nimetty rajapinta kannattaisi korjata
- Notifikaatioita voi näyttää ruudun keskiön sijaan oikeassa alareunassa
- Versiohallinnassa Branch ja Ticket on hyvä olla samannimisiä
- ViewID:t olivat Objekteina, vaikka ne sisälsivät pelkästään Stringin

Valkama huomautti, että kommentoidut rivit voi ottaa suoraan pois, sillä ne
löytyvät versiohallinnasta, jos niitä tarvitsee lisätä koodiin. Hän huomautti 
myös, että salasanoja ei ole salattu mitenkään. Hän myös kertoi, että 
tietokantatestausta ei kannata suorittaa oikealla tietokannalla, vaan käyttää 
testitietokantaa (H2 tai muuta vastaavaa) testaamiseen.

Santasen mielestä ohjelmakoodi sisälsi pääosin mukavan mittaisia koodirivejä.
Hän muistutti ryhmää tarkistamaan ja generoimaan JavaDocit ohjelmakoodista.
Lopuksi Santanen vielä kysyi, että onko dropTable-tyyppiset käyttäjän syötteet
estetty? Ryhmä ei ole tätä tehnyt ja Santanen totesi tähän, että tämä asia 
kannattaa kirjata jatkokehitysideoihin.

Kaiken kaikkiaan Valkama oli sitä mieltä, että ohjelmakoodi oli rakenteeltaan
erinomaista ja helposti luettavaa laatua. Valkama katsoo koodin läpi vielä
kertaalleen tänä viikonloppuna ja antaa tämän jälkeen lopullisen päätöksen
ohjelmakoodista ensi viikolla.