HMR - Revision history

Hluts: /* Klientrakenduse retsensioon */

2015-05-30T18:50:34Z

Klientrakenduse retsensioon

← Older revision		Revision as of 21:50, 30 May 2015
Line 699:		Line 699:

	== Klientrakenduse retsensioon ==		== Klientrakenduse retsensioon ==
	~~===~~ Sissejuhatus ~~===~~		''' Sissejuhatus '''

	Grupi KRTT rakendus on realiseeritud Windows WPF rakendusena. Tuleb tunnistada, et meie grupile oli valik vägagi üllatav, ei osanud seda oodata.		Grupi KRTT rakendus on realiseeritud Windows WPF rakendusena. Tuleb tunnistada, et meie grupile oli valik vägagi üllatav, ei osanud seda oodata.

Line 706:		Line 707:
	Samas kui valida projektis ASP NET MVC projekt, siis seal oleks Visual Studio ise väga palju käsitööd ära teinud (CRUD vaated, Bootstrap, vormikontroll ja palju muud). Võibolla oli asi tingitud sellest, et olles äsja tulnud C# ainest oli WPF rakenduse tegemise kogemus olemas ja seega otsustas grupp KRTT panustada enda tugevatele külgedele.		Samas kui valida projektis ASP NET MVC projekt, siis seal oleks Visual Studio ise väga palju käsitööd ära teinud (CRUD vaated, Bootstrap, vormikontroll ja palju muud). Võibolla oli asi tingitud sellest, et olles äsja tulnud C# ainest oli WPF rakenduse tegemise kogemus olemas ja seega otsustas grupp KRTT panustada enda tugevatele külgedele.

	~~===~~ Rakenduse testimine ~~===~~		''' Rakenduse testimine '''

	Lühidalt öeldes tuleb nentida, et rakenduse kasutajakogemus ei olnud väga meeldiv. Kasutajaliides polnud kõige kaunim, aga see pole meie hinnangul kõige suurem miinus. Peamine probleem seisnes selles, et WPF rakendus kippus kokku jooksma. See on väga suur miinus.		Lühidalt öeldes tuleb nentida, et rakenduse kasutajakogemus ei olnud väga meeldiv. Kasutajaliides polnud kõige kaunim, aga see pole meie hinnangul kõige suurem miinus. Peamine probleem seisnes selles, et WPF rakendus kippus kokku jooksma. See on väga suur miinus.

Line 721:		Line 723:
	Siinkohal on taaskord näha kui oluline on strateegiline otsus rakenduse tüübi valiku osas. MVC rakenduses oleks väga palju veahaldusest ära tehtud. See valik oleks tõenäoliselt väga palju säästnud grupi KRTT aega. Samas tuleb nentida, et ilsmelt pole grupp KRTT vigadesse väga kergekäoliselt suhtunud, vaid on proovinud neid ka lahendada. Sellele viitavad mitmed koodi sisse jäänud break pointid.		Siinkohal on taaskord näha kui oluline on strateegiline otsus rakenduse tüübi valiku osas. MVC rakenduses oleks väga palju veahaldusest ära tehtud. See valik oleks tõenäoliselt väga palju säästnud grupi KRTT aega. Samas tuleb nentida, et ilsmelt pole grupp KRTT vigadesse väga kergekäoliselt suhtunud, vaid on proovinud neid ka lahendada. Sellele viitavad mitmed koodi sisse jäänud break pointid.

	~~===~~ Soovitused valminud WPF rakenduse parendamiseks ~~===~~		''' Soovitused valminud WPF rakenduse parendamiseks '''

	Meie grupi soovitus oleks veidi mõelda üldisematele kujundusreeglitele. Hetkel on kasutajaliidese pool väga minimalistlik ja raskesti hoomatav.		Meie grupi soovitus oleks veidi mõelda üldisematele kujundusreeglitele. Hetkel on kasutajaliidese pool väga minimalistlik ja raskesti hoomatav.

