Koodikatselmointi

[laurivaananen]

Projekti ladattu 05.05 07:00

Toiminnalisuus

Osallistujien tietojen lisääminen, muokkaaminen, poistaminen ja tarkistaminen toimi hyvin.
Osallistujien osallistumisen lisääminen toimi, mutta muokkaamiseen tai poistamiseen ei löytynyt ominaisuutta.
Tulostiedoille ei löytynyt ominaisuutta.
Käyttäjän autorisointi toimi hyvin.
Sivulle rekisteröityminen ja kirjautuminen toimi hyvin.
Henkilöiden etsiminen nimen perusteella toimi erinomaisesti.

Bugeja

people/views.py

Globaalien muuttujien käyttäminen on yleensä erittäin huono käytäntö. Käyttäjä lisätään aina siihen tapahtumaan, mikä seriesIdGlobal on. Tapahtumaan lisäämisessä jos olet valinnut selectSeriesForm arvoksi A, mikä muuttaa seriesIdGlobal arvoksi 1 ja ennen kuin ehdit painaa Lisää Tapahtumaan nappia, joku toinen käyttäjä vaihtaa selectSeriesForm arvoksi B, mikä muuttaa seriesIdGlobal arvoksi 2 niin henkilö lisätään tapahtumaan B.

series/models.py

Tapahtumien listaus ei toiminut Herokussa, mutta toimii lokaalisti. Tämä johtuneen Sqliten ja Postgresql eroista. Postgresql on case insensitive ellet erikseen laita nimiä lainausmerkkien sisään. Eli komento SELECT Item.Name FROM Item muuttuu muotoon SELECT item.name FROM item. Koodissasi osa nimistä on lainausmerkkien sisällä ja osa ei. Hyvä käytäntö Postgresql käyttäessä on, että kaikki taulujen ja sarakkeiden nimet kirjoitetaan pienellä ja sanat erotellaan alaviivalla niin ei tule outoja bugeja. Lisää infoa aiheesta: https://blog.xojo.com/2016/09/28/about-postgresql-case-sensitivity/

forms.py

Tekstikenttien maksimi pituutta ei validoida. Päivämääriä ei validoida tarpeeksi. Esim syntymäpäiväksi voi valita päivämäärän mikä on tulevaisuudessa ja tilaisuuden lähtöajaksi ajan mikä on menneisyydessä.

templates/people/list.html

Riviltä 4 puuttuu formin lopusta puuttuu > merkki <form method="POST" action="{{ url_for('person_search') }}"

Sivun ulkoasu ja käytettävyys

Päivämääriä ja aikoja annettaessa voisi antaa vihjeen missä muodossa se pitäisi antaa esim (yyyy-mm-dd hh:mm:ss)

Sivun ulkoasu on siisti ja sitä on helppo navigoida.

Käyttäjän lisääminen johonkin tapahtumaan on hieman epäselvä. Tämän helpottamiseksi voisi tehdä omat näkymät käyttäjien listaamiseen ja käyttäjän lisäämisen tapahtumaan.