Talk:Meeskond Vargamae

From ICO wiki
Jump to navigationJump to search

XML failide retsensioon meeskonna "Kirves" poolt

Meeskond Vargamäe on oma töö teinud elektroonikat müüva veebipoe Euronics eeskujul. Xml failis on olemas nõutav arv dimensioone ning vähemalt kolmel dimensioonil eksisteerivad ID-st erinevad atribuudid. Lisaks toodete nimekirjale on .xml failis kuvatud ka kasutajate kommentaarid. Kirjelduste ja kommentaaride juures on kasutatud ka CDATA tage. Samas on elemendi “toode” atribuudil “nimi” kasutatud jutumärki ekraani tollide tähistusena, mis paraku muudab atribuudi vigaseks. Seda oli märgata mitmes kohas. Näiteks: nimi="32" Full HD LED LCD-teler, Samsung / Smart TV".

Nõutud kahe .xsl faili asemel on meeskond teinud tervelt kolm. Neist kahe tulemuseks on .html failid; vastavalt üldise kaupade nimekirja ning teisel juhul detailsete kirjelduste, piltide ja kommentaaridega ning kolmandal esialgsest failist erineva ülesehitusega uus .xml fail. Lisaks tavapärasele elementide ja atribuutide väärtuste kuvamisele on kasutatud ka for-each tsükleid ja vahemuutujaid.

Kuna programmeerimine on inglise keeles, võiks jätkata sama keele kasutamist ehk siis kasutada inglisekeelseid parameetrite ja muutujate nimesid. See muudab koodi terviklikumaks ja kergemini loetavaks.

Kolmandas .xsl failis on kasutatud copy-of funktsiooni - seda on kül mugav kasutada, aga see on samas ka resurssinõudev.

Kui jätta välja probleem jutumärkidega atribuudi väärtuses, mille näol on arvatavasti tegemist lihtsalt väikse kahe silma vahele jäänud hooletusveaga, vastab esitatud kodutöö täielikult seatud kriteeriumitele. XML fail, skeemifail ja transformatsioonifailid on loogilise ülesehitusega ja kood korralikult trepitud.


XML failide retsensioon meeskonna "Udud" poolt

XML fail vastab nõuetele, ehk sellel on rohkem kui neli loogilist dimensiooni ja vähemalt kolmel dimensioonil kasutatakse atribuute, mis on enamat, kui lihtsalt ID. XML on loogiliselt üles ehitatud.

Esimene asi, mis XML-ist silma torkab on dimensiooni <toode> atribuut nimi="32" Full HD LED LCD-teler, Samsung / Smart TV". Siin on kahjuks kahe silma vahele jäänud jutumärkide kasutamine selles nimes. 32“ asemel võiks näiteks kirjutada 32 inch, et saaks väljastada õige toote nime. Samas võiks faili paremini loetavuse pärast ka panna elemendi <saadavus> elemendi <toode> atribuudiks. Näiteks: <toode id="1" saadavus=“120“ hind="399" nimi="32" Full HD LED LCD-teler, Samsung / Smart TV">. Võib-olla võiks ka mõned kommentaarid lisada. Positiivse asjana võiks välja tuua, et XML failis on kasutatud ka CDATA tag’e.

XML faili skeemifailist tuleb ka välja, et osade atribuutide, mis ei saa kunagi negatiivsed olla (näiteks id ja saadavus) tüübiks on int. Selle asemel võiks kasutada tüüpi unsigned int.

XSL faile on tehtud piisavalt (kokku kolm). Tehtud XSL failide kohta ei saa nuriseda, nad on korralikult koostatud.

Kokkuvõttes võib öelda, et kõik tehtud failid olid päris hästi tehtud, ainult XML failis olid mõned väikesed vead.