Vertaisarviointi
Zatyri opened this issue · comments
Vertaisarviointi
Käytettävyys
Sovellus toimii hyvin ja käyttöliittymä on selkeä. Jostain syystä eivät rekisteröidy ja kirjaudu linkit näy mobiilissa. Tämä on toki sivuseikka eikä kurssin kannalta oleellista.
Readme:ssa kuvatut toiminnot toimivat niin kuin olettaisi. Olisi kiva jos arviointinäkymässä näkisi minkä arvosanan pelille on itse antanut, jolloin on myös selkeätä käyttäjälle että hän on arvioinut pelin. Tämän voi toki käydä tarkistamassa "omilta" sivuilta mutta aiheuttaa turhaa klikkailua.
Koodi
Koodi ja tiedostorakenne on selkeätä ja helposti luettavissa.
Taulukot ovat myös selkeitä mutta tässä on taulukko ilman nimeä. Tämä korajutuu varmasti kun rooli toiminnot toteutetaan.
SQL kyselyt ovat hyvin toteutettu ja käytetty erilaisia SQL "toimintoja". Mietin pitäisikö "rate_game" funktion kaksi kyselyä eriyttää eikä ajaa molemmat kyselyt aina. Tässä linkki
Käyttäjäsyötettä validoidaan tällä hetkellä vain backendissä. Tämän lisäksi voisi olla hyvä validoida syöte frontendissa html form validoinnin avulla. Näin ei synny turhia kutsuja backendille ja uudelleen "piirtämistä" frontendissa. Mainittakoon että backendista ei kannata ottaa validointia pois
Kokonaisuus
Commitit ovat kuvaavia, readme selkeä ja githubi "siisti". Oli ilo tutustua sovellukseen ja sen rakenteeseen. Jatka hyvää työtä!