Projekti ladattiin 21:13:57 4. syyskuuta 2019
Vertaisarvio
Käyn läpi aluksi mitä tein ja sitten paneudun tarkemmin yksityiskohtiin.
Projektin tarkastelu
Aloitin lukemalla dokumentaation ja toteamalla, että asettamasi tavoitteet olet myös saavuttanut. Molemmaissa user storyissa kuvatut käyttötapaukset toteutuvat.
Siirryin asennusohjeeseen ja asensin projektin. Hetken erilaisilla syötteillä pelleiltyäni totesin että kaikki toimii niin kuin pitääkin.
Siirryin sitten koodin pariin nähdäkseni, mitä "konepellin alta" oikein löytyy.
Koodin laatu
Kävin läpi koko projektirakenteen. Olit kommentoinut init.py tiedoston toimintoja, muttet mitään muita tiedostoja. Kuitenkin muuttujien ja luokkien nimentä oli niin suoraviivaista että oli selkeää mitä mikäkin tekee.
En huomannut koodistasi mitään epäjohdonmukaisuuksia tai bugialttiutta. Pidin tavasta, jolla olit käsitellyt esimerkiksi sen, jos käyttäjä on poistunut mutta joku tarkastelee käyttäjän tekemää tapahtumaa. Olisit voinut ihan vain haitanteon minimoimiseksi laittaa tapahtuman osallistujamäärän vain positiivisiin kokonaislukuihin rajoitetuksi, mutta tämä on hyvin pientä viilausta.
Olit kasannut layout.html tiedostossa selkeän sivupohjan kaikille sivuillesi, hyvä! Itse en ole saanut templates kansiota näin johdonmukaiseen kuntoon.
Kehitysideat
Ohjelmasi on hyvin simppeli mutta vakaa ja järkevä. Sinulla on paljon hyviä valmiita ratkaisuja, joita enemmän käyttäen voisit saada käyttäjälle mieluisaa tietoa helposti näkyville.
- Esimerkiksi kaikkien käyttäjien listaus / tapahtuman järjestän nimen linkittäminen profiiliin, jotta käyttäjä voisi kootusti nähdä yhden käyttäjän kaikki tapahtumat.
- Aika rapsakat rajat merkeille syötteessä. 4935 merkkiä ei räjäyttänyt tapahtuman kuvaus-kenttää ja saman veti sisäänsä otsikko-kenttäkin. Tämä on lähinnä huvittavaa, varsinkin kun tapahtuman omalla sivulla fontin koko on tasoa hevonen.
- Filtteröinti/hakutoiminnallisuus tapahtumien luonteen/paikan/sisällön/yms. mukaan.
Lopuksi
Ohjelmasi on huolella tehty ja tarkoituksensa täyttävä. Jos aikaa vielä löytyy niin keskittyisin sinuna muutoksiin, jotka tähtäävät siihen, että jos tietokannassa olisi paljon tapahtumia niin niiden selaaminen/hakutulosten rajaaminen olisi vaivatonta.
Hyvää työtä kaiken kaikkiaan! Mukavaa loppukurssia!