Talk:Meeskond ".NOT": Difference between revisions

From ICO wiki
Jump to navigationJump to search
No edit summary
Line 24: Line 24:


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.
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.

Revision as of 18:34, 16 May 2012

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.