Line 730:		Line 733:
	Samas me ei saa väita, et seda funktsionaalust pole olemas. Põhjus seisneb selles, et me ei suutnud WPF rakenduses ennast sisse logida ja rakendust proovida. Meie arvamus põhineb puhtalt koodianalüüsil. Me ei leidnud neid teenuste, põhiolemite ega vaatemudelite alt.		Samas me ei saa väita, et seda funktsionaalust pole olemas. Põhjus seisneb selles, et me ei suutnud WPF rakenduses ennast sisse logida ja rakendust proovida. Meie arvamus põhineb puhtalt koodianalüüsil. Me ei leidnud neid teenuste, põhiolemite ega vaatemudelite alt.

	~~===~~ Koodistiil ~~===~~		''' Koodistiil '''

	Koodistiili kohta pole tegelikult mitte midagi halba öelda. Kood on sarnaselt teenuse koodile kirjutatud väga puhtalt ja mõistetavalt, samuti on kood kenasti vormindatud. Samas on rakendusel väga palju bugisi, seetõttu hea koodistiil ei kaalu üles rakenduse bugisid.		Koodistiili kohta pole tegelikult mitte midagi halba öelda. Kood on sarnaselt teenuse koodile kirjutatud väga puhtalt ja mõistetavalt, samuti on kood kenasti vormindatud. Samas on rakendusel väga palju bugisi, seetõttu hea koodistiil ei kaalu üles rakenduse bugisid.

	~~===~~ Kokkuvõte ~~===~~		''' Kokkuvõte '''

	Kokkuvõtteks võib öelda, et WPF rakenduse arendamisega on mõistlik veel tegeleda. Kasutamisel ilmnes väga palju vigu ja seega me ei saanud kogu funktsionaalust testida.		Kokkuvõtteks võib öelda, et WPF rakenduse arendamisega on mõistlik veel tegeleda. Kasutamisel ilmnes väga palju vigu ja seega me ei saanud kogu funktsionaalust testida.

Hluts: /* Veebiteenuse retsensioon */

2015-05-30T18:49:37Z

Veebiteenuse retsensioon

← Older revision		Revision as of 21:49, 30 May 2015
Line 652:		Line 652:

	== Veebiteenuse retsensioon ==		== Veebiteenuse retsensioon ==
	~~===~~ Sissejuhatus ~~===~~		''' Sissejuhatus '''

	Grupi KRTT veebiteenus oli realiseeritud ASP.NET MVC Web API-na ning kasutatud oli REST teenust ning andmete käitlemiseks kasutati JSONi formaati. Teenuse puhul oli kasutatud töö spetsifikatsioonis ettenähtud tehnoloogiaid ning arendusmustreid. Töös oli kasutatud kenasti UOW mustrit, s.t päringud ei käinud otse andmebaasi vastu. Samuti oli andmekihis kasutatud interface, nii nagu oli projekti kirjelduses ette nähtud.		Grupi KRTT veebiteenus oli realiseeritud ASP.NET MVC Web API-na ning kasutatud oli REST teenust ning andmete käitlemiseks kasutati JSONi formaati. Teenuse puhul oli kasutatud töö spetsifikatsioonis ettenähtud tehnoloogiaid ning arendusmustreid. Töös oli kasutatud kenasti UOW mustrit, s.t päringud ei käinud otse andmebaasi vastu. Samuti oli andmekihis kasutatud interface, nii nagu oli projekti kirjelduses ette nähtud.

Line 659:		Line 660:
	Töös oli kasutatud code first lähenemist ning andmebaasi genereerimiseks on kasutatud Entity Frameworki. Samuti on töös kasutatud nõutul määral olemeid, arvestamata seejuures sisse autentimise toimimiseks vajalike olemite arvust.		Töös oli kasutatud code first lähenemist ning andmebaasi genereerimiseks on kasutatud Entity Frameworki. Samuti on töös kasutatud nõutul määral olemeid, arvestamata seejuures sisse autentimise toimimiseks vajalike olemite arvust.

	~~===~~ Projekti üldiste nõuete täidetus ~~===~~		''' Projekti üldiste nõuete täidetus '''

	Tulles nüüd detailide juurde siis alustuseks tuleb ära mainida, et grupi KRTT veebiteenus läks meie meeskonna liikmete arvutites ilma probleemideta tööle.		Tulles nüüd detailide juurde siis alustuseks tuleb ära mainida, et grupi KRTT veebiteenus läks meie meeskonna liikmete arvutites ilma probleemideta tööle.

