Talk:Liisa ja Poisid - Revision history

Jmaemets: /* Retsensioonid meeskonnalt TrainSmart */

2015-06-15T19:46:05Z

Retsensioonid meeskonnalt TrainSmart

← Older revision		Revision as of 22:46, 15 June 2015
Line 77:		Line 77:
	'''Veebiteenus ja Klientrakendus'''		'''Veebiteenus ja Klientrakendus'''

	Mees-/Naiskond Liisa ja poisid tegid selle aine raames rakenduse "Sportify", mis kujutab endast treeninpäevikut. Veebiteenuse pool tundub olevat tehtud viisakalt, kasutades erinevaid kihte.		Mees-/Naiskond Liisa ja poisid tegid selle aine raames rakenduse "Sportify", mis kujutab endast treeninpäevikut. Veebiteenuse pool tundub olevat tehtud viisakalt, kasutades erinevaid kihte. Mudelid on ilusti ära annoteeritud, mis on kindlasti kasulik, samas minu silmis segab natuke koodi loetavust.
	Kontrolleri ja repo vahel on olemas service layer, kus on kirjeldatud erinevad meetodid DTO-de ja listide loomiseks, et neid kontrolleris välja kutsuda. Kontrollerid on suhteliselt lihtsad, ära annoteeritud, tagastatakse ka http status koode, mis tulevad klientrakendust ehitades kindlasti kasuks. Web Api puhul on tagatud turvalisus, kasutades Identityt ning tokenit. Kõik kontrollerid on klassipõhiselt märgistatud [Authorize] annotatsiooniga, samas ei paista välja, et mõni päring oleks piiratud näiteks admin õigustega.		Kontrolleri ja repo vahel on olemas service layer, kus on kirjeldatud erinevad meetodid DTO-de ja listide loomiseks, et neid kontrolleris välja kutsuda. Kontrollerid on suhteliselt lihtsad, ära annoteeritud, tagastatakse ka http status koode, mis tulevad klientrakendust ehitades kindlasti kasuks. Web Api puhul on tagatud turvalisus, kasutades Identityt ning tokenit. Kõik kontrollerid on klassipõhiselt märgistatud [Authorize] annotatsiooniga, samas ei paista välja, et mõni päring oleks piiratud näiteks admin õigustega.
	Kasutatud on UOW-d, mida kasutati Identity puhul.		Kasutatud on UOW-d, mida kasutati Identity puhul.

Jmaemets: /* Retsensioonid meeskonnalt TrainSmart */

2015-06-15T19:38:57Z

Retsensioonid meeskonnalt TrainSmart

@@ Line 77: / Line 77: @@
 '''Veebiteenus ja Klientrakendus'''
-Mees-/Naiskond Liisa ja poisid tegid selle aine raames rakenduse "Sportify", mis kujutab endast
+Mees-/Naiskond Liisa ja poisid tegid selle aine raames rakenduse "Sportify", mis kujutab endast treeninpäevikut. Veebiteenuse pool tundub olevat tehtud viisakalt, kasutades erinevaid kihte.
 Kontrolleri ja repo vahel on olemas service layer, kus on kirjeldatud erinevad meetodid DTO-de ja listide loomiseks, et neid kontrolleris välja kutsuda. Kontrollerid on suhteliselt lihtsad, ära annoteeritud, tagastatakse ka http status koode, mis tulevad klientrakendust ehitades kindlasti kasuks. Web Api puhul on tagatud turvalisus, kasutades Identityt ning tokenit. Kõik kontrollerid on klassipõhiselt märgistatud [Authorize] annotatsiooniga, samas ei paista välja, et mõni päring oleks piiratud näiteks admin õigustega.
 Kasutatud on UOW-d, mida kasutati Identity puhul.
-Klientrakendust kahjuks rohkem, kui registreerimine testida ei saanud, kuna Api projekti minu VS
+Klientrakendust kahjuks rohkem, kui registreerimine testida ei saanud, kuna Api projekti minu VS-s millegipärast ei avanud, ja seega ei töötanud ka veebiteenus. Koodi poolt vaadates, tundub kasutaja jaoks olevat küllaltki mugav ja arusaadav UI. Üllatav oli full screen UI. Natukest ebameeldivust valmistas rakenduse sulgemine, kuna puudus Exit nupp kui selline. Õnneks jooksis mul rakendus sisselogimisel kokku, kuna puudus veebiteenusega ühendus, ühtlasi sain ka rakendusest välja.
 Üldiselt on klientrakenduse kood hästi organiseeritud ja ära kihistatud. Mudelid on pandud vastavusse JSON-ist tulevate andmetega, kasutatud on service-i kihti veebiteenusega suhtlemiseks.
