Talk:Meeskond:vertigo - Revision history

Ttallerm at 21:01, 15 June 2015

2015-06-15T21:01:49Z

← Older revision		Revision as of 00:01, 16 June 2015
Line 201:		Line 201:
	*MVC näeb sammuti täiesti adekvaatne välja. funktsionaalsused on kõik samad mis WPF-il. Sai ka kontrollitud turvalisust, ning see oli ilusti tagatud, kasutal ei olnud võimalik ligi pääseda infole, mida ta nägema ei peaks. <br>		*MVC näeb sammuti täiesti adekvaatne välja. funktsionaalsused on kõik samad mis WPF-il. Sai ka kontrollitud turvalisust, ning see oli ilusti tagatud, kasutal ei olnud võimalik ligi pääseda infole, mida ta nägema ei peaks. <br>
	Selle koodi ülevaatamine ja katsetamine oli üpriski huvitav, kuna meeskonna Vertigo idee oli väga hästi teostatud. Vigu väga palju ei leidnud, osad kohad tekitasid küsimusi, kuid muidu oli kõik suurepärane		Selle koodi ülevaatamine ja katsetamine oli üpriski huvitav, kuna meeskonna Vertigo idee oli väga hästi teostatud. Vigu väga palju ei leidnud, osad kohad tekitasid küsimusi, kuid muidu oli kõik suurepärane

			=Veebiteenuse retsensioon meeskonna Bananaphone poolt=

			Vertigo veebiteenuse solution koosneb kõigest vajaminevast – BLL, DAL, Domain, Identity ja WebService. Nagu kõigis teisteski projektides, on kasutusele võetud Andres Käveri Identity, mis on iluisti ära seotud oma mudelitega ning tööle saadud. Mudelites on pandud mõningad piirangud peale, kuigi neid võiks rohkem olla, näiteks mõni veateade. Hea on näha, et oma mudelid ja Identity mudelid on eraldiseisvad. Koodi on piisavalt kommenteeritud nagu nõutud. Andmebaasi loomisel lisatakse kohe esmased mõned andmed nagu näiteks admin user kõrgema juurdepääsu lubadega.<br>
			DAL kihis kasutatakse repositorysid, factoryt ja uow’d. Ilusti on kasutatud Nloggerit. Andmebaasi loomisel lisatakse kohe esmased mõned andmed nagu näiteks admin user kõrgema juurdepääsu lubadega. Huvitavalt on lahendatud piltide hankimise osa ja plusspunktid selle eest, ootasin pigem googlest leitud urle piltidele.<br>
			Kõik factoryd koosnevad CreatBasicDTO’dest, ning ka õigustatult, sest rohkem neil vaja ei lähe. Service’itest saab kõik vajaliku kätte. Midagi halba ei oska selle koha pealt öelda. Linq selekteerimist on kasutatud mõistlikult. On näha, et ei ole arutult ümber copytud DTO’si vaid vastavalt vajadusele neid loodud, nagu näiteks AnswerDTO ja GameAnswerDTO.<br>
			WebApi on korralikult tehtud, on olemas turvamine, ning kõige suurem pluss, et päringute piirangud, mida paljudel ei ole. Nagu ka eelnevalt on tähelepandud ja ka kaitsmisel välja toodud, et on kasutatud cache-imist, millega kõik hakkama ei saanud.<br>
			Kokkuvõttes võib öelda, et projekt on igati väärt oma punkte, kõik vajalik olemas, kui välja arvata eraldi log pidamine, mis tekitas kaitsmisel vastuväiteid kas seda oli ikka vaja. Kood on korralik, üleliigset ei ole, ja on aru saada, et teatakse mida on tehtud kus ja miks.

Tsuurkae at 17:50, 15 June 2015

2015-06-15T17:50:30Z

