Vertaisarviointi 1

[pllk]

Käyttökokemus:

• Sovelluksesta tulee hyvä ensivaikutelma: ulkoasu on selkeä ja tyylikäs. Sovelluksessa on jo monia toimintoja, joita voi käyttää.

• Varauksen tekemisen käyttöliittymää voisi vielä parantaa, koska nyt se on vähän vaivalloista eikä ole selvää, mitkä varaukset ovat mahdollisia. Kun yritin tehdä uuden päällekkäisen varauksen, niin myös aiemmin tekemäni varaus katosi väliaikaisesti (?).

Tekninen toteutus:

• Sovellus on ammattimaisesti toteutettu ja koodia on helppoa lukea.

• Tiedostossa municipality_repository.py rivien 17–18 kysely toimisi varmaan myös ilman GROUP BY ja voisi myös olla luontevampaa tehdä se ilman alikyselyä.

• Tiedostossa reservation_repository.py rivillä 11 sanan VALUES jälkeen tuntuisi paremmalta, että siinä on välilyönti.

• Tiedostossa register_routes.py funktio validate_email tuntuu hyvältä nykyisellään eli ei ole mielestäni tarvetta TODO-asialle. Sähköpostin varmistaminen oleellisesti tuota paremmin olisi luultavasti hyvin hankalaa.

• Joissain kohdissa koodissa luodaan totuusarvoja tyyliin is None ja is not None. Python-tyylisempää olisi jättää nämä pois, koska None tulkitaan suoraan epätodeksi ja muut todeksi.

• Osa tiedostoista on hyvin lyhyitä (kuten municipality.py ja review.py). Tällaisten tiedostojen yhdistämistä voisi harkita, jos se on mielekästä.

[mjjs]

• Varausten tekeminen on tosiaan hieman surkeaa tällä hetkellä. Näköjään olen unohtanut muuttaa sitä tehdessäni muut templatet uudestaan. Hyvä huomio!
• Käytössä olevien kuntien hakemisesta tosiaan voi pudottaa GROUP BY pois. Alikyselyn tekeminen tuntui itselle luontevalta tavalta, mutta täytyypä vielä tutkia jos keksisin jonkin järkevämmän tavan.
• Sähköpostiosoitteen järkevämpi validointi on tosiaan ehkä turhan kompleksia kurssin tavoitteisiin nähden, joten taidan jättää sen toteuttamatta.
• None tarkistukset korjattu enemmän Python-tyyliseksi!
• Sovelluksen käyttämät dataluokat (esim. Review) voisi tosiaan siirtää osaksi niitä käyttäviin tietokantatiedostoihin.

Tuota bugia missä vanhat varaukset katoaa en saanut toistettua, mutta tutkin onko siinä jotain outoa logiikkaa.