4.6.2018, kello ~18

Yleisiä asioita
Aiheen määrittely on riittävän tarkka, eli rakennettavan sovelluksen idea välittyy selkeästi. Tietokannan skeemaa tai käyttötapauksia ei ole erikseen listattu, joten niitä ei voida katselmoida. Sovellukseen on tehty kirjautuminen, mutta testitunnuksia ei ole käytettävissä. Tämän pystyi kiertämään editoimalla sovellusta niin, että kirjautumiseen ei ohjata.

Positiivisia asioita:

• Kirjoittamasi koodi on esimerkkimateriaalin mukaisesti muotoiltua ja helppolukuista. Jatka samaan malliin!

• Sovellus pyörii Herokussa ja tietokantamäärittelyt on tehty koodiin, eli olet pitänyt sovelluksen toimivassa tilassa. Tämä on hyvä asia, vaikka projektisi on selvästi keskeneräinen, niin se on ainakin toimiva.

• Koodi on selkeästi jaoteltuna vastuita hoitaviin luokkiin ja ohjelman rakenne on heti nopealla vilkaisulla selvä. Olet noudattanut kurssimateriaalin muotoiluja hyvin ja välttänyt sooloilemasta. Tein itse samalla tavalla ja lopputulos on mukavan selkeä. Pieni kommentointi ei olisi pahitteeksi, mutta ei toki vaadittua.

Parannettavaa:

• Tietokantakaaviota ei ole näkyvissä. Kyseinen asia taisi olla viime viikon etapeissa, joten suosittelen piirtämään jonkin suuntaa antavan kaavion ensitilassa.

• Käyttötapauksia ei ole kirjattu. Suosittelen kirjaamaan muutaman yksinkertaisen käyttötapauksen ensitilassa.

• Kirjautuminen on tehty, mutta kirjautumistiedot eivät ole kirjattuna repositorion readmehen. Tämä aiheutti hieman päänvaivaa ja jouduin manuaalisesti editoimaan koodiasi, jotta pääsin tutkimaan toiminnallisuutta käytännössä. Kirjaa testitunnukset readmehen!

• (Kevyt CSS antaisi hieman virkeyttä projektille, mutta tämä ei ole pakollista tässä vaiheessa projektia)

Loppusanat
Projektisi on selvästi vielä keskeneräinen ja siitä puuttuu viikon etappien vaatimuksiakin. Olet kuitenkin aloittanut ja kirjoittamasi koodi on kohtalaisen laadukasta ja selkeää, joten en epäile etteikö projektistasi tulisi vielä hyvä kunhan annat sille hieman aikaa ja vaivaa.

Koodikatselmointi

Koodi ladattiin 19.6.18 noin kello 7:45.

Ohjelman ulkoasu on selkeä. Aluksi ihmettelin, missä ovat testitunnukset, mutta sitten tajusin, että sellaisia ei tarvita, koska jokainen voi luoda ne itse.

Käyttötapaukset

Käyttötapauksista henkilöiden lisääminen, poistaminen ja listaaminen onnistuu. Myös kirjautumistoiminnallisuus on toteutettu ja jopa uusien käyttäjien luominen on mahdollista. Sisäänkirjautumis- ja uuden käyttäjän luomis -näkymät ovat hyvin samanlaiset, ainoa ero on lomakkeen painikkeen tekstissä. Tämä onnistui sekoittamaan ensimmäisellä käyttökerralla. Näkymiin voisi lisätä otsikon, joka kertoisi selkeämmin mitä ollaan tekemässä.

Työntekijöille on mahdollista lisätä rooli, roolit kannattaisi varmaan lisätä omaan tauluunsa ja antaa käyttäjälle ainoastaan mahdollisuus valita jo valmiiksi määritellyistä rooleista sopiva. Pääkäyttäjä voisi hallita rooleja ja lisätä niitä.

Käyttötaupauksissa kuvatuista ominaisuuksista ainoa puuttuva on työtuntien lisääminen. Tällekin näytti koodissa olevan toiminnallisuusraakile valmiina, mutta käyttöliittymässä sitä ei päässyt vielä kokeilemaan. Tuntien lisäys -lomakkeen sai pienellä refaktoroinnilla näkymään, myös muu lisäystoiminnallisuus vaati vielä hieman hiomista, jotta lisäys onnistuisi.

Koodi oli selkeää ja tuntien lisäyksen toteutus oli ymmärrettävä, vaikkei toiminnallisuus ollutkaan vielä valmis. "Turhaa" koodia olivat esimerkiksi lomakkeen syötteen validiuden tarkastaminen, vaikkei lomakkeen kentille ole lisätty ehtoja. Toisaalta ominaisuus on hyvä olla olemassa, ettei sen toteuttaminen unohdu, jos kentille jossain vaiheessa lisätään ehtoja.

Yhtään virhettä ei testaamisen aikana tullut vastaan!

Parannusehdotuksia

• Tietokantamäärittelyissä voisi hyödyntää perintää. Copy-paste koodi vähenisi.
• Eri käyttäjille näkyy omat työntekijänsä, ei toisten, onko tämä tarkoituksellista?
• Etusivulla näkyvän työntekijälistan voisi muuttaa samanlaiseksi listaussivulla näkyvän kanssa.

Tsemppiä loppurutistukseen
Tuomas