vertaisarviointi 2

[Owlaboy]

Ohjelma näyttää todella hyvältä ja laajalta.

Yleisesti kun puhutaan kirjoista niihin viitataan niiden ISBN luvuilla eikä ainoastaan niiden nimien tai kirjailijoiden nimien kautta. Joten tämä olisi varmaankin hyvä lisäys kirjojen etsimiseen.

Huomauttaisin myös siitä että joillain error sivuilla ei ole samaa css ulkoasua käytössä, joten ne näyttävät omituisilta muihin sivuihin verrattuna.

Koodinlaadusta sen verran sanoisin että olisi ehkä kannattavaa refaktoroida koodia sen verran ettei kaikki koodin palat olisivat samassa tiedostossa. Olisi myös kätevää eristää tietokanta haut omaan tiedostoon ja tehdä niistä omia funktioita, koska tällä hetkellä on paljon koodi toistoa. Haku: "SELECT username FROM users WHERE username = :username" löytyi ainakin kolmeen kertaan eri paikoissa routes.py tiedostossa. Eristäminen myös lyhentäisi ja selventäisi routes.py tiedostoa.

Viimeisenä haluaisin vielä sanoa pienen asia README:sta. Asennus ohjeissa olisi mukavaa jos .env olisi muutettu "koodi" tekstiksi .env. Samoin myös teksti joka on tarkoitus laittaa .env tiedoston sisälle.

Projekti näyttää kuitenkin todella hyvältä ja oli kiva tutustua toisenlaiseen projektiin. 👍

[irismayigyu]

Kiitos kommentistasi, otan pointtejasi huomioon :)