Vertaisarviointia 16.11.2020 #1
Description
Moikka! Sovellus vaikuttaa hieman suppealta ja tietenkin tässä vaiheessa on vielä selvästi kesken. Yleisesti voit vielä tässä vaiheessa miettiä että olisiko järkevää keksiä jokin lisätoiminnallisuus, esimerkiksi käyttäjäryhmiä tms. Laitan tähän listaa selkeimmistä parannusta vaativista kohteista.
- pitkät rivit koodissa kannattaa pilkkoa useampaan riviin
- camelCaset ei tainnut kuulua pythonin ohjelmointiin? lisäksi oli eri tyylejä sekaisin
- koko sovellus on yhdessä tiedostossa ja mm. metodit olivat kirjoitettu sekaisin. koodi vaatii vielä paljon refaktorointia
- koodia pitää kommentoida muutenkin kuin satunnaisesti
- viestiketjun luonnissa voisi olla samassa lomakkeessa ensimmäisen viestin luonti
- testausta varten olisi hyvä olla valmiiksi luodut testikäyttäjät, jotta pääsisi kokeilemaan myös admin-tunnuksia ja testaaminen olisi helppoa ja suoraviivaista
- html-tiedostossa oli otsikko
<p>-tagin sisällä, sivun rakenne tulisi ottaa huomioon jotta eri selaimet näyttää sivun järkevästi - ulkoasu on myös tärkeä! itseäni motivoi se että sivut ovat nettisivun näköiset, toiminnallisuuksia on helppo rakennella valmiiseen pohjaan kun niitä pääsee heti testaamaan.
- ”allowed users”-käyttäjien syöttäminen yksityisen keskustelun luontikaaviossa on monimutkainen. kannattaa miettiä yleisestikin ratkaisuja, joiden käyttämiseen ei tarvitse kirjallisia ohjeita
Paljon on vielä tekemistä ja alku on aina vaikeinta, olet kuitenkin saanut toimivan pohjan aikaan ja joitain toiminnallisuuksia. Tsemppiä rakenteluun! :)