Lõptoote retsensioon BUG meeskonnale

Kõigepealt olgu öeldud, et minu arvates on tegemist parima projektiga kõigi kodutööde hulgast. Seda arvestades nii realiseeritud funktsionaalsusi kui ka üldist läbimõeldust. Kui peaks selguma, et ükski neist inimestest pole arendajana töötanud, siis ma oleksin tõesti üllatunud. Rakenduse olemus on täiesti arusaadav ning funktsionaalsuse osas pole mitte mõeldud ainult esmajärgus vajalike asjade peale vaid ka kasutajamugavuse jms peale, ehk siis on mindud suur samm edasi lihtsast soovist midagi töötavat ära esitada. GUI on küll väga lihtne ja ilma kellade viledeta, kuid suhteliselt intuitiivne ja kuigi alguses ei pruugi kõigi nuppude tähendus peale vaadates selge olla, siis saab kõik väikse katsetamise järel selgeks. Samuti tutvusin peale rakenduse katsetamist ka kasutusjuhendiga ning see oli täiesti piisav ning oleks asjad kiiremini selgeks teinud kui katsetamine. Kokkuvõttes võib öelda, et tehtud töö maht oli kahtlemata üpris suur, isegi arvestades meeskonna suurust. Järgnevalt toongi välja isiklikud positiivsed ja negatiivsed tähelepanekud:

Positiivsed

Projekti on selgelt analüüsitud ja planeeritud, mitte lihtsalt täiesti jooksvalt tehtud

Funktsionaalsused on üpris loogiliselt kombineeritud ja toetavad rakenduse eesmärki

GUI on piisavalt lihtne ja intuitiivne

Kasutusjuhend on täiesti piisav

Kood on hästi struktureeritud, välditud on hiiglaslikke faile, klasse ja funktsioone

Kood on hästi loetav

Ei suutnud tuvastada suuremaid koodikordusi

Negatiivsed

Üksikud võimalikud veakohad, kus 99% juhtudest tagastatakse oodatud vastus, kuid pole arvestatud, et ka vastupidine olukord saab tekkida (nt IsikUtils.cs:101 hakatakse muutma isikut mis on leitud ID järgi, kuid pole kontrollitud kas otsing üldse midagi tagastas. Sama ka AmetinimiUtils.cs:56 jms kohtades), kuid neid kohti pole palju ja nad ei riku rakenduse üldpilti.

Koodis on päris palju väljakommenteeritud koodilõike (mitte iseenesest väga paha, aga teeb lugemise raskemaks ja ei loeta heaks tavaks)

HTMLi eksportimisel ei konverteerita kasutajasisendeid (ei ole antud hetkel pea üldse oluline, sest tegemist pole veebirakendusega)

Release versiooni puudumine lõpptootes (mingil põhjusel eksisteerib ainult Debug)