Koodikatselmointi

[otahontas]

Projekti ladattu 21.8. klo 22.32

Projektin asentaminen ja perustoiminallisuudet:

• Asennus onnistui mainiosti lokaalisti, requirements oli ok ja sovellusta pääsisi hyvin kehittäjänäkökulmasta edistämään nykytilanteesta eteenpäin.
• CRUD-toiminnot korttien ja pakkojen osalta toimivat oikein hyvin! Lisäksi kirjautuminen ja rekisteröityminen pelittivät ja formseja validoidaan tarvittavissa paikoissa. Käyttäjänä ei pääse lisäämään tyhmiin paikkoihin tietoja, jotka johtaisivat ohjelman kaatumiseen - tästä aina plussaa!

Koodin laatu ja projektin rakenne

• Koodi oli pääosin hyvin luettavaa ja ymmärrettävää: luokat ja metodit olivat selkeillä paikoillaan ja niihin pääsi käsiksi. Toisteisuutta oli jonkin verran ja jonkinlaista abstrahointia olisi syytä tehdä luokkiin. Lisäksi toistat joissain metodeissa samaa koodia, ja näissä paikoissa voisit eriyttää saman koodin omaksi metodikseen, jota kutsua kummastakin metodista.
• Koodin luokat eivät noudattaneet repossa ollutta database-schemaa. Liitostaulujen osalta koodissa ei ollut hirveän selkeää, miten suhteet vaikkapa deckkien ja korttien välillä menevät.
• Useissa metodeissa käytät valmiita SQL-komentoja. Tämä lienee kurssin osalta makukysymys, mutta minusta SQLalchemyn tarjoamat metodit tekevät koodista vähän luettavampaa.

Käytettävyys- ja saavutettavuusnäkökulmia:

• Ohjelmassa ei ollut erityisen paljon ohjeita, eikä peli ole minulle kauhean tuttu, joten oli hankala tietää, mitä mihinkin kenttiin tulee syöttää. Virheilmoitukset eivät kerro, mihin kenttiin tulee syötettyä vääriä tietoja. Oletuksena taitaa toki olla, että käyttäjä tietää, mitä tekee, mutta pientä feedbackia kannattaa antaa - parantaa käytettävyyttä. (ks. esim Flaskin flash-ominaisuudet tähän)
• Onnistuneista toiminnoista ei tule ilmoiksiä (esim. kortin syöttäminen pakkaan, kirjautuminen, rekisteröityminen), enkä aina tiennyt, onnistuiko toiminto. Feedbackin antamiseen myös tämän osalta kannattaisi minusta keskittyä.
• Responsiivisuus: pienikokoisella näytöllä sivut eivät ole kauhean luettavat

User storyt

• Ensimmäisen user storyn osalta toiminallisuudet vaikuttivat olevan kunnossa
• Loppujen user storien osalta toiminnallisuuksissa olisi vielä paljon parantamisen varaa, mutta nämä taitavatkin olla kehityksen alla?

Bugeja / korjausehdotuksia:

• Responsiivisuus: pienillä näytöillä saitit ei oo kauheen responsiiviset tai luettavat
• En saanut pakkaan lisäämistä toimimaan lainkaan, eivätkä kortit ilmestyneet kantaan.
• Templatet olivat monelta osin vielä aika barebones-versioita, vaikka Bootstrap olikin pohjalla. Kannattanee toiminnallisuuksien toteuttamisen jälkeen viilailla vielä ulkoasu käytettävyyttä tukevaan kuntoon.

Kokonaisuutena vaikuttaisit olevan hyvin vauhdissa ja sovelluksesta syntynee hyvä lopullinen tuotos. Tsemppiä loppukurssiin!