Talk:Hubris

XML retsentsioon meeskonnalt Liisa ja Poisid

XML on üldjoontes suhteliselt kena ja lihtsasti mõistetav. XML koodis ei ole üleliigset pahna ja kood on ilusti trepitud. CDATA-s olev informatsioon on hea ja kongreetne. Töö vastab nõuetele. Likes ja dislikes võiks kuidagi teistmoodi lahendatud olla, kuid praeguna on ka okei.

XSD on hea. Koodis on kasutatud unsignedInt-i, mis on antud koodi puhul täiesti sobilik. Jällegi, kood vastab nõuetele ja sellega ei tohiks tekkida probleeme tulevikus.

XSLT koha pealt, kõik mis on, tundub täiesti korralik ja hea. Esimene kuvab ilusti välja kõik mis on xml-is. Teine kuvab kõik raamatud ja kolmas kõik, millel on rohkem kui 100 like-i. Kahjuks ei olnud koodi, mis muudaks xslt xml-iks, kuid pole hullu, kõik muu on väga hää.

Ülesandepüstituses on nõutud kolmel dimensioonil atribuute, mis on enamat kui 'id'. Antud töös leidub atribuute ainult ühel dimensioonil ja ainult üks atribuut, mis ei ole 'id'. Oleks võinud veel otsida atribuutide jaoks kasutuskohti. Näiteks elemendid 'videourl' ja 'weburl' võiks ühendada üheks elemendiks 'url' ning atribuudiga määrata sellele tüüp. Nii oleks võimalik hiljem veel vajadusel erinevaid url-e lisada. Segadust tekitas ka elemendi 'date' eesmärk: kas tegemist on sisestuse avaldamise või lisamise kuupäevaga (need võivad ju erineda). Lisaks võiks 'date' olla lammutatud kolmeks alamelemendiks vastavalt 'day', 'month' ja 'year', mis hõlbustaks kuupäeva töötlemist. Viimane soovitus on lisada struktuuri veel üks element 'author'. Antud juhul on muusika/raamatu puhul olemas pealkiri ja kirjeldus, aga autorit mitte.

XSD puhul märgiks ära, et advert-elemendi id puhul tuleks kinni pidada CLS(Common Language Specification) nõudest kasutada id väärtuste salvestamiseks integer tüüpi.Kuupäeva hoitakse hetkel string-tüübina. Võib-olla oleks sobivam kasutada date-andmetüüpi või meie soovituse järgi kuupäev tükkideks jagada ning kasutada unsignedShorti aasta puhul ning byte-tüüpi kuu ning päeva puhul.

XSLT transformatsioonid töötavad hästi. Väljund HTML-kood võiks samuti valideeruda (hetkel 1 viga - element 'head' puudumine).

Kokkuvõttes on töö korralik, väikeste puudujääkidega.Soovime teile edasiseks palju jõudu ja edu!

Tegi meeskond VariableMoods

Kuupäev: 19.märts 2015