← Older revision		Revision as of 20:50, 15 June 2015
Line 182:		Line 182:

	Jällegi tuleb tunnistada, et olenemate pisikestest vigades on Vertigo teinud suurepärast tööd. Hindeks meie poolt järjekordselt maksimum.		Jällegi tuleb tunnistada, et olenemate pisikestest vigades on Vertigo teinud suurepärast tööd. Hindeks meie poolt järjekordselt maksimum.

			=Veebiteenuse ja Klientrakenduse retsensioon meeskonna Liisa ja Poisid poolt=

			==Veebiteenus==

			Kohe kui Vertigo Veebiteenuse projekt lahti võtta on märgata, et kõrgemal asuv projekti struktuur on paigas. Ilusti on kõik ära jaotatud 5 erinevasse projekti. <br>
			*Nagu kõikides paljudes projektides, on ka siin kasutatud Andres Käveri poolt tehtud Identity-t. <br>
			*Domain-is on lisaks Identity Model-itele ka veel lisaks 6 Vertigo meesokonna poolt koostatud mudelit. Annotatsioone on ilusti kasututatud seal, kus neid on kõige rohkem vaja. Üliselt on Domain üpris korralik. <br>
			*BLL on hästi koostatud. Kohe näha et seda tehes on kõik ilusti läbi mõeldud. Kõikide mudelite jaoks on vastavuses olemas ka DTO-d. <br>
			*DAL on täiseti hea ja mõistetav. DAL-is on kasutatud UOW-d. Kõik mida on vaja andmebaasiga suhtlemiseks, on ilusti olemas. Samuti on kasutatud Nlog, mis on alati hea.
			*WebService on korralikult tehtud ning vastab nõuetele. Kasutatud on ninjecti ja cache-imist. <br>
			Kokkuvõttes, kood on ilus ja puhas. Pole palju pahna ning meetodite kirjeldamiseks on ka koodi kommenteeritud. Hindeks paneks sellisele tööle suurepärane.

			==Klientrakendus==

			Esimese asjana mida märkab, on see, et meeskond on näinud vaeva ning teinud mitte ainult WPF-i vaid ka MVC. See on vägagi positiivne ning kindlasti lisab kogu projekti vaatusele pluss punkte. <br>
			*WPF nägi disaini poolest kena välja. Käima pannes tervitas vastu kohe keerlev pilt, mis tegi programmi vägagi ainulaadseks ja meeldejäävaks. Kuna mingit quest kasutaja võimalust see programm ei võimaldanud, tuli teha uus konto. Uue konto tegemine õnnestus ladusalt. Kontrollitud ka sai andmebaas , et kindel olla, et ikka kasutaja sinna tekiks. Kui kasutajaga oli sisse logitud siis muidu WPF-is navigeerimine oli vägagi lihtne. Mäng ise töötas, ning peale mängu mängimist oli ka näh tulemust leaderboards-is. See võis olla tingitud sellest, et ma olen seda programmi juba varem näinud, kuid siiski, usun, et iga uus kasutaja saaks ka hakkama selle kasutausega. <br>
			*MVC näeb sammuti täiesti adekvaatne välja. funktsionaalsused on kõik samad mis WPF-il. Sai ka kontrollitud turvalisust, ning see oli ilusti tagatud, kasutal ei olnud võimalik ligi pääseda infole, mida ta nägema ei peaks. <br>
			Selle koodi ülevaatamine ja katsetamine oli üpriski huvitav, kuna meeskonna Vertigo idee oli väga hästi teostatud. Vigu väga palju ei leidnud, osad kohad tekitasid küsimusi, kuid muidu oli kõik suurepärane

Ssaliste: /* Klientrakenduse retsensioon meeskonna R0ds poolt */

2015-06-14T19:51:02Z

Klientrakenduse retsensioon meeskonna R0ds poolt