-millegipärast ei avanud, ja seega ei töötanud ka veebiteenus. Koodi poolt vaadates, tundub
+Kuna vaateid on ~7, siis oleks võinud vaatemudeleid olla rohkem kui 1, mis tagaks koodi parema haldamise. Klientrakendust testimata tundub, et erinevate vaadete vahel liikumiseks ei tekitata lihtsalt uusi aknaid kuskile, vaid on kasutatud Switcher klassi, mis vaadete vahetamisega tegeleb.
 Üldiselt on kood hästi struktureeritud ja vastab aine läbimise nõuetele, samas kommentaaride poole pealt on asi suhteliselt nutune.

Jmaemets at 19:37, 15 June 2015

2015-06-15T19:37:11Z

← Older revision		Revision as of 22:37, 15 June 2015
Line 70:		Line 70:
	Klientrakendus on loodud WPFi näol. Rakendus on üles ehitatud kasutades häid tavasi ning C# aines õpitud MVVMi, mis muudab projekti väga hästi hallatavaks. Kasutatud on enamikke loodud teenuseid ning seeläbi on klientrakenduses funktsionaalsust küll. Ei õnnestunud taaskord leida kommentaare, kood on lihtne, kuid vöiks sellegipoolest veidi kommenteerida. Pannes esmakordselt rakenduse käima, ei saanud ma seda kuidagi enam kinni, sest puudus vastav ristike või nupp login ja register vaates. Lisaks sellele jäi silma, et vaateid on päris palju, kuid olemas on ainult üks vaatemudel.		Klientrakendus on loodud WPFi näol. Rakendus on üles ehitatud kasutades häid tavasi ning C# aines õpitud MVVMi, mis muudab projekti väga hästi hallatavaks. Kasutatud on enamikke loodud teenuseid ning seeläbi on klientrakenduses funktsionaalsust küll. Ei õnnestunud taaskord leida kommentaare, kood on lihtne, kuid vöiks sellegipoolest veidi kommenteerida. Pannes esmakordselt rakenduse käima, ei saanud ma seda kuidagi enam kinni, sest puudus vastav ristike või nupp login ja register vaates. Lisaks sellele jäi silma, et vaateid on päris palju, kuid olemas on ainult üks vaatemudel.
	Lühikese jutu kokkuvõtteks ütleks, et kõik on nõudmiste kohaselt struktureeritud, funktsionaalsust on kasutajate halduse näol puudu, kuid sellegipoolest asi töötab ja on piisav.		Lühikese jutu kokkuvõtteks ütleks, et kõik on nõudmiste kohaselt struktureeritud, funktsionaalsust on kasutajate halduse näol puudu, kuid sellegipoolest asi töötab ja on piisav.




			=Retsensioonid meeskonnalt [https://wiki.itcollege.ee/index.php/TrainSmart TrainSmart] =
			'''Veebiteenus ja Klientrakendus'''

			Mees-/Naiskond Liisa ja poisid tegid selle aine raames rakenduse "Sportify", mis kujutab endast

			treeninpäevikut. Veebiteenuse pool tundub olevat tehtud viisakalt, kasutades erinevaid kihte.
			Kontrolleri ja repo vahel on olemas service layer, kus on kirjeldatud erinevad meetodid DTO-de ja

			listide loomiseks, et neid kontrolleris välja kutsuda. Kontrollerid on suhteliselt lihtsad, ära

			annoteeritud, tagastatakse ka http status koode, mis tulevad klientrakendust ehitades kindlasti

			kasuks. Web Api puhul on tagatud turvalisus, kasutades Identityt ning tokenit. Kõik kontrollerid

			on klassipõhiselt märgistatud [Authorize] annotatsiooniga, samas ei paista välja, et mõni päring

			oleks piiratud näiteks admin õigustega.
			Kasutatud on UOW-d, mida kasutati Identity puhul.

			Klientrakendust kahjuks rohkem, kui registreerimine testida ei saanud, kuna Api projekti minu VS

			millegipärast ei avanud, ja seega ei töötanud ka veebiteenus. Koodi poolt vaadates, tundub

			kasutaja jaoks olevat küllaltki mugav ja arusaadav UI. Üllatav oli full screen UI. Natukest ebameeldivust valmistas rakenduse sulgemine, kuna puudus Exit nupp kui selline. Õnneks jooksis mul rakendus sisselogimisel kokku, kuna puudus veebiteenusega ühendus, ühtlasi sain ka rakendusest välja.
			Üldiselt on klientrakenduse kood hästi organiseeritud ja ära kihistatud. Mudelid on pandud

			vastavusse JSON-ist tulevate andmetega, kasutatud on service-i kihti veebiteenusega suhtlemiseks.
			Kuna vaateid on ~7, siis oleks võinud vaatemudeleid olla rohkem kui 1, mis tagaks koodi parema

			haldamise. Klientrakendust testimata tundub, et erinevate vaadete vahel liikumiseks ei

			tekitata lihtsalt uusi aknaid kuskile, vaid on kasutatud Switcher klassi, mis vaadete

			vahetamisega tegeleb.
			Üldiselt on kood hästi struktureeritud ja vastab aine läbimise nõuetele, samas kommentaaride poole pealt on asi suhteliselt nutune.

Ksiiner at 19:29, 15 June 2015

2015-06-15T19:29:04Z

← Older revision		Revision as of 22:29, 15 June 2015
Line 44:		Line 44:

	Kokkuvõtteks võib öelda, et projekt on viisakas, kuid jääb tunne, et on jäetud päeva pealt pooleli ning jäetud pisiasjad lisamata/parandamata(detailidesse pole süübitud).		Kokkuvõtteks võib öelda, et projekt on viisakas, kuid jääb tunne, et on jäetud päeva pealt pooleli ning jäetud pisiasjad lisamata/parandamata(detailidesse pole süübitud).



			=Retsensioonid meeskonnalt KTT=
			'''Veebiteenus'''

			SportifyLibrary – olemas on täpselt 6 andmebaasi olemit, mis vastab projekti nõuetele, lisaks sellele on olemas ka identity-ga kaasnevad klassid. Klassides on ilusti märgitud peale kõik vajalikud atribuudid ning lisatud ka display atribuudid, mis muudavad vaated ilusamaks.

			DAL - on kasutatud kõiki vajalikke mustreid : Interfaced, repod, helpreid, UOW ja identity-ga kaasnevad klassid. Kuna üldjoontes DAL-i puhul ongi tegu suhteliselt standardse asjaga, mis ülesehitluselt väga ei erine, siis siin ei ole midagi kommenteerida, kõik vajalik on olemas ja nõuded täidetud.

			BLL – projekt on aine raames õpetatud heade kommete kohaselt struktureeritud. DTO-d , servicid ja muud suuremad osad on jaotatud eraldi kaustadesse, mis on väga hea. Servicite näol on olemas terve CRUD, ühtegi custom meetodit silma ei hakanud. Üldjoontes on kõik vajalik tehtud ning korrapäraselt. Ainukeseks norimiskohaks võiks olla ObjectDTOFactory kaust, mille kõik klassid oleks võinud väga edukalt ühte klassi, näiteks ModelFactory-sse kirjutada.

			Identity – tegu on Andres Käveri poolt antud koodiga, mis on kasutusel kõikidel meeskondadel ning siin ei ole kommenteerida midagi.

			DummyDbData – projekti näol on tegemist lihtsalt testandmete lisamisega, mis arvatavasti lihtsustas ja kiirendas testimist.

			SportifyWebApi – meeskonna kodulehel on öeldud, et tegu on API-ga, mis ei tööta ning kasutada soovitatakse WebApiAppi. Sellisel juhul oleks vöinud selle projektist üldse eemaldada.

			WebApiApp – veebiteenus, millest vaatavad vastu kõik CRUD meetodid vajalike olemitega, mis projektis olemas on. Mingeid muid keerulisemaid meetodeid silma ei hakka, kuid järelikult polnud neid siis vaja. Kuna CRUD on tehtud ja üldjoontes on kasutusel vajalikud olemid, siis vöib öelda, et tööd on tehtud piisavalt ning selle käigus on kindlasti omandatud väärtuslikke teadmisi.

			Veebiteenuse kokkuvõtteks ütleks, et aine raames seatud eesmärgid on täidetud, küll minimaalselt, kuid siiski täidetud. Silma jäi see, et üheski kohas ei olnud kommentaare, mitte, et tegu oleks väga keeruka asjaga, mida põhjalikult kommenteerima peaks, kuid hea tava oleks üht-teist siiski kirjutada.

			'''Klientrakendus'''

			Klientrakendus on loodud WPFi näol. Rakendus on üles ehitatud kasutades häid tavasi ning C# aines õpitud MVVMi, mis muudab projekti väga hästi hallatavaks. Kasutatud on enamikke loodud teenuseid ning seeläbi on klientrakenduses funktsionaalsust küll. Ei õnnestunud taaskord leida kommentaare, kood on lihtne, kuid vöiks sellegipoolest veidi kommenteerida. Pannes esmakordselt rakenduse käima, ei saanud ma seda kuidagi enam kinni, sest puudus vastav ristike või nupp login ja register vaates. Lisaks sellele jäi silma, et vaateid on päris palju, kuid olemas on ainult üks vaatemudel.
			Lühikese jutu kokkuvõtteks ütleks, et kõik on nõudmiste kohaselt struktureeritud, funktsionaalsust on kasutajate halduse näol puudu, kuid sellegipoolest asi töötab ja on piisav.

Tpaaro: /* Klientrakenduse retsensioon meeskonnalt Lupardid */

2015-06-15T18:30:22Z

Klientrakenduse retsensioon meeskonnalt Lupardid

← Older revision		Revision as of 21:30, 15 June 2015
Line 40:		Line 40:
	* Ma eeldan, et vanuse peaks automaatselt arvutama, kuid seda ta ei tee,		* Ma eeldan, et vanuse peaks automaatselt arvutama, kuid seda ta ei tee,
	tagastab arvu 0.		tagastab arvu 0.
	* ~~Sulgemisnupp ei teeks paha~~.		* Rakenduse sulgemisfunktsiooni puudumine.


	Kokkuvõtteks võib öelda, et projekt on viisakas, kuid jääb tunne, et on jäetud päeva pealt pooleli ning jäetud pisiasjad lisamata/parandamata(detailidesse pole süübitud).		Kokkuvõtteks võib öelda, et projekt on viisakas, kuid jääb tunne, et on jäetud päeva pealt pooleli ning jäetud pisiasjad lisamata/parandamata(detailidesse pole süübitud).

Tpaaro: /* Klientrakenduse retsensioon meeskonnalt Lupardid */

2015-06-15T18:29:18Z

Klientrakenduse retsensioon meeskonnalt Lupardid

@@ Line 22: / Line 22: @@
 ==Klientrakenduse retsensioon meeskonnalt [https://wiki.itcollege.ee/index.php/Lupardid_VR2 Lupardid]==
 * Registreerides ei tulnud veateateid - tekitas palju frustratsiooni ja segadust, sest ei saanud aru, kas kasutaja sai loodud või mitte.
 * Treeningut sisestades ei saanud treeningu kestvust valida(combobox tühi) ning
@@ Line 30: / Line 42: @@
 * Sulgemisnupp ei teeks paha.
-Pole kasutatud UOWd(on olemas, ninjectis sõltuvused süstitud,
+Kokkuvõtteks võib öelda, et projekt on viisakas, kuid jääb tunne, et on jäetud päeva pealt pooleli ning jäetud pisiasjad lisamata/parandamata(detailidesse pole süübitud).

Tpaaro: /* Retsensioon meeskonnalt Lupardid */

2015-06-15T18:25:12Z

Retsensioon meeskonnalt Lupardid

← Older revision		Revision as of 21:25, 15 June 2015
Line 19:		Line 19:
	XML transformatsioon on võib-olla liiga lihtne aga kuna selle kohta mingeid nõudeid ei olnud siis ütleks, et nutikas lähenemine.		XML transformatsioon on võib-olla liiga lihtne aga kuna selle kohta mingeid nõudeid ei olnud siis ütleks, et nutikas lähenemine.
	Kõik muu on hästi.		Kõik muu on hästi.

			==Klientrakenduse retsensioon meeskonnalt [https://wiki.itcollege.ee/index.php/Lupardid_VR2 Lupardid]==

			* Registreerides ei tulnud veateateid - tekitas palju frustratsiooni ja segadust, sest ei saanud aru, kas kasutaja sai loodud või mitte.
			* Treeningut sisestades ei saanud treeningu kestvust valida(combobox tühi) ning
			salvestades jookseb kogu rakendus kotti. Comboboxi asemel oleks võinud kasutada lihtsat textboxi, mis tundub palju loogilisem ja mugavam kasutajale.
			* Üks viewmodel kõige jaoks - veidi halb, võiks olla organiseeritum.
			* Ma eeldan, et vanuse peaks automaatselt arvutama, kuid seda ta ei tee,
			tagastab arvu 0.
			* Sulgemisnupp ei teeks paha.

			Pole kasutatud UOWd(on olemas, ninjectis sõltuvused süstitud,
			kuid pole kuskil kasutatud). Tundub siiski üpris puhtalt ja
			loogiliselt üles ehitatud projekt.
			Arvatavasti ei hakatud UOWd kasutama, kuna ei osatud seda korrektselt rakendada ning jäeti see osa lihtsalt implementeerimata.

			DTO’d on olemas, viisakad. Iga DTO jaoks loodud ka DTO factory.
			Service’sid sisaldavad custom meetodeid. Igal service’l olemas interface’id.

			Olemeid on 6. Olemite väljadel atribuudid kirjeldatud(stringi pikkused, kuvatavad nimed), täpselt nagu nõutud oli.

Ttriisa: /* Retsensioon meeskonnalt Lupardid */

2015-03-14T15:21:43Z

Retsensioon meeskonnalt Lupardid

← Older revision		Revision as of 18:21, 14 March 2015
Line 3:		Line 3:

	XML vastab nõuetele: sügavust on üle miinimumi ning atribuudid on olemas. Nii XML, XSD eraldi kui ka XML XSD põhjal valideeruvad.		XML vastab nõuetele: sügavust on üle miinimumi ning atribuudid on olemas. Nii XML, XSD eraldi kui ka XML XSD põhjal valideeruvad.
	CDATA kasutust on üleliigselt, eriti kohtades kus XSD määrab ära selle, et elemendi väärtus on arvu kujul. Miks on aasta, kuu ja päev eraldi elemendid mitte atribuudina? Arvutid pole nii aeglased, et teatud kuu kanded kõikidest kannetest üles otsida. Lisaks pole "aasta" nimeline element semantiline (nimi ei kirjelda sisu). Kuna minimaalne sügavus jääb siiski täidetuks tunduks mõistlikum kui oleks järgnevalt:		CDATA kasutust on üleliigselt, eriti kohtades kus XSD määrab ära selle, et elemendi väärtus on arvu kujul. Miks on aasta, kuu ja päev eraldi elemendid mitte atribuudina? Arvutid pole nii aeglased, et teatud kuu kanded kõikidest kannetest üles otsida. Lisaks pole "aasta" nimeline element semantiline (nimi ei kirjelda sisu). Antud lähenemine pole vale aga pole ka mõistlik. Kuna minimaalne sügavus jääb siiski täidetuks tunduks mõistlikum kui oleks järgnevalt:
	<code>		<code>
	<practices>		<practices>

Ttriisa: /* Retsensioon meeskonnalt Lupardid */

2015-03-14T15:19:05Z

Retsensioon meeskonnalt Lupardid

← Older revision		Revision as of 18:19, 14 March 2015
Line 16:		Line 16:
	'''Transformatsioonid'''		'''Transformatsioonid'''

	HTML'i genereerimine töötab, andmed näidatakse välja. Küll aga on HTML'i enda struktuur väga kehv ning ei valideeru, see polnud vist küll ülesandes vajalik aga et te teaksite. Listide sisse ei panda päiseid ega ~~suure~~ hulki tekste, lisaks ilma põhjuseta ei pea kõiki elemente ümbritsema teiste elementidega (antud juhul ~~paragraafi~~ elementidega).		HTML'i genereerimine töötab, andmed näidatakse välja. Küll aga on HTML'i enda struktuur väga kehv ning ei valideeru, see polnud vist küll ülesandes vajalik aga et te teaksite. Listide sisse ei panda päiseid ega suuri hulki tekste, lisaks ilma põhjuseta ei pea kõiki elemente ümbritsema teiste elementidega (antud juhul paragrahvi elementidega).
	XML transformatsioon on võib-olla liiga lihtne aga kuna selle kohta mingeid nõudeid ei olnud siis ütleks, et nutikas lähenemine.		XML transformatsioon on võib-olla liiga lihtne aga kuna selle kohta mingeid nõudeid ei olnud siis ütleks, et nutikas lähenemine.
	Kõik muu on hästi.		Kõik muu on hästi.

Ttriisa: /* Retsensioon meeskonnalt Lupardid */

2015-03-14T15:16:59Z

Retsensioon meeskonnalt Lupardid

← Older revision		Revision as of 18:16, 14 March 2015
Line 9:		Line 9:
	<type><![CDATA[Zumba]]></type>		<type><![CDATA[Zumba]]></type>
	<lostCalories>1980</lostCalories>		<lostCalories>1980</lostCalories>
	~~</practice>~~
	~~<practice ...>~~
	~~...~~
	</practice>		</practice>
	</practices>		</practices>