Talk:Repi: Difference between revisions
| (One intermediate revision by the same user not shown) | |||
| Line 10: | Line 10: | ||
==XSL== | ==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== | ==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. | |||
Latest revision as of 21:08, 27 March 2017
Ü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.
