Koodikatselmointi

[ghost]

Sisältö ladattu 31.5.2015 kello 19:08

Dokumentaatio on hyvää ja selkeän näköistä. Käyttötapauskaaviot puuttuvat. Käyttötapaukset voisivat olla hieman tarkemmin kuvattuina.

Tietosisältö ja siihen liittyvät kaaviot ovat muun dokumentaation tapaan säntillisen oloisesti tehtyjä, mutta tietosisältökaavio ei taida olla ihan ajan tasalla: Lento -entiteetti, joka tietosisältökaaviosta löytyy, puuttuu sekä sanallisesta tietosisältökuvauksesta, tietokantakaaviosta, että SQL/DDL-lauseista (tosin, nyt tarkemmin katsottuani se alkaa vaikuttamaan tietoiselta valinnalta). Lisäksi Ateria- ja Aterian valmistaja -entiteettien välinen suhde ei taida olla ihan oikein tietosisältökaaviossa.

Tehdyissä DDL-lauseissa ei näy olevan minkäänlaista yhteyttä Asiakas -entiteetin ja Ateria -entiteetin, eikä Asiakas -entiteetin ja Tilaus -entiteetin välillä. Myöskään Tilaus -entiteetin ja Tavara -entiteetin välillä ei yhteyttä ole. Nuo yhteydet kuitenkin ilmenevät dokumentaatiosta.

Repositorio on, kuten projektin yleisolemus muutenkin, hyvin selkeästi organisoitu. Koodi on hyvin jäsennelty.

Kokeilemaan en tuota päässyt. Etusivulla oleva linkki sovellukseen näyttää pelkästään "Hello World!", ja linkit käyttöliittymään osoittavat GitHubin tiedostoihin, joista näen kyllä sorsat, mutta en ulkoasua. NetBeansilläkään en saanut projektia ajettua, mikä saattoi johtua siitä ettei omalla koneellani ollut kaikkea tarvittavaa asennettuna. En vaivautunut selvittelemään noita tarkemmin, eikä se taida olla tässä katselmoinnissa pointtikaan.

Kaiken kaikkiaan tämä on minusta monessa mielessä aika lupaavan näköinen. Hiukan mulle tulee mieleen tuosta suunnitelmasta, että onko projektin scope liian laaja (mikä on itsellenikin sangen tuttua); sisällöllisesti paljon suppeampikin suunnitelma mahdollistaisi parhaan mahdollisen arvosanan. Ajatuksena heittäisin että voisi olla hyvä katsoa tuota vähän kauempaa ja miettiä josko tuosta voisi jättää jotain (esimerkiksi koko Ateria -haaran?) suosiolla pois, tai vaikka jättää taka-alalle siihen saakka että muusta osasta on toimiva runko kasassa.

Onnea harjoitustyön jatkolle!