Line 672:		Line 674:
	Projekti nõudeks oli ka teenuse poole pöördumiste arvu piiramine ning piirangute haldamine. See funktsionaalsus oli samuti implementeerimata. Vähemalt meie grupp selle kohta projektis viiteid ei leidnud.		Projekti nõudeks oli ka teenuse poole pöördumiste arvu piiramine ning piirangute haldamine. See funktsionaalsus oli samuti implementeerimata. Vähemalt meie grupp selle kohta projektis viiteid ei leidnud.

	~~===~~ Üldine lahenduse (solution) ülesehitus ~~===~~		''' Üldine lahenduse (solution) ülesehitus '''

	Lahendus on kenasti struktureeritud ja jaotatud eri projektide vahel. See muudab lahendusest aru saamine väga mugavamaks. Kasutatud on levinud ülesehtist, olemid on omaette projektis. Andmebaasi loogika on omaette projektis ning samuti ka API. Kuna meie grupp kasutas oma projektis sarnast loogikat, siis oli väga kerge projektis orienteeruda.		Lahendus on kenasti struktureeritud ja jaotatud eri projektide vahel. See muudab lahendusest aru saamine väga mugavamaks. Kasutatud on levinud ülesehtist, olemid on omaette projektis. Andmebaasi loogika on omaette projektis ning samuti ka API. Kuna meie grupp kasutas oma projektis sarnast loogikat, siis oli väga kerge projektis orienteeruda.

	Väikese märkusena võiks lisada, et meie soovitus oleks olemite projektile (Domain) teha spetsiaalne kaust, kus on kirjas projekti olemid. Praegu olid nad otse lahenduses, hiljem kui peaks tekkima vajadus lahendust laiendada, siis erinevate olemite kirjeldamine projekti üldises juurkataloogis võib põhjustada ebamugavusi.		Väikese märkusena võiks lisada, et meie soovitus oleks olemite projektile (Domain) teha spetsiaalne kaust, kus on kirjas projekti olemid. Praegu olid nad otse lahenduses, hiljem kui peaks tekkima vajadus lahendust laiendada, siis erinevate olemite kirjeldamine projekti üldises juurkataloogis võib põhjustada ebamugavusi.

	~~===~~ Koodistiil ~~===~~		''' Koodistiil '''

	Koodi puhul tuleb nentida, et see on kenasti vormistatud ja kergesti jälgitav. Samuti on olemite atribuutide ja meetodite nimed nimetatud nõnda, et nendest on võimalik kogu atribuudi ja meetodi sisu välja lugeda. Sellest tuleneval on koodi lugemine väga mugav ning sellest aru saamine väga intuitiivne.		Koodi puhul tuleb nentida, et see on kenasti vormistatud ja kergesti jälgitav. Samuti on olemite atribuutide ja meetodite nimed nimetatud nõnda, et nendest on võimalik kogu atribuudi ja meetodi sisu välja lugeda. Sellest tuleneval on koodi lugemine väga mugav ning sellest aru saamine väga intuitiivne.

	Mainime ära, et käesoleva töö analüüsijatele antud lahendus meeldib. Selle asemel et kirjutada väga ebaloogiliste muutujate nimedega koodi (näiteks string x ning int y) ning seejärel kommentaarides täpsustades muutujate olemust annavad käesolevas koodis olevad muutujate ning meetodite nimetused kohe vajaliku info kätte. Meie grupp pooldab väga taolist lahendust, et kood kirjutatakse nõnda, et see kommenteerib end niiöelda ise ja ohtraid kommentaare pole tarvis.		Mainime ära, et käesoleva töö analüüsijatele antud lahendus meeldib. Selle asemel et kirjutada väga ebaloogiliste muutujate nimedega koodi (näiteks string x ning int y) ning seejärel kommentaarides täpsustades muutujate olemust annavad käesolevas koodis olevad muutujate ning meetodite nimetused kohe vajaliku info kätte. Meie grupp pooldab väga taolist lahendust, et kood kirjutatakse nõnda, et see kommenteerib end niiöelda ise ja ohtraid kommentaare pole tarvis.

	~~===~~ Üldised kommentaarid ~~===~~		''' Üldised kommentaarid '''

	Meie meeskonnale meeldis fakt, et mudelites oli kasutatud enumeid. Kuigi mainiksime siin ära väikese kriitikana, et võibolla oleks ka nende valikuväljade puhul kasutada erinevaid objekte, et neid näiteks hiljem kasutajaliidese kaudu muuta ning hallata. Samas me ka ei kritiseeri nende kasutamist, kindlasti nad lühendasid arendamise aega.		Meie meeskonnale meeldis fakt, et mudelites oli kasutatud enumeid. Kuigi mainiksime siin ära väikese kriitikana, et võibolla oleks ka nende valikuväljade puhul kasutada erinevaid objekte, et neid näiteks hiljem kasutajaliidese kaudu muuta ning hallata. Samas me ka ei kritiseeri nende kasutamist, kindlasti nad lühendasid arendamise aega.

	Rääkides API projektist (WebApiApp), siis grupi kiituseks tuleb öelda, et seal on kontrollerite meetodid kenasti kommenteeritud. Samas kõrvalmärkusena võib öelda, et meetodid on nõnda nimetatud, et need annavad kohe aimu meetodi iseloomust. Samas plusspunktid nende eest.		Rääkides API projektist (WebApiApp), siis grupi kiituseks tuleb öelda, et seal on kontrollerite meetodid kenasti kommenteeritud. Samas kõrvalmärkusena võib öelda, et meetodid on nõnda nimetatud, et need annavad kohe aimu meetodi iseloomust. Samas plusspunktid nende eest.

	~~===~~ Kokkuvõte ~~===~~		''' Kokkuvõte '''

	Kokkuvõtteks võib öelda, et API tehnilise poole pealt pole ühtegi negatiivset sõna. Kõik toimib kenasti. Kood on kirjutatud arusaadavalt, samuti on lahendus loogiliselt struktureeritud.		Kokkuvõtteks võib öelda, et API tehnilise poole pealt pole ühtegi negatiivset sõna. Kõik toimib kenasti. Kood on kirjutatud arusaadavalt, samuti on lahendus loogiliselt struktureeritud.

