Vertaisarviointi 2

[riikkayoki]

Käyttökokemust

• Käyttökokemus on hyvä.
• Lisätessä ainesosia ostoslistaan, tulee ''Bad Request'' -sivulle.
• Reseptiäluodessa tulee internal service error.
• Sovelluksen navigointia on helppoa.

Muuten kaikki toimi hyvin!

Tekninen toteutus

• Koodi on siistiä ja selkeää.
• Koodin voisi jakaa vielä apufunktioihin niin, että yksi funktio käsittelee vain yhtä asiaa. Esimerkiksi add_recipe() ja add_ingredients() funktioista voisi poistaa kaiken koodin mikä ei liity tietokantoihin.
• Yleisesti funktioiden nimet ovat kuvaavia lukuunottamatta paria poikkeusta esim. juuri add_recipe() nimi kertoo, että luotaan resepti, mikä on kyllä totta, mutta toisaalta funktiosta haetaan recipe_id. Myös tästä syystä funktioita voisi jakaa apufunktioihin.
• Tietokantafunktioihin voisi lisätä try: except: https://docs.python.org/3/tutorial/errors.html (kohta 8.3)
• Tietokantatauluja on hyvä määrä, mutta niiden käsittely jää hieman pinnalliseksi. Jos koodin mukaan saisi mahdollisesti muutaman ''LEFT JOIN'' toiminnon niin tämä olisi hyvä!
• Kirjautuminen ja rekisteröinti koodattu hyvin, mutta tähän voisi lisätä vielä enemmän validointia. Kokeilin rekisteröityä käyttäjänimellä u ja salasanalla u ja antoi minun rekisteröityä ja kirjautua. Koodin sekaan voisi lisätä salasanan ja käyttäjätunnuksen minimi pituuden esim. ''if len(password) < 7'' ...jne.

Muuta

• Commitit sopivan kokoisia ja kuvaavia.
• Dokumentaatio on siisti
• Kaikki näyttää olevan viimeistelyä varten valmista, hyvä!