vesulius / kirja-arvostelu-sovellus Goto Github PK

Ulkoasu on siisti ja sovelluksen käyttö on intuitiivista. Olisi kiva, jos etusivulla näkyisi joku lista ainakin joistain (esim. viimeisimmistä) jätetyistä kirja-arvosteluista. Kirjautumissivulla olisi ehkä parempi, jos ei heitettäisi uudelle sivulle väärää salasanaa syötettäessä. Joitakin vikoja löytyy (esim. haku hajoaa joidenkin merkkien kanssa, tyhjät arvioinnit kaataa sivun jne.) mutta nämä pitäisi olla hyvin korjattavissa tässä reilun viikon aikana.

Koodi ja tietokannan toteutus näyttää järkevältä. Koodi on järjestetty loogisesti eri moduuleihin ja sitä on selkeä seurata. Aika monet kommentit ovat kuitenkin ehkä vähän turhia. Esim. funktiossa login on aika itsestäänselvää mitä se tekee nimen ja koodin perusteella, joten tuollaisia ei mielestäni tarvitsisi erikseen kommentoida. Ei kuitenkaan ole mikään iso ongelma ja ehkä vähän mielipidekysymys. Monet muut kommentit on kuitenkin ihan hyödyllisiä.

Kokonaisuudessaan homma näyttäisi olevan ihan hyvällä mallilla.

Hienosti kommentoitua koodia ja hyvin nimettyjä muuttujia -- helppo lukukokemus!

Pieniä epätarkkuuksia toteutuksessa on siellä täällä. Esimerkiksi mille tahansa kirjalle osoitettu arviointi menee yhdelle tietylle kirjalle. Arvioinnin teksti näkyy oikein, mutta arvosana tulee väärästä kentästä, mikä johtaa sen näkymiseen virheellisesti.

Yhden turvallisuusuhan havaitsin: Etusivua lukuunottamatta mikään sivu tai toiminto ei missään vaiheessa tarkista, onko käyttäjä kirjautunut. Näin ollen kirjautumaton käyttäjä voi mennä suoraan sivulle /display_reviews/[id] ja jättää arvion.

Uudelleenohjauksia kannattaa miettiä. Parilta näkymältä, joihin käyttäjä ohjataan, ei pääse pois ilman selaimen painikkeita. Esimerkiksi haun jälkeen ei pääse takaisin etusivulle, ja mikäli haku ei tuottanut tulosta, on seurauksena tyhjä sivu. Myöskään arvostelun jättämisen jälkeen sivulla ei ole yhtäkään linkkiä -- uudelleenohjaus voisi periaatteessa ohjata takaisin kirjan omalle sivulle.

Kaiken kaikkiaan lupaava projekti ja koodin kommenteista päätellen suurin osa yllämainituista asioista olikin jo tiedossa.