← Older revision		Revision as of 22:51, 14 June 2015
Line 165:		Line 165:

	'''WPF'''		'''WPF'''

	Enne rakenduse käivitamist tuleb käivitada veebiteenus, seejärel valida lähteprojekt ning registreeruda ja sisse logida. Kasutaja luua tuleb, kuna veebiteenuse kasutajaga sisse logida ei olnud võimalik(vertigo/admin).		Enne rakenduse käivitamist tuleb käivitada veebiteenus, seejärel valida lähteprojekt ning registreeruda ja sisse logida. Kasutaja luua tuleb, kuna veebiteenuse kasutajaga sisse logida ei olnud võimalik(vertigo/admin).
	Kasutaja registreerimisega probleeme esialgu ei esinenud, kuid kui näiteks üritasime luua täpselt samade andmetega kasutajat uuesti, siis anti vähe üldine veateade“Oops, something went wrong...“. Sama lugu oli, kui jätsime täitmiseks antud väljad tühjaks või üritasime parooli vahetada, kas liiga lühikese parooli vastu või parooli vahetuse korral, vale vana parooli ette söötmise puhul. Kasutaja registreerimise korral tuleb ’kasutajanimeks’ panna e-mail, kuid sisselogimise korral küsitakse ’username’-i, sellise pisikese vea võiks ka ära parandada. Kui ma ei eksi, siis ei tööta ka e-maili sisestamise verifitseerimine 100%, kuna e-mail võib olla ka selline: „r@r“.		Kasutaja registreerimisega probleeme esialgu ei esinenud, kuid kui näiteks üritasime luua täpselt samade andmetega kasutajat uuesti, siis anti vähe üldine veateade“Oops, something went wrong...“. Sama lugu oli, kui jätsime täitmiseks antud väljad tühjaks või üritasime parooli vahetada, kas liiga lühikese parooli vastu või parooli vahetuse korral, vale vana parooli ette söötmise puhul. Kasutaja registreerimise korral tuleb ’kasutajanimeks’ panna e-mail, kuid sisselogimise korral küsitakse ’username’-i, sellise pisikese vea võiks ka ära parandada. Kui ma ei eksi, siis ei tööta ka e-maili sisestamise verifitseerimine 100%, kuna e-mail võib olla ka selline: „r@r“.
Line 171:		Line 172:

	'''MVC'''		'''MVC'''

	Väga palju midagi lisada ei ole, suhteliselt sarnane WPF rakendusega. Ainukesteks erinevusteks on see, et kasutaja ei saa oma parooli muuta millegipärast MVC-s ning e-maili verifitseerimine töötab veidi paremini kui WPF-l. Positiivne on see, et kasutaja ei saa vaatada skoori ega mängida(URL-i kaudu) ilma sisselogimata. Selle on taganud Rolli annotatsioonid kontrollerites.		Väga palju midagi lisada ei ole, suhteliselt sarnane WPF rakendusega. Ainukesteks erinevusteks on see, et kasutaja ei saa oma parooli muuta millegipärast MVC-s ning e-maili verifitseerimine töötab veidi paremini kui WPF-l. Positiivne on see, et kasutaja ei saa vaatada skoori ega mängida(URL-i kaudu) ilma sisselogimata. Selle on taganud Rolli annotatsioonid kontrollerites.

	'''Üldist'''		'''Üldist'''

	Koodi poolepealt on kena näha APIClient->Services->BaseService-t, kuna muudeti vajadusel koodi hilisem redigeerimine tunduvalt lihtsamaks ja vähendati ühe ja sama koodi korduvat ümberkopeerimist. Mudelitele on lisatud ka andmete annotatsioonid, mis piiravad erinevate Stringide pikkusi ning ei lase kasutajatel suvalisi romaani näiteks ’kategooria’ alla kirjutada(Required, MaxLength).		Koodi poolepealt on kena näha APIClient->Services->BaseService-t, kuna muudeti vajadusel koodi hilisem redigeerimine tunduvalt lihtsamaks ja vähendati ühe ja sama koodi korduvat ümberkopeerimist. Mudelitele on lisatud ka andmete annotatsioonid, mis piiravad erinevate Stringide pikkusi ning ei lase kasutajatel suvalisi romaani näiteks ’kategooria’ alla kirjutada(Required, MaxLength).

	'''Kokkuvõte'''		'''Kokkuvõte'''

	Jällegi tuleb tunnistada, et olenemate pisikestest vigades on Vertigo teinud suurepärast tööd. Hindeks meie poolt järjekordselt maksimum.		Jällegi tuleb tunnistada, et olenemate pisikestest vigades on Vertigo teinud suurepärast tööd. Hindeks meie poolt järjekordselt maksimum.

