Difference between revisions of "Talk:Meeskond "H2""

From ICO wiki
Line 74: Line 74:
  
 
'''Martin Lensment'''
 
'''Martin Lensment'''
 +
------
 +
Tuleb nõustuda eelnevate kommentaaridega, et ei sobi selline "puu" lahendus. Kõik teenuse võimalused olid välja toodud ja töötas nii nagu ette nähtud oli, suuremaid vigu leida ei õnnestunud. Samuti jäi silma, et kohviku nimele klikkides võiks avaneda vastava kohviku menüü. Kokkuvõtvalt võiks öelda, et nõuded on täidetud ning mingisuhune töötav variant olemas.

Revision as of 12:31, 6 June 2011

Retsensioon

Meeskonna H2 XML faili retsensioon


Selle meeskonna esindajad otsustasid teha kohviku süsteemi tarvis XML faili.

Peale faili uurimist saan aru et selline süsteem töötab väga hästi üksiku kohviku puhul. Kui soovida suurendada kohvikute arvu: st. lisada programmile ka juurde veel teised ümbruskonna sööklad/kohvikud siis tuleks seda veel natukene kohendada. Muidu on ülesanne korralikult täidetud, ning esijalgne eesmärk kindlasti täidetud.

D-22 Karl Männil


Retsensioon

Nagu eespool mainitud, siis toetab see XML ainult ühe kohviku menüü lisamist. Samas kui teil on plaan ainult ITK kohviku jaoks infosüsteem teha, siis see takistuseks ei oleks. Paraku aga hakkas mulle silma, et antud XML-i ei ole võimalik salvestada täis/pool portsude hindasid, mis ITK kohviku puhul on kindlasti vajalik. Või mõtlete nii teha, et suure ja väikese portsu korral lisate kaks korda sama nimega toitu, millel on lihtsalt erinevad hinnad? Selle-eest aga teie XSLT on vägagi korralik ja suudab ilusa tabelina andmeid kuvada.

Siim Aaver, D21

Klientrakenduse arvustus

Esimese asjana jäi silma disaini keerukus. Isiklikult "puu " arhidektuur ei meeldinud. Palju lihtsam oleks kui oleks iga kohviku nimi, ja siis nimele peale klõpsates avaneks kõik selle alla kuuluv. Näiteks vajutad kohviku nimele, ja siis kuvab kohviku menüü. Menüüs toidule või joogile klikkides saab selle info ja näeb ka arvamusi. Kohviku nimele vajutades peaks saama näha ka kohviku arvamusi. Jäi häirima ka kommentaaride ja arvamuste lisamine (just see et ta koguaeg ekraanil on). Muidu tundus rakendus töötavat ja toimivat. Jäi mulje et rakendus on üksi tehtud, seega arusaadavad ka mõned apsud. Tubli töö siiski!

Jaak Lehtsalu, D22

Veebiteenus

Veebiteenus oli väga hästi ülesehitatud. Meetodite nimed olid selged (teeb seda mis meetodi nimi ütleb). Mõndades kohtades oli id tüübiks string ning meetodite nimede järgi vaadates tundub et tuleb otsida kohvikute nimede järgi, kuigi palju mugavam oleks otsida id järgi.

Jarko Kaskmaa, D21

Klientrakendus

Puu ülesehitus oli väga keeruline. Ka oli peale sisse logimist kommentaaride ja muude sissekannete lisamine väga keerukas (keris kogu aeg lehega kaasa mis oli väga halb). Nõustun eelmise arvustusega, et oleks võinud teha menüü, kus on kõik kohvikud ning kohviku nimele klikkides tuleks ette kõik selle kohviku juurde kuuluv. Siiski olid kõik teenuse võimalused välja toodud ja kõik töötas nii nagu peab.

Jarko Kaskmaa, D21


Veebiteenus

Väga põhjalikult tehtud teenus. Ajakokkuhoiu kui ka närvide säästmiseks oleks võinud M$ andmebaasi kasutamisel abiks võtta ka Entity Frameworki võimalusi - ei peaks ise nii palju boilerplate koodi kirjutama. Kuna otsustati native queryide kasuks jäi silma ka see, et ühes kohas kasutati placeholdereid ja teises mitte (public string Registreeru(.. ). Siis ei peaks rakenduskihis ise kasutajasisendit injectimise vastu kontrollima või seda tegema käsitsi teenuskihis. Koodistiilist veel niipalju, et sõnede liitmisel võiks kasutada stringbuilderit:

           foreach (JoogiLiik x in olemasolevadLiigid)
               s += x.nimetus;

puhul luuakse iga liitmisega uus objekt, mis oleks võrdeline

           var s = "sisukas sõne";
           foreach (JoogiLiik x in olemasolevadLiigid)
           s = new String(s + x.nimetus);

ja seda iga tsükli puhul. Teenuse meetodid olid hästi kommenteeritud ja korralike nimedega, kordagi ei tekkinud kahtlust "kas see teeb ikka seda".

Joosep Ilves, D21



Arvustus veebiteenuse klientidele

Üldiselt rakendus on tehtud arusaadav. Kirjeldatud funktsioonid on olemas, tähtsad toimingud on ka tehtud. Disain on, kliendi rakendus on arusaadav ja korralikult läbimõeldud. Team tegi wikileht ka korralikult.

Jana Mrots, D22

Klientrakenduse arvustus

Klientrakendus kasutas kogu veebiteenuse funktsionaalsust ära, kuid rakenduse kitsaskohaks jäi selle ülesehitus ja disain. Nagu juba eelnevalt mitmel korral mainitud, on puu vaade väga keeruline. Tõesti võiks olla lihtsalt kohvikute list ja kui listis kohvik valida, siis näeks kogu kohvikuga seonduvat infot. Mulle meeldis, et arvamuste lisamisel oli kasutatud AJAXit. Seega ei toimunud lehekülje värskendamist ja lehekülg ei kerinud ennast ülaserva tagasi.

Martin Lensment


Tuleb nõustuda eelnevate kommentaaridega, et ei sobi selline "puu" lahendus. Kõik teenuse võimalused olid välja toodud ja töötas nii nagu ette nähtud oli, suuremaid vigu leida ei õnnestunud. Samuti jäi silma, et kohviku nimele klikkides võiks avaneda vastava kohviku menüü. Kokkuvõtvalt võiks öelda, et nõuded on täidetud ning mingisuhune töötav variant olemas.