Talk:Meeskond ".NOT"

Retsensioon XMLile (Meeskond "Hello Kitty")

Kiidusõnad meeskonnale. XML'i struktuuri oskavad korralikult kokku panna ning stiilifail ja skeemifail kah korralikult üles ehitatud.

Kui nüüd pisut nokkida ka, siis silma hakkas mõningane elementide ja atribuutide ebakooskõla. Juba eelnevalt välja toodud märkus, et miks on mootori maht lisatud mootorile atribuudina, aga muud samatähenduslikud väärtused elementidena. Isiklikult oleksin rohkem atribuutide kasutust oodanud.

Antud xml'i kohta on välist infot väga vähe. Kogu sisu on ära kirjeldatud ühe lausega. Lugejal oleks kindlasti suurem huvi xmli struktuuri läbi sirvida, kui oleks teema kohta parem tagataust teada.

Xslt ja xsd kohta pole rohkem samuti midagi mainida, mis tähendab, et ülesehitus lihtne ja korrektne. Nii nagu peabki. Struktuuri kommenteerimine oleks ka suureks plussiks olnud. Siinkohal viide meeskond Haroldile, kelle kommenteeritud koodi oli lausa lust lugeda.

Retsensioon (Meeskond "Harold")

Üldiselt eeskujulik sooritus, on näha, et meeskond tunneb XML ja XSL failide struktuuri ja oskab neid koostada nii praktikas kui disainis (mõelda, kuidas XML struktuurselt üles ehitada).

Vormilisest küljest

XML-i oli wikist ebamugav lugeda. Palun kasutage <pre> tag asemel <source lang="xml">. XML failile oleks kohe juurde võinud lisada viite transformatsioonile (<?xml-stylesheet type="text/xsl" href="stiil.xsl"?>), samuti oleks võinud XML + XSL failid kuhugile hostida, et retsensent ei peaks seda ise tegema. P.S! < märke, mis transformatsiooni autorile probleeme tekitas, saab sisestada <source> tagide vahele või kodeeritult.

XML

XML tutvustust oleks tekstiliselt tahtnud pikemalt lugeda: miks valiti just see teema (aktuaalne?), mida toodud XML struktuur võimaldab. Struktuuriliselt on XML üles ehitatud küllaltki hästi - võimalik on lisada 0 ... n sõidukit ning kirjeldada nende omadusi, elementidel on id kiireks otsimiseks.

Omaduste kirjeldamist oleks muidugi üldisemalt saanud teha - ma oleksin ise tõenäoliselt kohelnud enamus asju (sh mootor, kere) auto komponentidena, millel on komponendi ID. See võimaldaks vaadelda autot kui mingit pakki erinevatest komponentidest ning näiteks e-poes (või mõnes muus infosüsteemis) kõik komponendid klikatavaks teha. Ehk siis, inimene näeb nimekirja autodest (auto kirjeldus on hunnik meta-andmeid nagu XML-s toodud mark, tootmisaasta, hind) ning klikates lahti mõne auto, näeb nimekirja komponentidest, mis on omakorda klikatavad, et nende infot vaadata. Varustuselemendid siis samamoodi - ka neid võib vaadelda komponentidena (või miks mitte, pakettidena).

Transformatsioon

Transformatsioonid on hästi tehtud, HTMLs kuvatakse kogu vajalik info kenasti nimekirjadena välja. Siiski, natuke CSS-i ei teeks paha loetavuse mõttes.

Retsensioon (Meeskond LLL)

Üldiselt on meeskond teinud head tööd ja üpriski hästi hakkama saanud. Kuigi oleks võinud kodutööd kirjeldada rohkem. Miks just selline teema valiti, mida võimaldab jne?

Xmli tutvustus oleks võinud veidi pikem ja põhjalikum olla. Xmli oli veidi ebamugav lugeda, kuigi trepitud oli õigesti. Ülesehituselt on xml hästi ülesse ehitatud. Mootori maht oli lisatud atribuudina, aga kütus ja võimsus elemendina. Samas oleks ilusam olnud, kui need oleks kõik kirjutatud atribuudi või elemendina. Hea on see, et on jagatud tehnilised andmed ja varustus eraldi, mis teeb lugemist ja käsitlemist lihtsamaks.

Transformatsioon on muidu üsna korralikult tehtud, kuigi oleks võinud veidike rohkem tähelepanu panna css’ile. Kuigi esimeses XSLT's on paar treppimise viga, aga teistel need puuduvad.

Retsentsioon teenusele (Peeter_VR2)

Projektiga on kaasas SQL skriptid mille põhjal sai endale andmebaas loodud. Kahjuks ei tulnud kummagi skriptiga kaasa andmeid. Seega teenust testida otseselt ei saanud. Teenust käivitades läks teenus ilusasti tööle ja sai kenasti funktsioonide vahel valida. Loomulikult tuli nullpointer exception, kui miskit invokesin – tabelid tühjad. See ei seganud edasist koodi poolset vaatamist. Teenuse ülesehitust vaadates, siis teenus on väga hästi ja läbimõeldult tehtud. Funktsionaalsus, mis on teenusele juurde pandud on hulgaliselt. Kogu funktsionaalne pool on ära ka kirjeldatud Wiki lehel, kust sai alguses hea ülevaate teenusest.

Backendi ja SQLi ühendamine on tehtud LINQ abiga. Sealjuures on igast tabelist tehtud eraldi klass, et objektidega oleks targem ja lihtsam ümber käia. Väga hea lahendus. Samuti osadele klassidele on ehitatud Utils klassid, kus on lisafunktsionaalsus juures. Kenasti on ära kasutatud objekt-orienteeritud lähenemist. Vajalike funktsioonide, mida hiljem teenus välja kutsub, kirjeldamine on igas klassis tehtud ja teenuse all on ainult õiged objektid välja kutsutud. Tegemist siis peamiselt static meetodidega.

