Talk:Repi

From ICO wiki
Revision as of 21:08, 27 March 2017 by Msavila (talk | contribs) (→‎Kokkuvõte)
(diff) ← Older revision | Latest revision (diff) | Newer revision → (diff)
Jump to navigationJump to search

Ülesande "XML fail" retsensioon meeskonnale Repi meeskonnalt Muru

XML

XML paistab olevat hangete informatsiooni vahendamiseks. Üldjoontes on kenasti tehtud, kuid mõnes kohas (<hankija>) pole kasutatud CDATA-t, kuigi võiks, sest võib esineda hankijaid, kelle nimes on näiteks "&" märk.

XSD

XSD on loogiline ja täidab eesmärki. Väike märkus oleks <hange> atribuudi "aktiivne" teemal - oleks meeldiv, kui oleks kasutatud "boolean" tüüpi (https://www.w3.org/TR/xmlschema-2/#boolean). Hetkel võib tekkida küsimus, et mis saab siis kui atribuudi aktiivne väärtus on "2" või "42" - kas see on siis tõene või mitte? Usun, et selline laiem tõlgendusvõimalus võib klientsüsteemides tekitada segadusi.

XSL

XSLT puhul sooviksin näha visuaalselt tulemust. Koodi järgi tundub huvitav, jällegi märkus ülal mainitud "aktiivne" teemal - siin on aru saada, et kui see on suurem nullist, siis käsitletakse seda tõesena. Kuigi XSLT puhul mingit võitu ei saaks, võiks ikkagi booleanile sobivas kohas kasutada booleani.

Kokkuvõte

Tubli töö. Näha on, et autor on mõelnud asjale piisavalt ja usun, et esimene samm neid XML-e kasutava rakenduse arendamisel sujub kenasti. Väikesed märkused olid seoses andmetüüpidega, seega ütleksin, et väga hea töö ja natuke saaks veel paremini.