Talk:KKMK

From ICO wiki
Jump to navigationJump to search

Retsensioon meeskonna "KKMK" XML ülesande kohta

Meeskond KKMK on teinud XMLi genereeruvaks C# koodi kaudu, mis on väga lahe kuna tiim on ilmselgelt vaeva näinud. Sellegipoolest tekib küsimus, et miks on vaja sellist asja luua, sest ilmselgelt antud ülesande juures oleks käsitsi selle XMLi tegemine tunduvalt kiirem olnud. Üldiselt on C# kood siiski selge ja arusaadav. Koodi poolt genereeritud XML fail on korrektne ja täidab ka antud ülesande nõudmisi. Stiilifail 1 on hästi tehtud, see väljastab korrektse HTMLi ja lisab sinna ka stiilid. Muidu inline stiilid ei ole küll viisakad, aga sellega demonstreeris tegija oma CSS oskusi, mis on alati teretulnud (näiteks antud ülesandes polnud neid isegi nõutud) – tulemuseks on ilus HTML leht. Stiilifail 2 ja 3 annavad vastuse XML kujul ja ülesanne on täidetud, kuid jällegi ei saa aru miks ei oleks võinud vastused olla lihtsalt HTML kujul nagu esimese stiilifaili puhul. XML vastab nii automaatselt kui ka manuaalselt genereeritud skeemifailile ehk ülesanne täidab oma eesmärki. Skeemifaili juures kiidaks meeskonda selle puhul, et nad on lisaks automaatselt genereeritud skeemifailile teinud ka ise skeemifaili. Kokkuvõtteks võib öelda, et meeskond on oma ülesannetega küll väga hästi hakkama saanud, kuid samas leiame, et nad on teinud palju üleliigset ja ebavajalikku tööd.

Meeskond: "BitByBit"

Meeskond „V“ retsensioon meeskond „KKMK“ teenuse kohta – 4.06.2013

Meeskond "V"

Meeskond „V“ retsensioon meeskond „KKMK“ klientrakenduse kohta – 4.06.2013

Meeskond on loonud klientrakenduse enda pakutavale Forumite teenusele ning teostanud selle ASP.NET MVC4 baasil. Esmasel klientrakenduse käivitamisel näeme tühja lehte ilma ühegi navigeerimisvõimaluseta, kuid pärast pilku meeskonna wiki kasutusjuhendisse saame teada, et esmasel käivitusel tuleb navigeerida „setup“ lehele. Setup lehele andmete sisestamine tulemus on aga erinevad veateated kuni lõpuks ilmub „Epic Fail“ veateade. Iseenesest on veahalduse tegemine hea asi, kuid esimsele korral saadud veateada „Error when creating the membership“ ei olnud minu hinnangul piisavalt informatiivne. Esmase foorumi lisamine õnnestus siiski lõpuks pärast teenuse poolelt veateadete uurimist, millest tuli välja, et minu sisestatud parool ei vasta nõuetele. Antud klientrakenduses oleks võinud sel kohal olla ka sisendite valideerimine.

Kui foorum edukalt loodud, siis edasine kategooriate, teemade ja postituste lisamine tundus edukalt töötavat. Administraatorina on võimalik mul ka kasutajaid hallata, mis sisaldab nende pehmet kustutamist (kustutamiskuupäeva sättimine) ja profiilide muutmist, uusi kasutajaid ei saa administraator luua. Kasutajate kustutamise järel on võimalik neid ka taastada, kuid katsetamisel selgus, et selleks pole vajadust, sest kasutajaid saavad ka kustutatud olekus sisse logida ja rõõmsalt postitusi teha.(See viga on ühe autori andmetel nüüdseks parandatud 04.06.2013 - 20:50) Kustutamisel aga blokeeritakse kasutaja redigeerimine, seega selle üritamine tagastab „kollase surmaekraani“ koos huvitava informatsiooniga.

Olles ühe foorumi juba infoga täitnud, selle sulgenud ja uue loonud, satutakse olukorda, kus eelmisse foorumisse enam ei pääse ja uude samuti mitte. Olles aadressiribale kustutatud foorumi nime kirjutanud, siis pääseb sellele taas ligi, kuid uuele foorumile ei tundu olevat mingit võimalus administraatori ligipääsu luua. Samas positiivselt küljelt niipalju, et administraatori osa tundub olevat korralikult turvatud, erinevalt kasutajate poolest. Näiteks postitusi saab teha, redigeerida ja kustutatada igaüks, kusjuures igaühe postitusi.

Rakendus vajaks küll veel testimist, kuid esmamulje hindaks siiski heaks. Rakenduse koodi poolelt on teretulnud nähtus, et vaadetesse antakse info ette vaatemudelite kaudu, mitte niinimetatud vaatekotis (viewbag). Mõnes kohas koodis tekkis esmapilgul küsimusi, et milleks just nii ning mida see koodiosa nüüd teeb. Seega oleks lihtsama arusaamise nimel võinud kasutada kommenteerimist, seda enam, et projekt oli teostatud meeskonna peale. Hea on näha, et kohati on kasutatud ka javascripti, ilma milleta tänapäeval veebilehti tehes tihtipeale ei saa.

Kokkuvõtteks on tegemist siiski asjaliku rakendusega, mis täidab esmased ootused ehk foorumi loomine, haldamine ja sellesse postitamine. Eelnevalt olen toonud välja küll vigu, aga nagu meeskond ise oma wiki lehel mainis, siis arenemisruumi on tõesti. Veelgi enam, et vigu leida ja neid välja tuua on kerge, kuid välisel vaatlusel hinnata inimeste panust (loe: valatud verd ja higi) on keeruline.


Meeskond "V"