Hluts: /* Klientrakendus */

2015-05-30T18:45:17Z

Klientrakendus

← Older revision		Revision as of 21:45, 30 May 2015
Line 692:		Line 692:
	Miinuspoole pealt peab mainima, et kahjuks täit rakenduse funktsionaalsust (näiteks kasutusstatistika) pole hetkel veel realiseeritud.		Miinuspoole pealt peab mainima, et kahjuks täit rakenduse funktsionaalsust (näiteks kasutusstatistika) pole hetkel veel realiseeritud.

	== ~~Klientrakendus~~ ==		== Klientrakenduse retsensioon ==
	=== Sissejuhatus ===		=== Sissejuhatus ===
	Grupi KRTT rakendus on realiseeritud Windows WPF rakendusena. Tuleb tunnistada, et meie grupile oli valik vägagi üllatav, ei osanud seda oodata.		Grupi KRTT rakendus on realiseeritud Windows WPF rakendusena. Tuleb tunnistada, et meie grupile oli valik vägagi üllatav, ei osanud seda oodata.

Hluts: /* Klientrakendus */

2015-05-30T18:45:03Z

Klientrakendus

← Older revision		Revision as of 21:45, 30 May 2015
Line 693:		Line 693:

	== Klientrakendus ==		== Klientrakendus ==
			=== Sissejuhatus ===
			Grupi KRTT rakendus on realiseeritud Windows WPF rakendusena. Tuleb tunnistada, et meie grupile oli valik vägagi üllatav, ei osanud seda oodata.

			Meie hinnangul pole antud valik selles mõttes õnnestunud, et WPF puhul tuleb väga suur osa kasutajaliidesest ise käsitsi kirjutada. Antud valikuga kaasnes ilmselt väga palju ajakulu.

			Samas kui valida projektis ASP NET MVC projekt, siis seal oleks Visual Studio ise väga palju käsitööd ära teinud (CRUD vaated, Bootstrap, vormikontroll ja palju muud). Võibolla oli asi tingitud sellest, et olles äsja tulnud C# ainest oli WPF rakenduse tegemise kogemus olemas ja seega otsustas grupp KRTT panustada enda tugevatele külgedele.

			=== Rakenduse testimine ===
			Lühidalt öeldes tuleb nentida, et rakenduse kasutajakogemus ei olnud väga meeldiv. Kasutajaliides polnud kõige kaunim, aga see pole meie hinnangul kõige suurem miinus. Peamine probleem seisnes selles, et WPF rakendus kippus kokku jooksma. See on väga suur miinus.

			Toome ühe kasutusjuhtumi analüüsi, kus rakendus lakkas töötamast. Üritades end rakenduusega registreeruda ilmneb tõsiasi, et peale vajalike andmete sisestamist mingit registreerumist ei toimu ning sisse logida pole võimalik.

			Negatiivne on ka see aspekt, et peale seda kaob menüüs ära “Register” nupp, mis võimaldaks tekitada teise kasutaja, igaks juhuks testimaks, et äkki siis logimine õnnestub. Samuti puudub võimalus andmete muutmiseks. Kui on soov “Register” nupp tagasi saada siis tuleb selleks kas vajutada “Menu” või “Home” nupule. Tegemist ei ole hea kasutajakogemusega ja soovitaks seetõttu kasutajaliidese paremini läbi mõelda.

			Kasutaja registreerimise vormi kohta oleks märkuseks, et salasõna väljadel ei ole hea kuvada sisestatud salasõna. Sobilik oleks kasutada standartset lahendust, kus sisestatud salasõna kuvatakse täppidena.

			Viga ilmnes ka siis, et kui proovida eelnevalt Kui proovida eelnevalt sisestatud kasutajaga rakendusse siseneda, siis on tulemuseks exception, mida ei käsitleta, s.t koodis puudub „try … catch“ plokk. Viimane on aga veatundliku koodi puhul teatavasti oluline. Kasutaja seisukohast ei saa seega mitte mingisugust informatsiooni selle kohta, mis valesti läks.

			Samamoodi kukub rakendus kokku kui vajutada nupule “Load cars”, sel puhul samuti ei kuvata mingisugust veateadet. Võib vaid oletada, mis on selle vea põhjuseks, kas näiteks rakendus ei saa teenusega ühendust või on mingi muu probleem. Igal juhul oleks hea sellest probleemist kasutajat informeerida.

			Siinkohal on taaskord näha kui oluline on strateegiline otsus rakenduse tüübi valiku osas. MVC rakenduses oleks väga palju veahaldusest ära tehtud. See valik oleks tõenäoliselt väga palju säästnud grupi KRTT aega. Samas tuleb nentida, et ilsmelt pole grupp KRTT vigadesse väga kergekäoliselt suhtunud, vaid on proovinud neid ka lahendada. Sellele viitavad mitmed koodi sisse jäänud break pointid.

			=== Soovitused valminud WPF rakenduse parendamiseks ===
			Meie grupi soovitus oleks veidi mõelda üldisematele kujundusreeglitele. Hetkel on kasutajaliidese pool väga minimalistlik ja raskesti hoomatav.

			Hetkel on suur valge ala “MainWindow” keskel, mis koodianalüüsist lähtudes tundub olevat auto pildi kuvamiseks, seda võib aru saada tutvudes faili Cars.xaml sisuga. Samas kui ühtegi autot valitud pole, siis jääb üldmulje vägagi tühjaks.

			Meie hinnangul puuduvad WPF rakenduses kasutajaliideses kohad kõikide veebiteenuse olemite manipuleerimiseks. Näiteks meie grupile jäi selgusetuks kuidas sisestada ettevõtet (olem Firm), rendilepingut, ülevaatust või autovarustust.

			Samas me ei saa väita, et seda funktsionaalust pole olemas. Põhjus seisneb selles, et me ei suutnud WPF rakenduses ennast sisse logida ja rakendust proovida. Meie arvamus põhineb puhtalt koodianalüüsil. Me ei leidnud neid teenuste, põhiolemite ega vaatemudelite alt.

			=== Koodistiil ===
			Koodistiili kohta pole tegelikult mitte midagi halba öelda. Kood on sarnaselt teenuse koodile kirjutatud väga puhtalt ja mõistetavalt, samuti on kood kenasti vormindatud. Samas on rakendusel väga palju bugisi, seetõttu hea koodistiil ei kaalu üles rakenduse bugisid.

			=== Kokkuvõte ===
			Kokkuvõtteks võib öelda, et WPF rakenduse arendamisega on mõistlik veel tegeleda. Kasutamisel ilmnes väga palju vigu ja seega me ei saanud kogu funktsionaalust testida.

			Meie poolne soovitus oleks, et grupp KRTT kaaluks näiteks mõne spetsiaalse veebiliidese arendamist, miks mitte näiteks ASP NET MVC rakendus. Seal võtab raamistik rakenduse baasfunktsionaalsuse osas väga palju tööd enda kanda ning arendajatel jääb aega eriarendustega tegelemiseks.

	= Logi =		= Logi =