Ssaliste at 19:50, 14 June 2015

2015-06-14T19:50:32Z

← Older revision		Revision as of 22:50, 14 June 2015
Line 158:		Line 158:

	Meeskond Vertigo väärib igati tehtud töö eest maksimumpunkte. Kogu lahendus on kenasti koostatud, esineb vaid väiksemaid vigu, üldiselt ei ole mittemillegi üle väga nuriseda.		Meeskond Vertigo väärib igati tehtud töö eest maksimumpunkte. Kogu lahendus on kenasti koostatud, esineb vaid väiksemaid vigu, üldiselt ei ole mittemillegi üle väga nuriseda.

			=Klientrakenduse retsensioon meeskonna [https://wiki.itcollege.ee/index.php/Meeskond:R0ds R0ds] poolt=
			'''Vertigo – Klientrakendus'''

			Võrreldes nii mõnegi teise meeskonna tööga on näha, kui põhjalikult on asi valmis tehtud. Lisaks korralikule veebiteenusele on üles ehitatud nii WCF kui ka MVC rakenddused.

			'''WPF'''
			Enne rakenduse käivitamist tuleb käivitada veebiteenus, seejärel valida lähteprojekt ning registreeruda ja sisse logida. Kasutaja luua tuleb, kuna veebiteenuse kasutajaga sisse logida ei olnud võimalik(vertigo/admin).
			Kasutaja registreerimisega probleeme esialgu ei esinenud, kuid kui näiteks üritasime luua täpselt samade andmetega kasutajat uuesti, siis anti vähe üldine veateade“Oops, something went wrong...“. Sama lugu oli, kui jätsime täitmiseks antud väljad tühjaks või üritasime parooli vahetada, kas liiga lühikese parooli vastu või parooli vahetuse korral, vale vana parooli ette söötmise puhul. Kasutaja registreerimise korral tuleb ’kasutajanimeks’ panna e-mail, kuid sisselogimise korral küsitakse ’username’-i, sellise pisikese vea võiks ka ära parandada. Kui ma ei eksi, siis ei tööta ka e-maili sisestamise verifitseerimine 100%, kuna e-mail võib olla ka selline: „r@r“.
			Visuaalselt on rakendust väga ilus vaadata ja funktsionaalsuse kohapealt kõik nupud ja sisestused toimivad probleemideta.
			Lisaks ei teeks paha kui rakendusel oleks üleval väike menüü riba lisa funktsioonidega. Näiteks võimalus akent sulgeda, info pärimise jaoks rakenduse tegijatelt jms. Ka parooli uuendus ning väljalogimine võiks isegi sealtkaudu käia.

			'''MVC'''
			Väga palju midagi lisada ei ole, suhteliselt sarnane WPF rakendusega. Ainukesteks erinevusteks on see, et kasutaja ei saa oma parooli muuta millegipärast MVC-s ning e-maili verifitseerimine töötab veidi paremini kui WPF-l. Positiivne on see, et kasutaja ei saa vaatada skoori ega mängida(URL-i kaudu) ilma sisselogimata. Selle on taganud Rolli annotatsioonid kontrollerites.

			'''Üldist'''
			Koodi poolepealt on kena näha APIClient->Services->BaseService-t, kuna muudeti vajadusel koodi hilisem redigeerimine tunduvalt lihtsamaks ja vähendati ühe ja sama koodi korduvat ümberkopeerimist. Mudelitele on lisatud ka andmete annotatsioonid, mis piiravad erinevate Stringide pikkusi ning ei lase kasutajatel suvalisi romaani näiteks ’kategooria’ alla kirjutada(Required, MaxLength).

			'''Kokkuvõte'''
			Jällegi tuleb tunnistada, et olenemate pisikestest vigades on Vertigo teinud suurepärast tööd. Hindeks meie poolt järjekordselt maksimum.

Ddobrus: /* Veebiteenuse retsensioon meeskonna R0ds poolt */

2015-06-14T19:00:42Z