Talk:DevHelpVR: Difference between revisions

From ICO wiki
Jump to navigationJump to search
No edit summary
Line 1: Line 1:
'''Retsensioonid'''
'''Retsensioon projektile DevHelpVR'''
 
=Sissejuhatus=
Retsensiooni eesmärgiks oli võrrelda püstitatud ülesannet lõpptulemusega ning analüüsida esmalt veebiteenuse poolt, kasutatud andmemudeleid, mustreid ning koodi jälgitavust ehk visuaalset poolt. Teiseks tuli analüüsida ka kliendirakendust, mis eesmärgipäraselt peaks suhtlema loodud veebiteenusega.
Retsensiooni käigus hindasime veebiteenuse osas vastavalt järgmiseid nüansse:
1. Andmebaasi olemite arv
2. Kastutatav tehnoloogia (WCF või ASP.NET MVC Web API)
3. Veebiteenuse olemasolu
4. Mitmekasutaja funktsionaalsuse olemasolu
5. Kasutajate tuvastamist ja haldamist
6. Kasutusstatistika üle arve pidamist
7. Piirangute olemasolu pöördumiste arvu osas ning nende haldust
8. Kasutusjuhendi olemasolu
 
==Andmemudel, olemid==
Projekti eesmärgiks oli luua rakendus, mis aitaks arendajal oma töid paremini planeerida ja omada ülevaadet käimasolevatest ning järgmisena algavatest ülesannetest.
Projekti andmemudel on ilusti üles ehitatud, kuid võrreldes esialgu planeerituga esineb erinevus olemite koguarvus, samas on põhifunktsionaalsus kenasti olemas. Andmebaasi olemite arv, ilma Identity olemiteta, on 10. Analüüsi käigus ei suutnud tuvastada ühtegi suurt viga, küll aga saaksime teha mõned soovitused. Nimelt enamik olemeid, milles esineb nn algus ja lõppaeg, võiks lõppaja väli olla pigem valikuline, kuna alati ei pruugi projektil, meeskonnal või muul olemil olla teada lõppaeg selle objekti sisestamisel - ehk et see võib selguda alles peale mõningast analüüsi. Lisaks, tõenäoliselt tähelepanuta jäänud, on UserStory olemi Storypoints attribuudil annotatsioon [Required]. Seda aga tegelikult vaja pole, kuna int on väärtustüüpi muutuja ja nõuab vaikimisi väärtust.
 


=XML ülesannete retsensioon meeskonnalt [https://wiki.itcollege.ee/index.php/PC_KartKicker PC KartKicker]=
=XML ülesannete retsensioon meeskonnalt [https://wiki.itcollege.ee/index.php/PC_KartKicker PC KartKicker]=

Revision as of 20:12, 14 June 2017

Retsensioon projektile DevHelpVR

Sissejuhatus

Retsensiooni eesmärgiks oli võrrelda püstitatud ülesannet lõpptulemusega ning analüüsida esmalt veebiteenuse poolt, kasutatud andmemudeleid, mustreid ning koodi jälgitavust ehk visuaalset poolt. Teiseks tuli analüüsida ka kliendirakendust, mis eesmärgipäraselt peaks suhtlema loodud veebiteenusega. Retsensiooni käigus hindasime veebiteenuse osas vastavalt järgmiseid nüansse: 1. Andmebaasi olemite arv 2. Kastutatav tehnoloogia (WCF või ASP.NET MVC Web API) 3. Veebiteenuse olemasolu 4. Mitmekasutaja funktsionaalsuse olemasolu 5. Kasutajate tuvastamist ja haldamist 6. Kasutusstatistika üle arve pidamist 7. Piirangute olemasolu pöördumiste arvu osas ning nende haldust 8. Kasutusjuhendi olemasolu

Andmemudel, olemid

Projekti eesmärgiks oli luua rakendus, mis aitaks arendajal oma töid paremini planeerida ja omada ülevaadet käimasolevatest ning järgmisena algavatest ülesannetest. Projekti andmemudel on ilusti üles ehitatud, kuid võrreldes esialgu planeerituga esineb erinevus olemite koguarvus, samas on põhifunktsionaalsus kenasti olemas. Andmebaasi olemite arv, ilma Identity olemiteta, on 10. Analüüsi käigus ei suutnud tuvastada ühtegi suurt viga, küll aga saaksime teha mõned soovitused. Nimelt enamik olemeid, milles esineb nn algus ja lõppaeg, võiks lõppaja väli olla pigem valikuline, kuna alati ei pruugi projektil, meeskonnal või muul olemil olla teada lõppaeg selle objekti sisestamisel - ehk et see võib selguda alles peale mõningast analüüsi. Lisaks, tõenäoliselt tähelepanuta jäänud, on UserStory olemi Storypoints attribuudil annotatsioon [Required]. Seda aga tegelikult vaja pole, kuna int on väärtustüüpi muutuja ja nõuab vaikimisi väärtust.


XML ülesannete retsensioon meeskonnalt PC KartKicker

XML fail

Näidisfail valideerub ning vastab struktuuri keerukuskriteeriumitele - on vähemalt neli loogilist dimensiooni, kahel dimensioonil (iterations ja userstory) on kasutatud ID-st informatiivsemaid atribuute.

XML fail on loogiliselt struktureeritud ja kergesti arusaadav, seda ka vaatamata kommentaaride puudumisele. Elementide ja atribuutide nimed on loogilised ja järgivad ühtset stiili. Andmete paigutus atribuutidesse ja elementidesse tundub üldjoontes mõistlik, aga mina isiklikult oleks erinevates kohtades kasutatud 'name' elemendi asemel selle info pannud atribuudiks.

XML schema

Näidisfail vastab schema tingimustele. Andmetüübid vastavad andmete sisule ja eeldatavale kasutusele, aga nt. kuna identifikaatorid on üldjuhul positiivse väärtusega, siis võiks 'id' andmetüüp olla integer asemel unsignedInt. Üritasin erinevaid kavalaid sigadusi korraldada XML failis ja iga kord oli Visual Studio kohe jaol tänu XSD failile, seega on XSD põhjalikult koostatud. Lisaks oleks olnud hea näha kommentaarides XSD valikute seletust äriloogika vaatepunktist.

Transformatsioonid

XML -> HTML transformatsioon

Transformatsioonis on kasutatud mitut 'foreach' tsüklit. Arvutusi ja tingimuste kontrolli transformatsioon ei sisalda. Transformatsiooni tulemusel tekib funktsionaalse kujundusega ja hea loetavusega HTML fail. XSLT fail valideerub. Väljundiks olev HTML annab W3C validaatoris ühe vea ja kolm hoiatust:

1. viga: Line 1, Column 1: no document type declaration; implying "<!DOCTYPE HTML SYSTEM>"

1. hoiatus: Unable to Determine Parse Mode!

2. hoiatus: No DOCTYPE found! Checking with default HTML 4.01 Transitional Document Type.

3. hoiatus: Byte-Order Mark found in UTF-8 File.


XML -> XML transformatsioon

Transformatsioonis on kasutatud parameetreid, ühte 'foreach' tsüklit ning sisaldab mitmeid arvutusi ja boonusena ka sorteerimist. XSLT fail valideerub. Väljundiks olev XML läbis W3C validaatori vigadeta.

Lõpptoote retsensioon meeskonnalt PC KartKicker (Broneering)