Talk:Meeskond "GAP"

From ICO wiki
Revision as of 16:21, 5 June 2011 by Psiimo (talk | contribs)
Jump to navigationJump to search

XML-i arvustus

Väga paljude võimalustega XML (mitmed erinevad keeled, valuutad jne.). Meeldib. Helenil oli hea ettepanek: "Hinna väärtuse all võiks olla näiteks ka summa täisosa ja poolosa (või midagi taolist)". Vaartuse all võiks siis veel olla näiteks täissumma ning peale komakohta lahtrid.

Indrek Kõue D21



XML-i hinnang

XML-i lõpust on puudu <TeenusedKaubad>, võib-olla on kopeerimisel Wikisse lihtsalt maha jäänud. Hinna väärtuse all võiks olla näiteks ka summa täisosa ja poolosa (või midagi taolist), sest kui valuuta on eraldi välja toodud, siis mõnel maal võivad ju hinnas nö kopikad ka olla. Muidu on XML hea lihtne. XSLT oli suhteliselt lihtsasti loetav, mõned probleemsed kohad tekkisid, kuid täpsemini uurides sain aru, mida mõeldud oli. Tore, et oli lisatud ka pilt, kuidas XML koos stiilifailiga välja peaks nägema. Hästi tehtud, kuid on ka parendamisruumi. Wiki lehel võiks olla ka teema kirjeldus või lahtiseletus. Jõudu!

Helen Muidik D21


XML-i arvustus

Kõik kolm hinnatavat asja XML, XML skeem ja Extensible Stylesheet Language Transformations ehk XSLT on olemas.

  • XML struktuur on korrektne, kuid lõpus on tõesti puudu märgend <TeenusedKaubad>. Elementide sisse oleks võinud andmeid ka näiteks panna.
  • XML Schema. Kuna XML-is polnud näiteks andmeid sisestatud, siis ei teadnud Visual Studio mis tüüpi andmeid elemendid peaksid sisaldama.
  • XSLT samuti olemas ja lisatud pilt näitab kuidas lõpptulemus välja paistab.

Ilmar Telga (D21)


Veebiteenuse arvustus

Esimene asi, mis silma jäi oli see, et kood on täielikult kommenteerimata. Iseeneset on kood korralikult ja puhtalt kirjutatud, kuid mõnel juhul jäi arusaamatuks, mida selle koodujupiga on mõeldud või see tegema peaks. Kood on korralikult jaotatud klassidesse ja lihtsustamisse mõttes ka regioonidesse. Wiki lehel on küll antud paari sõnaline seletus meetoditele, kuid sellest ei piisa. Ühesõnaga on teenuse dokumentatsioon puudulik. Üldine mulje koodis on hea, kuid dokumentatsiooni poolt ja kommentaare koodil võiks olla rohkem.

Seda Sahradyan D21


Kopeerimisel maha jäänud tag on nüüdsest tagasi omal kohal. Tänud märkamast! GAP


RE: kommentaarid Olen seisukohal, et selguse huvides ei peaks kommenteerima koodi vaid ainult teenuse meetodeid. Kui tiimis on kokkulepitud ühtne struktuur, kuidas koodi kirjutatakse, siis igasugused liigsed kommentaarid ainult segavad koodi loetavust.

Priit