Koodikatselmointi

[lex]

Mon Oct 10 19:56:10 EEST 2016

• selvää ja lyhyttä koodia
• hienot värit napeissa
• Stories pitäisi olla Story. Siellä on kuitenkin vain yhden Storyn tiedot sisällä.
• OwnPage ei välttämättä tarvitsisi omaa modeliaan, kun se on kuitenkin Story.
• createdby varcharilla näyttää ongelmalliselta ja siitä tulee ylimääräisiä tavuja. Voisi olla ihan vain user_id references User ja username = User.get(user_id).username if user_id else 'anonymous'. Nykyisellään se estää ainakin käyttäjänimen vaihdon, ellei samalla tee suuria queryjä muiden taulujen päivittämiseen.
• text storyssä mahdollistaa niin suuren tekstin syöttämisen tietokantaan kuin huvittaa.
• passwordia ei välttämättä kannata ottaa autentikoinnissa ulos asti.