2. Koodikatselmointi

[Sihuvi84]

Latasin projektisi: 17.4.2015, kello 08:31

Tässä myöhässä oleva koodikatselmointi:

Koodisi on erittäin selkeää, ja sitä on todella mukava lukea. Työn aihe on mielenkiintoinen - odotan kiinnostuneena, millainen lopputulos onkaan.

Huomasin, että kaikki tekemäsi muutokset eivät ole vielä testattavissa ellei tiedä tarkkoja linkkejä. En jaksanut alkaa testaamaan sellaista, mitä et ollut kirjoittanut readme-tiedostoon. Huomasin myös, että syötteiden validointia ei vielä ollut toteutettu lainkaan, työsi vaikuttaisi olevan aikataulusta myöhässä, minkä tietysti tiedätkin. En kuitenkaan usko, ettetkö saisi työtä ajoissa valmiiksi: aihe, selkeä koodaustyylisi ja dokumentaation alku vakuuttavat minut siitä, että lopputulos on nähtävä.

Oma koodaukseni tämän työn osalta on ollut melko sekavaa, nimeämiskäytäntöni epälooginen ja koodi muutenkin melkoista spagettia. Tutkiessani sinun työtäsi, tulee itsellenikin inspiroitunut olo siivota omat sotkuni ja pyrkiä yhtä selkeään ulosantiin.

Kritiikkini on, että olisin halunnut päästä testaamaan toimintoja jo tässä vaiheessa, vaikka ne eivät mielestäsi olisikaan vielä täydellisiä ja valmiita. Jos olisin asiakas, haluaisin nähdä mahdollisimman varhaisessa vaiheessa, minne päin ollaan menossa. Olet myöhässä aikataulusta, mutta se ei kai ole mitenkään vakava asia.

Muuten kaikki, dokumentaatio mukaan lukien, tuntui olevan kunnossa.