Talk:DevHelpVR: Difference between revisions

From ICO wiki
Jump to navigationJump to search
No edit summary
No edit summary
Line 1: Line 1:
'''Retsensioon projektile DevHelpVR'''
=Retsensioon projektile DevHelpVR=


=Sissejuhatus=
==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 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:
Retsensiooni käigus hindasime veebiteenuse osas vastavalt järgmiseid nüansse:
1. Andmebaasi olemite arv
1. Andmebaasi olemite arv
2. Kastutatav tehnoloogia (WCF või ASP.NET MVC Web API)
2. Kastutatav tehnoloogia (WCF või ASP.NET MVC Web API)
3. Veebiteenuse olemasolu
3. Veebiteenuse olemasolu
4. Mitmekasutaja funktsionaalsuse olemasolu
4. Mitmekasutaja funktsionaalsuse olemasolu
5. Kasutajate tuvastamist ja haldamist
5. Kasutajate tuvastamist ja haldamist
6. Kasutusstatistika üle arve pidamist
6. Kasutusstatistika üle arve pidamist
7. Piirangute olemasolu pöördumiste arvu osas ning nende haldust
7. Piirangute olemasolu pöördumiste arvu osas ning nende haldust
8. Kasutusjuhendi olemasolu
8. Kasutusjuhendi olemasolu


==Andmemudel, olemid==
==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 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.
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 (esialgses analüüsidokumendis saime neid kokku 16).  
 
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.
 
Kolmanda soovitusena võiks mainida ka seda, et tegelikult saaks edukalt ära kasutada ka Identity-ga kaasa tulevaid olemeid näiteks rollide ja kasutajate kirjeldamiseks.
=XML ülesannete retsensioon meeskonnalt [https://wiki.itcollege.ee/index.php/PC_KartKicker 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>"
Välja võib tuua seda, et kõik olemid ja seosed on hästi läbimõeldud, kasutatud on piisavalt annotatsioone ja kõik viitetüüpi olemi atribuudid on sobivalt piiratud (eelkõige string muutujad).


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.


=Tase 1 [https://wiki.itcollege.ee/index.php/PC_KartKicker Link]=
==Tase 2==


'''XML -> XML transformatsioon'''
'''Pealkiri 13'''


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 [https://wiki.itcollege.ee/index.php/PC_KartKicker PC KartKicker] (Broneering)=
=Lõpptoote retsensioon meeskonnalt [https://wiki.itcollege.ee/index.php/PC_KartKicker PC KartKicker] (Broneering)=

Revision as of 20:22, 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 (esialgses analüüsidokumendis saime neid kokku 16). 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. Kolmanda soovitusena võiks mainida ka seda, et tegelikult saaks edukalt ära kasutada ka Identity-ga kaasa tulevaid olemeid näiteks rollide ja kasutajate kirjeldamiseks.

Välja võib tuua seda, et kõik olemid ja seosed on hästi läbimõeldud, kasutatud on piisavalt annotatsioone ja kõik viitetüüpi olemi atribuudid on sobivalt piiratud (eelkõige string muutujad).



Tase 1 Link

Tase 2

Pealkiri 13


Lõpptoote retsensioon meeskonnalt PC KartKicker (Broneering)