Ladattu 25.02.2019 klo 22.00

Koodikatselmointi

En saanut käyttöohjeiden mukaan sovellusta toimimaan paikallisesti sillä se ei käynnistyessään luo tietokantatauluja ja täten valittaa niiden olemassaolemattomuudesta.

Käyttöohjeissa myös ensimmäinen kohta väärin, pitäisi olla pip install -r requirements.txt.

Toiminnallisuus

Herokussa sovellus toimii ohjeiden mukaisesti Lisää työntekijä -sivulle navigoidessa voi lisätä työntekijän ja validointi toimii oikein. Työntekijä tulee lisäämisen jälkeen näkyviin listaukseen. Listauksessa työntekijän tilaa vaihtava nappi tuntuu hieman turhalta sillä tilan vaihtamisella ei näytä olevan mitään käytettävyyttä.

Kiirellisyysluokan lisääminen ja näyttäminen toimii myös moitteettomasti.

Työviikkojen lisääminen tuntuu hieman turhalta kun eikö ne voisi olla jo järjestelmässä kalenterina josta voisi sitten muuttaa tuntien kiireellisyysluokkaa? Viikon lisääminen ei myöskään validoi, että ei voi lisätä jo mennyttä viikkoa.

Koodi

Sovelluksessa on aikas monta tiedostoa ja kansiota, jotka on kuitenkin nimetty johdonmukaisesti joten koodia oli helpohkoa seurata. En löytänyt koodista mitään kritisoitavaa kaikki modelit, viewsit ja formit tekevät mitä niiden kuuluukin tehdä.

Monimutkaisempia tetokantakyselyjä ei ollut kylläkään kuin yksi, jossa etsitään kaikki työntekijät joilla on alle 40h töitä mutta tämä ei oikein tällä hetkellä tee mitään sillä työntekijöille ei voi vielä edes asettaa työtunteja.

Ulkoasu

Sovellus näyttää kivalta ja listat ja lomakkeet on tyylitelty.

Sovelluksen avautuessa käyttäjän voisi ohjata suoraan kirjautumisruutuun sillä tyhjä valkoinen sivu ei ehkä ole ihan paras ensivaikutelma sovelluksesta.

Samoin sovellukseen kirjauduttua sillä voisi olla jokin etusivu johon vasemmassa yläkulmassa oleva sovelluksen nimi linkkaa sillä tällä hetkellä nimi vaikuttaa klikattavalta ja kursori muuttuu pointeriksi mutta linkki ei johdakkaan mihinkään.

zip ladattu 13.2.2019 klo 9:15

Sovelluksen toiminta selaimella (Heroku)

• Sovelluksen rakenne on looginen ja kenttien validointi toimii hyvin. Saavutettavuudessa ei mielestäni ole selkeitä ongelmia.
• Työntekijöiden lisääminen tunteihin ei sovelluksella tällä hetkellä onnistu eikä työntekijä pysty tarkastelemaan omia viikkojaan. Nämä ominaisuudet ovat varmaankin tulossa myöhemmin.
• Työntekijöiden listausnäkymästä löytyvän painikkeen tarkoitus jäi epäselväksi. Ilmeisesti sillä voi vaihtaa työntekijän aktiivisuuden tilaa, mutta mihin tilan muuttaminen liittyy?
• Työviikon listausnäkymän muotoilu ei toimi oikein.

Sovelluksen koodi

• Luokkien ja metodien nimeäminen on johdonmukaista ja kansiorakenne selkeä.
• Metodissa viikot_create on kohta "viikko.account_id = current_user.id" (rivi 28), jonka tarkoituksena on ilmeisesti liittää nykyinen käyttäjä luotuun viikkoon. Viikko-tietokantataulussa ei kuitenkaan ole account_id -attribuuttia, joten kyseinen komento ei taida tällä hetkellä tehdä mitään.
• Tietokantakaaviossa kiireellisyysluokka- ja tunti -käsitteiden välille on merkitty yhden suhde moneen -yhteys, mutta itse sovellukseen kyseistä yhteyttä ei ole toteutettu.

Sovelluksen tietokantakaavio

• Tietokantakaavio vastaa suurelta osin toteutettua tietokantaa.
• Olisi mielestäni selkeämpää, jos kaaviossa esitellyt attribuutit olisi nimetty samalla tavalla kuin tietokannassakin. Monen taulun kohdalla on nyt niin, että tietokannassa attribuutit ovat englanniksi (esim. name) ja kaaviossa puolestaan suomeksi (nimi).
• Tietokantakaavioon voisi kaikkien käsitteiden kohdalle lisätä myös date_created ja date_modified -attribuutit, jotka kuitenkin sisältyvät kaikkiin luotuihin tietokantatauluihin.
• Tietokantakaavioon voisi päivä -käsitteen kohdalle lisätä foreign key -viittauksen viikko -käsitteeseen.
• Työntekijä -käsitteeseen voisi lisätä attribuutit username ja hallinto.
• Tietokannan rakenne on tällä hetkellä aika monimutkainen. Voisiko esim. viikon, päivän ja tunnin yhdistää samaan tietokantatauluun? (Ainakin tämänhetkisen toiminnallisuuden perusteella tuntuu siltä, että kaikille tietokantatauluilla ei ole selkeää tarvetta. On tietysti mahdollista, että tilanne tulee myöhemmin muuttumaan eikä tämä ehdotus ole enää järkevä.)

Muuta

• Dokumentaatiossa on jonkin verran kirjoitusvirheitä.