Ensimmäinen vertaisarviointi

Käyttökokemus

Käyttäjätunnuksen luominen, merkin lisääminen yhteisöön ja sieltä omiin kokoelmiin onnistui vaivatta. Ulkoasu on selkeä ja oletan sen olevan optimoitu puhelimia varten, koska sitä kautta se ainakin näyttää selkeämmältä.

Huomioita

Etusivulla ei ole selkeää funktiota, joka liittyisi sovelluksen toimintaan olennaisesti.
Uuden merkin lisääminen ensin yhteisöön ja sitten kokoelmaan on hieman kankeaa. Käyttäjälle voisi antaa valinnaksi merkin lisääminen suoraan omaan profiiliin.
"Katso lisätietoja / lisää merkki omaan kokoelmaasi" on vähän turhan pitkä teksti, eikä se täysin kuvaa napin toimintaa

Koodikanta

Koodi on selkeää, jaoteltu tarpeeksi fiksusti ja noudattaa pääsääntöisesti Pythonin yleisiä sopimuksia. En näe räikeitä virheitä tai puutteita, joten nillitän alempana hieman kurssin tavoitteisiin sopivammista tyyleistä tehdä samat toiminnallisuudet. Mainitsen myös räikeästä tietoturvaongelmasta.

Huomioita

https://github.com/lamtonylam/haalarimerkkidex/blob/49f5006a39572b19ef556e084842e1ff6e113664/sendpatch.py#L10 Voit korvata tämän komennon SQL:n puolella käyttämällä tietokannassa UNIQUE-rajoitusta. Sovelluksen toiminta luottaa ilmeisesti nimien uniikkiuteen, joten se kannattaa varmistaa ihan tietokannan puolella myös.

https://github.com/lamtonylam/haalarimerkkidex/blob/49f5006a39572b19ef556e084842e1ff6e113664/schema.sql#L15 Pääsääntöisesti kannattaa viitata taulusta tauluun aina id:llä.

https://github.com/lamtonylam/haalarimerkkidex/blob/49f5006a39572b19ef556e084842e1ff6e113664/routes.py#L44 Voit pistää nämä testit suoraan SQL:aan myös if-elsenä

https://github.com/lamtonylam/haalarimerkkidex/blob/49f5006a39572b19ef556e084842e1ff6e113664/routes.py#L195 Luonteva uudelleenohjaus olisi uuden merkin sivulle. Tällöin käyttäjä voi lisätä merkin suoraan kokoelmaansa. Toki monen merkin lisääminen on vähän vaikeampaa... Makukysymys.

https://github.com/lamtonylam/haalarimerkkidex/blob/49f5006a39572b19ef556e084842e1ff6e113664/routes.py#L137 Tämä kannattaa heittää .env-tiedostoon.

https://github.com/lamtonylam/haalarimerkkidex/blob/1f8767dfe6a5321b969935b602487a6f311f29e7/routes.py#L97 Sama kuin if data

Lopetus

Hyvin on lähtenyt projekti käyntiin. Pientä hiomista ja muutama ominaisuus lisää, niin hyvä tulee. Ei ole tässä vaiheessa mikään kiire, mutta kannattaa kokeilla hieman tyylitellä sivua jossain välissä. Käyttäjäkokemuksen helppokäyttöisyyteen kannattaa panostaa, koska käyttäjäkuntasi toimii pääsääntöisesti kognitiivisesti rajoittuneessa tilassa, jossa pienetkin jutut voivat ärsyttää.

lamtonylam / haalarimerkkidex