Silma häirib, et koodi juures puudub igasugune dokumentatsioon. Meetode pole kommenteeritud. Võiks olla, saaks kiiremini ülevaate, mida mingi meetod teeb või miks ta just nii kirjeldatud on. Vaatamata sellele on kood loetav ja ja pärast mõningat uurimist ka arusaadav, mida miski teeb. Tuleviku mõttes võiks olla kommentaarid juures, kui projekt peaks kasvama.

Vaeva on nähtud lisatööga - teenusele on juurde ehitatud ka turva-sertifikaadid. Nagu näha on need teenuse juures kasutusel - ServiceSecurityContext Projekti on juurde ehitatud, lisatud ka konsoolirakendus, mis oli mõeldud teenuse testimiseks. Ise kasutada seda ei saanud kuna andmebaas oli tühi.

Retsensioon teenusele (Meeskond LLL)

Teenus analüüsi on teostatud põhjalikult ja kõiki funktsionaalsusi, mis võiksid olemas olla on ka realiseeritud. Töö lugemine ei ole rakse ja mulle isiklikult meeldib, et töö pole liialt suureks ja kirjuks paisutatud igasuguste pikkade kommentaaride ja kirjelustega, aga töös võiks natukene rohkem lühikesi kirjeldusi olla. Töö on ise hästi tehtud.

Retsensioon kliendile (Meeskond LLL)

Klient on väga hästi tehtud. Kõiki teenuse funktsionaalsusi saab kasutada ja eriti meeldiv asjaolu on sertifikaatide olemasolu ja nende kasutus. Sertifikaatide tööle saamiseks võis minna palju aega ja vaeva, mis on juba ise suur näitaja kirjeldamaks, kui hästi ja põhjalikult tööd tehti. Ise arvan, et töö võiks olla eeskujuks teistele, kaasaarvatud minu enda meeskonnale.

Retsensioon kliendile (Meeskond "PhoneBook")

Kahjuks jäi testimise pool ära, sest mingil teadmatul põhjusel kaasasolevad scriptid ei teinud andmebaasi, vaid tegid tabelid ja andsid erroreid, sest need tabelid lisati süsteemi tabelite alla. Võimalik, et viga on minu enda arvutis, sest teiste kommentaaride järgi võib eeldada, et nad on seda ka testinud. Seetõttu on kommentaarid ainult koodi ja esitluse põhjal!

Veebiteenus

Ilmselgelt on ära tehtud väga suur töö, mida peaksid enamus gruppe eeskujuks võtma. Väga hea on see, et igal pool on kasutatud objekte ja pole mingi puder. Paraku pole koodi kommenteeritud, kuid enamjaolt võib asjast aru saada siis, kui vaadata meetodi nime või uurida sisu. Samas jääb näiteks täiesti arusaamatuks see, et mis mõte sellel ActionMessagel on - mõnede klasside järgi võiks pakkuda, et vigade kogumine? Mõnes mõttes on arusaamatu ka see, et miks on kõik auto kohta käivad väljad kohustuslikud ning vähemalt auto võimsuse puhul puudub võimalus lisada "+" märk, kui on autol midagi muudetud ja pole täpselt teada võimsus. Hetkel on eeldus auto pass, mis ei pruugi kogu olulist infot anda. Lisadena võiks olla näiteks kommentaari väli kuhu saaks mingeid lisasid märkida kas auto ajaloo kohta või muude oluliste detailide jaoks. Samuti võiks olla pildi lisamise ja vaatamise võimalus, sest nagu öeldakse: "pilt ütleb rohkem kui 1000 sõna." Üks vähestest lahendustest, kus on kasutatud ka sertifikaate. Kõik etteantud nõuded on ka täidetud - teenus toimib, kasutajate tuvastamine ja haldus on olemas ning ka statistika toimib.

Klientrakendus

Korralik ja toimiv töö. Väga hea on see, et kasutatud on WPF-i mitte mõnda muud lahendust ning klientrakendus on loodud enda teenusele mitte mingile suvalisele asjale. Positiivne on ka see, et kommentaare on igale poole tekkinud, kuid kohati üsna mõttetuid, mis ei anna mingit lisainfot ja ei tee asja üldse arusaadavamaks. Disain on lihtne ja arusaadav. Küsitav on muidugi see registreerimise osa lingina, kuid võib olla tõesti isegi õigem kui nupp.

Retsensioon veebiteenusele ja klientrakendusele meeskonnalt Hello Kitty

Klientrakendus

Kliendi tööle saamiseks läks suhteliselt kaua aega. Siis aga töötas nii nagu vaja ja asi oli korralikult tehtud. Vähemalt selle testimise raames, mis mina tegin. Ei laskunud eriti sügavale, et otsida, kuidas saaks tekitada vigu. Disain oli puhas ja arusaadav.

Kui koodi vaatasin, siis kood oli puhas ja arusaadav. Kuid samas oleks võinud olla kood kommenteeritud. Kui projekt liiga suureks muutub, siis saaksid ka teised aru, kes pole sellega varem tegelenud. Aga muidu paistis asi olevat korralik.

Veebiteenus

Meeldis see, et oli kasutatud dll faili, et teenus oleks rohkem lahku löödyd. Kuid kui võtsid lahti CarService.cs, siis alguses oli seda väga raske lugeda. Oleks võinud olla veidi rohkem struktureeritud. Muidu jäin asjaga rahule. Lugemise tegi lihtsamaks interface klass, kus said otsida kiiresti mingi funktsiooni üles, kui seda oli vaja leida. Hästi tehtud.