Hluts: /* Veebiteenus */

2015-05-30T18:42:26Z

Veebiteenus

← Older revision		Revision as of 21:42, 30 May 2015
Line 651:		Line 651:
	Meie grupp valis retsenseeirmiseks grupi KRTT (https://wiki.itcollege.ee/index.php/KRTT#Veebiteenus_ja_klientrakendus) töö. Retsensioon on kahes osas, kõigepealt teenuse ja seejärel rakenduse retsensioon.		Meie grupp valis retsenseeirmiseks grupi KRTT (https://wiki.itcollege.ee/index.php/KRTT#Veebiteenus_ja_klientrakendus) töö. Retsensioon on kahes osas, kõigepealt teenuse ja seejärel rakenduse retsensioon.

	== ~~Veebiteenus~~ ==		== Veebiteenuse retsensioon ==
	=== Sissejuhatus ===		=== Sissejuhatus ===
	Grupi KRTT veebiteenus oli realiseeritud ASP.NET MVC Web API-na ning kasutatud oli REST teenust ning andmete käitlemiseks kasutati JSONi formaati. Teenuse puhul oli kasutatud töö spetsifikatsioonis ettenähtud tehnoloogiaid ning arendusmustreid. Töös oli kasutatud kenasti UOW mustrit, s.t päringud ei käinud otse andmebaasi vastu. Samuti oli andmekihis kasutatud interface, nii nagu oli projekti kirjelduses ette nähtud.		Grupi KRTT veebiteenus oli realiseeritud ASP.NET MVC Web API-na ning kasutatud oli REST teenust ning andmete käitlemiseks kasutati JSONi formaati. Teenuse puhul oli kasutatud töö spetsifikatsioonis ettenähtud tehnoloogiaid ning arendusmustreid. Töös oli kasutatud kenasti UOW mustrit, s.t päringud ei käinud otse andmebaasi vastu. Samuti oli andmekihis kasutatud interface, nii nagu oli projekti kirjelduses ette nähtud.

Hluts: /* Retsensioonid veebiteenusele ja klientrakendusele */

2015-05-30T18:42:05Z

Retsensioonid veebiteenusele ja klientrakendusele

← Older revision		Revision as of 21:42, 30 May 2015
Line 649:		Line 649:
	= Retsensioonid veebiteenusele ja klientrakendusele =		= Retsensioonid veebiteenusele ja klientrakendusele =

	Meie grupp valis retsenseeirmiseks grupi KRTT (https://wiki.itcollege.ee/index.php/KRTT#Veebiteenus_ja_klientrakendus ) töö.		Meie grupp valis retsenseeirmiseks grupi KRTT (https://wiki.itcollege.ee/index.php/KRTT#Veebiteenus_ja_klientrakendus) töö. Retsensioon on kahes osas, kõigepealt teenuse ja seejärel rakenduse retsensioon.

	== Veebiteenus ==		== Veebiteenus ==

Hluts: /* Retsensioonid veebiteenusele ja klientrakendusele */

2015-05-30T18:41:02Z

Retsensioonid veebiteenusele ja klientrakendusele

← Older revision		Revision as of 21:41, 30 May 2015
Line 648:		Line 648:

	= Retsensioonid veebiteenusele ja klientrakendusele =		= Retsensioonid veebiteenusele ja klientrakendusele =

			Meie grupp valis retsenseeirmiseks grupi KRTT (https://wiki.itcollege.ee/index.php/KRTT#Veebiteenus_ja_klientrakendus ) töö.

	== Veebiteenus ==		== Veebiteenus ==
Line 691:		Line 693:

	== Klientrakendus ==		== Klientrakendus ==


	= Logi =		= Logi =

Hluts: /* Veebiteenus */

2015-05-30T18:40:26Z

Veebiteenus

← Older revision		Revision as of 21:40, 30 May 2015
Line 650:		Line 650:

	== Veebiteenus ==		== Veebiteenus ==
			=== Sissejuhatus ===
			Grupi KRTT veebiteenus oli realiseeritud ASP.NET MVC Web API-na ning kasutatud oli REST teenust ning andmete käitlemiseks kasutati JSONi formaati. Teenuse puhul oli kasutatud töö spetsifikatsioonis ettenähtud tehnoloogiaid ning arendusmustreid. Töös oli kasutatud kenasti UOW mustrit, s.t päringud ei käinud otse andmebaasi vastu. Samuti oli andmekihis kasutatud interface, nii nagu oli projekti kirjelduses ette nähtud.

			Projektis oli kasutatud code first lähenemist ning andmebaasi genereerimiseks kasutati Entity Frameworki. Et API ei annaks välja ebavajalikku infot oli projektis kenasti realiseeritud DTO (Data Transfer Object) vahekiht.

			Töös oli kasutatud code first lähenemist ning andmebaasi genereerimiseks on kasutatud Entity Frameworki. Samuti on töös kasutatud nõutul määral olemeid, arvestamata seejuures sisse autentimise toimimiseks vajalike olemite arvust.

			=== Projekti üldiste nõuete täidetus ===
			Tulles nüüd detailide juurde siis alustuseks tuleb ära mainida, et grupi KRTT veebiteenus läks meie meeskonna liikmete arvutites ilma probleemideta tööle.

			Tulles nüüd üldisemate teenuse poolele esitatud nõuete juurde, siis kõigepealt mainime ära, et teenus toimib kenasti. Projektis oli teenusele esitatud turvatuse nõue, vaatlusaluse projekti puhul on see nõue täidetud.

			Samas oli teenuse puhul väike soovitus, et erinevateks toiminguteks oleks vastav haldusliides. Mingit rakendust kahjuks haldamiseks polnud. See tõstatas mõningaid küsimusi, nimelt vaikimisi oli õppejõu Andres Käveri näidiskoodis olemas MVC rakenduse kujul kasutajahalduse rakendus. Kahjuks need põhjused, miks antud rakendusest otsustati veebiteenuses loobuda ei ole meile teadlikud. Väikese märkusena võiks siiski öelda, et rakenduse olemasolu annaks API kasutamiskogemusele palju juurde.

			Ilmselt siis kogu kasutajahaldus on jäetud otse andmebaasi vastu käima, see kindlasti pole hea ega turvaline praktika. Kui grupil on huvi antud projekti veel edasi arendada, siis kindlasti soovitaks luua mingi API poolse haldamise rakenduse.

			Teine nõue projektis oli see, et teenus peaks pidama arvet kasutajate ning kasutusstatistika üle kasutajate lõikes. See nõue käesolevas projektis polnud täidetud, vähemalt meie grupp ei leidnud nii teenuse kui rakenduse väljundis sellele viiteid. Samuti ei leidnud me ka koodis mingeid kohti, mis viitaksid sellele, et antud nõue oleks implementeeritud.

			Projekti nõudeks oli ka teenuse poole pöördumiste arvu piiramine ning piirangute haldamine. See funktsionaalsus oli samuti implementeerimata. Vähemalt meie grupp selle kohta projektis viiteid ei leidnud.

			=== Üldine lahenduse (solution) ülesehitus ===
			Lahendus on kenasti struktureeritud ja jaotatud eri projektide vahel. See muudab lahendusest aru saamine väga mugavamaks. Kasutatud on levinud ülesehtist, olemid on omaette projektis. Andmebaasi loogika on omaette projektis ning samuti ka API. Kuna meie grupp kasutas oma projektis sarnast loogikat, siis oli väga kerge projektis orienteeruda.

			Väikese märkusena võiks lisada, et meie soovitus oleks olemite projektile (Domain) teha spetsiaalne kaust, kus on kirjas projekti olemid. Praegu olid nad otse lahenduses, hiljem kui peaks tekkima vajadus lahendust laiendada, siis erinevate olemite kirjeldamine projekti üldises juurkataloogis võib põhjustada ebamugavusi.

			=== Koodistiil ===
			Koodi puhul tuleb nentida, et see on kenasti vormistatud ja kergesti jälgitav. Samuti on olemite atribuutide ja meetodite nimed nimetatud nõnda, et nendest on võimalik kogu atribuudi ja meetodi sisu välja lugeda. Sellest tuleneval on koodi lugemine väga mugav ning sellest aru saamine väga intuitiivne.

			Mainime ära, et käesoleva töö analüüsijatele antud lahendus meeldib. Selle asemel et kirjutada väga ebaloogiliste muutujate nimedega koodi (näiteks string x ning int y) ning seejärel kommentaarides täpsustades muutujate olemust annavad käesolevas koodis olevad muutujate ning meetodite nimetused kohe vajaliku info kätte. Meie grupp pooldab väga taolist lahendust, et kood kirjutatakse nõnda, et see kommenteerib end niiöelda ise ja ohtraid kommentaare pole tarvis.

			=== Üldised kommentaarid ===
			Meie meeskonnale meeldis fakt, et mudelites oli kasutatud enumeid. Kuigi mainiksime siin ära väikese kriitikana, et võibolla oleks ka nende valikuväljade puhul kasutada erinevaid objekte, et neid näiteks hiljem kasutajaliidese kaudu muuta ning hallata. Samas me ka ei kritiseeri nende kasutamist, kindlasti nad lühendasid arendamise aega.

			Rääkides API projektist (WebApiApp), siis grupi kiituseks tuleb öelda, et seal on kontrollerite meetodid kenasti kommenteeritud. Samas kõrvalmärkusena võib öelda, et meetodid on nõnda nimetatud, et need annavad kohe aimu meetodi iseloomust. Samas plusspunktid nende eest.

			=== Kokkuvõte ===
			Kokkuvõtteks võib öelda, et API tehnilise poole pealt pole ühtegi negatiivset sõna. Kõik toimib kenasti. Kood on kirjutatud arusaadavalt, samuti on lahendus loogiliselt struktureeritud.

			Miinuspoole pealt peab mainima, et kahjuks täit rakenduse funktsionaalsust (näiteks kasutusstatistika) pole hetkel veel realiseeritud.

	== Klientrakendus ==		== Klientrakendus ==

Mkabanen: /* 30.05.15 - Teenus ja rakendus */

2015-05-30T17:11:26Z

30.05.15 - Teenus ja rakendus

← Older revision		Revision as of 20:11, 30 May 2015
Line 671:		Line 671:
	Teenus ja rakendus enam-vähem(pigem vähem) valmis ning konstruktiivse kriitika jaoks avalikustatud.		Teenus ja rakendus enam-vähem(pigem vähem) valmis ning konstruktiivse kriitika jaoks avalikustatud.

	== 30.05.15 - ~~Teenus ja rakendus~~ ==		== 30.05.15 - Retsensioonid ==
	Retsensioonid ~~tehtud~~
			Retsensioonid Veebiteenusele ja klientrakendusele valmis.

Mkabanen: /* 24.05.15 - Teenus ja rakendus */

2015-05-30T17:10:53Z

24.05.15 - Teenus ja rakendus

← Older revision		Revision as of 20:10, 30 May 2015
Line 670:		Line 670:
	== 24.05.15 - Teenus ja rakendus ==		== 24.05.15 - Teenus ja rakendus ==
	Teenus ja rakendus enam-vähem(pigem vähem) valmis ning konstruktiivse kriitika jaoks avalikustatud.		Teenus ja rakendus enam-vähem(pigem vähem) valmis ning konstruktiivse kriitika jaoks avalikustatud.

			== 30.05.15 - Teenus ja rakendus ==
			Retsensioonid tehtud