Code review 1

[mikaelri]

Toiminnot, jotka sovellukseen on implementoitu toimivat hyvin, esimerkiksi kirjautuminen on estetty, jos käyttäjä ei ole luonut tunnuksia. Lisäksi kun koittaa lisätä saman käyttäjän kuin jo on luotu on se estetty. Käyttäjä pystyy lisäämään tekstejä ja antamaan näille arvioita ja arvioista muodostuu keskiarvo. Arviot myös pysyvät tietokannassa kun kirjautuu ulos ja takaisin sisälle.

Alla kommentteja toiminnallisuudesta, koodista ja yleisesti, jotka voivat olla jo sinulla työn alla ja/tai tiedossa, mutta listaan ne, jos niistä voisi olla hyötyä projektissa.

Kommentteja toiminnallisuudesta:
1.) Voisiko tunnuksen luomisessa miettiä sitä, että salasana pitäisi asettaa kaksi kertaa, jotta käyttäjä varmasti luo salasanan, minkä on halunnut (eli, että olisi lisäksi lomake “Salasana uudestaan”).
2.) Kun luodaan uusi käyttäjä, niin jos ei haluaisikaan luoda, ei ole mahdollista palata takaisin, eli voisiko tässä olla jokin “palaa takaisin”-mahdollisuus?

Kommentteja koodista ja tietokannasta:
1.) Koodi on selkeää ja helposti luettavaa ja se on pilkottu osiin.
2.) Riippuen kuinka paljon työ tulee muuttumaan, niin voisiko ajatella, että app.py jaettaisiin vielä erikseen esim. app.py, routes.py, db.py ja tarvittaessa jotain muita. Tämä on varmasti makuasia ja riippuu siitä, kuinka paljon ja monimutkaista koodista tulee.
3.) Tietokanta vaikuttaa selkeältä tässä kohtaa projektia, mutta voi miettiä haluaako siirtää sen toiseen tiedostoon (liittyen kommenttiin 2.) yllä).

Yleisenä kommenttina myös projektin testausta varten voisi lisätä tarkemmat ohjeet readme-tiedostossa, mikä helpottaa testaajan työtä. Lisäksi tähän samaan liittyen oma näkemys on, että olisi hyvä lisätä schema.sql ja requirements.txt -tiedostot, jotta testaaja saa suoraan oikeat taulut ja riippuvuudet tietoon. Nämäkin saattavat olla jo sinulla tiedossa ja työn alla.

Toivottavasti kommenteista on hyötyä ja tsemppiä projektiin!