Talk:Meeskond "H2"
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
Klientrakendus
Ü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
