Talk:Liisa ja Poisid

From ICO wiki
Jump to navigationJump to search

Retsensioon meeskonnalt Lupardid

Struktuur

XML vastab nõuetele: sügavust on üle miinimumi ning atribuudid on olemas. Nii XML, XSD eraldi kui ka XML XSD põhjal valideeruvad. CDATA kasutust on üleliigselt, eriti kohtades kus XSD määrab ära selle, et elemendi väärtus on arvu kujul. Miks on aasta, kuu ja päev eraldi elemendid mitte atribuudina? Arvutid pole nii aeglased, et teatud kuu kanded kõikidest kannetest üles otsida. Lisaks pole "aasta" nimeline element semantiline (nimi ei kirjelda sisu). Antud lähenemine pole vale aga pole ka mõistlik. Kuna minimaalne sügavus jääb siiski täidetuks tunduks mõistlikum kui oleks järgnevalt:

 <practices>
   <practice startDate="2015-03-14T17:09" endDate="2015-03-14T18:55" weight="85">
       <type><![CDATA[Zumba]]></type>
       <lostCalories>1980</lostCalories>
   </practice>
 </practices>

Skeemifailis on defineeritud täisarvu ja kuupäeva tüübid ning märgitud ka esinemiste arv kommentaari elemendil, see on hea.

Transformatsioonid

HTML'i genereerimine töötab, andmed näidatakse välja. Küll aga on HTML'i enda struktuur väga kehv ning ei valideeru, see polnud vist küll ülesandes vajalik aga et te teaksite. Listide sisse ei panda päiseid ega suuri hulki tekste, lisaks ilma põhjuseta ei pea kõiki elemente ümbritsema teiste elementidega (antud juhul paragrahvi elementidega). XML transformatsioon on võib-olla liiga lihtne aga kuna selle kohta mingeid nõudeid ei olnud siis ütleks, et nutikas lähenemine. Kõik muu on hästi.