Lõptoote retsensioon BUG meeskonnale: Difference between revisions
(Created page with "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õeldu…") |
No edit summary |
||
| (One intermediate revision by the same user not shown) | |||
| Line 1: | Line 1: | ||
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 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: | 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''' | '''Positiivsed''' | ||
Projekti on selgelt analüüsitud ja planeeritud mitte lihtsalt täiesti jooksvalt tehtud | Projekti on selgelt analüüsitud ja planeeritud, mitte lihtsalt täiesti jooksvalt tehtud | ||
Funktsionaalsused on üpris loogiliselt kombineeritud ja toetavad rakenduse eesmärki | Funktsionaalsused on üpris loogiliselt kombineeritud ja toetavad rakenduse eesmärki | ||
| Line 20: | Line 20: | ||
'''Negatiivsed''' | '''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) | Koodis on päris palju väljakommenteeritud koodilõike (mitte iseenesest väga paha, aga teeb lugemise raskemaks ja ei loeta heaks tavaks) | ||
| Line 25: | Line 27: | ||
HTMLi eksportimisel ei konverteerita kasutajasisendeid (ei ole antud hetkel pea üldse oluline, sest tegemist pole veebirakendusega) | 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) | |||
Latest revision as of 19:55, 11 January 2014
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)
