Talk:Meeskond ".NOT"

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 töö hulgast, mis selles tehti. Ise arvan, et töö võiks olla eeskujuks teistele, kaasaarvatud minu enda meeskonnale.