Hops - Revision history

Ekivisal at 18:36, 27 May 2013

2013-05-27T18:36:50Z

← Older revision		Revision as of 21:36, 27 May 2013
Line 356:		Line 356:
	Stiil on puhas, vajalikud ja ebaselged kohad ära kommenteeritud. Ära on märgitud viited kui kasutatud weebi ja õppejõudude materjali abi.		Stiil on puhas, vajalikud ja ebaselged kohad ära kommenteeritud. Ära on märgitud viited kui kasutatud weebi ja õppejõudude materjali abi.
	Meeldib, et lahendusele on lähenetud minimalistliku printsiibiga: Teha mustris võimalikult vähe muudatusi kui võimalik. Iga projekti Referencide nimekirja vaadates ei hakanud midagi kummalist silma, kõik on selliselt nagu peab.		Meeldib, et lahendusele on lähenetud minimalistliku printsiibiga: Teha mustris võimalikult vähe muudatusi kui võimalik. Iga projekti Referencide nimekirja vaadates ei hakanud midagi kummalist silma, kõik on selliselt nagu peab.



	=== Veebiteenuse retsensioon - meeskond Leemur===		=== Veebiteenuse retsensioon - meeskond Leemur===
Line 369:		Line 371:
	GET meetodi test: http://localhost:5189/api/currency/ aadress tagastab vastuseks		GET meetodi test: http://localhost:5189/api/currency/ aadress tagastab vastuseks

	<ArrayOfCurrency>		<ArrayOfCurrency>
	<Currency>		<Currency>
	<Created>2013-05-27T12:15:44.867</Created>		<Created>2013-05-27T12:15:44.867</Created>
Line 384:		Line 386:
	<Name>USD</Name>		<Name>USD</Name>
	</Currency>		</Currency>
	</ArrayOfCurrency>		</ArrayOfCurrency>

	GET (GetById) meetodi test: http://localhost:5189/api/currency/2 aadress tagastab vastuseks		GET (GetById) meetodi test: http://localhost:5189/api/currency/2 aadress tagastab vastuseks

	<Currency>		<Currency>
	<Created>2013-05-27T12:15:44.867</Created>		<Created>2013-05-27T12:15:44.867</Created>
	<CreatorUser i:nil="true"/>		<CreatorUser i:nil="true"/>
Line 394:		Line 396:
	<IsVisible>true</IsVisible>		<IsVisible>true</IsVisible>
	<Name>USD</Name>		<Name>USD</Name>
	</Currency>		</Currency>

	Lisades GET meetodi aadressile id väärtuse, mida baasist ei ole võimalik leida (näiteks http://localhost:5189/api/currency/3 ), siis saab vastuseks:		Lisades GET meetodi aadressile id väärtuse, mida baasist ei ole võimalik leida (näiteks http://localhost:5189/api/currency/3 ), siis saab vastuseks:

	<Currency i:nil="true"/>		<Currency i:nil="true"/>

	Seega midagi on tehtud, aga mitte liiga palju. Samas on tehtud osa realiseeritud nii, nagu koolis õpitud ja kodutöös ka nõuti. Ehk et eraldi on andmete ligipääsu kiht (DAL), mudelid ning teenuse enda osa.		Seega midagi on tehtud, aga mitte liiga palju. Samas on tehtud osa realiseeritud nii, nagu koolis õpitud ja kodutöös ka nõuti. Ehk et eraldi on andmete ligipääsu kiht (DAL), mudelid ning teenuse enda osa.
Line 410:		Line 412:
	Teenusele eraldi dokumentatsiooni lisatud ei ole. Otseselt see nõutud ei ole, seega puuduseks just lugeda ei saa. Samas, kuna kood ei ole ka sisuliselt üldse kommenteeritud, siis on see üsna suureks puuduseks.		Teenusele eraldi dokumentatsiooni lisatud ei ole. Otseselt see nõutud ei ole, seega puuduseks just lugeda ei saa. Samas, kuna kood ei ole ka sisuliselt üldse kommenteeritud, siis on see üsna suureks puuduseks.

	====4. kirjeldada puudusi===		====4. kirjeldada puudusi====

	Kasutajate tuvastamise ning halduse ja statistika osa on täielikult puudu. Seda, et enne teenuse kasutamist ja testimist tuleb konsooli rakendus käivitada, oleks võinud ka kusagil mainida, näiteks meeskonna wiki lehel.		Kasutajate tuvastamise ning halduse ja statistika osa on täielikult puudu. Seda, et enne teenuse kasutamist ja testimist tuleb konsooli rakendus käivitada, oleks võinud ka kusagil mainida, näiteks meeskonna wiki lehel.

Ekivisal at 18:35, 27 May 2013

2013-05-27T18:35:11Z

← Older revision		Revision as of 21:35, 27 May 2013
Line 356:		Line 356:
	Stiil on puhas, vajalikud ja ebaselged kohad ära kommenteeritud. Ära on märgitud viited kui kasutatud weebi ja õppejõudude materjali abi.		Stiil on puhas, vajalikud ja ebaselged kohad ära kommenteeritud. Ära on märgitud viited kui kasutatud weebi ja õppejõudude materjali abi.
	Meeldib, et lahendusele on lähenetud minimalistliku printsiibiga: Teha mustris võimalikult vähe muudatusi kui võimalik. Iga projekti Referencide nimekirja vaadates ei hakanud midagi kummalist silma, kõik on selliselt nagu peab.		Meeldib, et lahendusele on lähenetud minimalistliku printsiibiga: Teha mustris võimalikult vähe muudatusi kui võimalik. Iga projekti Referencide nimekirja vaadates ei hakanud midagi kummalist silma, kõik on selliselt nagu peab.

			=== Veebiteenuse retsensioon - meeskond Leemur===

			Retsenseerija on kasutanud testimiseks ja retsensiooni kirjutamiseks meeskonna tähtajaks tehtud osa („deadline“ branch).

			====1. töö vastavust esitatud tingimustele====

			Tehtud töö ei vasta praegu veel täielikult esitatud tingimustele, kuna kolmest alamülesandest on vaid üks realiseeritud, nimelt teenuse osa. Kasutajate tuvastamine ja haldus ning statistika osa on ei ole vähemalt hetkel avalikult kättesaadav.

			Teenuse poole pealt on realiseeritud Currency kontroller osaliselt – realiseeritud on GET meetodid, mis töötavad ning väljastavad andmeid XML vormingus. PUT, POST ja DELETE meetodid tagastavad NotImplemented teate.

			GET meetodi test: http://localhost:5189/api/currency/ aadress tagastab vastuseks

			<ArrayOfCurrency>
			<Currency>
			<Created>2013-05-27T12:15:44.867</Created>
			<CreatorUser i:nil="true"/>
			<CurrencyID>1</CurrencyID>
			<IsVisible>true</IsVisible>
			<Name>EUR</Name>
			</Currency>
			<Currency>
			<Created>2013-05-27T12:15:44.867</Created>
			<CreatorUser i:nil="true"/>
			<CurrencyID>2</CurrencyID>
			<IsVisible>true</IsVisible>
			<Name>USD</Name>
			</Currency>
			</ArrayOfCurrency>

			GET (GetById) meetodi test: http://localhost:5189/api/currency/2 aadress tagastab vastuseks

			<Currency>
			<Created>2013-05-27T12:15:44.867</Created>
			<CreatorUser i:nil="true"/>
			<CurrencyID>2</CurrencyID>
			<IsVisible>true</IsVisible>
			<Name>USD</Name>
			</Currency>

			Lisades GET meetodi aadressile id väärtuse, mida baasist ei ole võimalik leida (näiteks http://localhost:5189/api/currency/3 ), siis saab vastuseks:

			<Currency i:nil="true"/>

			Seega midagi on tehtud, aga mitte liiga palju. Samas on tehtud osa realiseeritud nii, nagu koolis õpitud ja kodutöös ka nõuti. Ehk et eraldi on andmete ligipääsu kiht (DAL), mudelid ning teenuse enda osa.

			====2. programmikoodi loetavus ning kommenteeritus====

			Koodis on palju välja kommenteerimisi, aga mitte liiga palju kommentaare. See teeb asjast aru saamise natuke keeruliseks. Samas on kood minimalistlik ja igasuguse liiasuseta.

			====3. dokumentatsiooni olemasolu====

			Teenusele eraldi dokumentatsiooni lisatud ei ole. Otseselt see nõutud ei ole, seega puuduseks just lugeda ei saa. Samas, kuna kood ei ole ka sisuliselt üldse kommenteeritud, siis on see üsna suureks puuduseks.

			====4. kirjeldada puudusi===

			Kasutajate tuvastamise ning halduse ja statistika osa on täielikult puudu. Seda, et enne teenuse kasutamist ja testimist tuleb konsooli rakendus käivitada, oleks võinud ka kusagil mainida, näiteks meeskonna wiki lehel.

Atsernik: /* TESTI KIRJELDUS 2 - Vastus JSON-formaadis */

2013-05-26T15:03:27Z

TESTI KIRJELDUS 2 - Vastus JSON-formaadis

← Older revision		Revision as of 18:03, 26 May 2013
Line 311:		Line 311:

	===== TESTI KIRJELDUS 2 - Vastus JSON-formaadis =====		===== TESTI KIRJELDUS 2 - Vastus JSON-formaadis =====
			HTTP päringu päise sisu:
			GET http://localhost:5189/api/currency HTTP/1.1
			Host: localhost:5189
			User-Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:20.0) Gecko/20100101 Firefox/20.0
			Accept: text/json

			HTTP vastus päringule:
			HTTP/1.1 200 OK
			Cache-Control: no-cache
			Pragma: no-cache
			Content-Type: text/json; charset=utf-8
			Expires: -1
			Server: Microsoft-IIS/8.0
			X-AspNet-Version: 4.0.30319
			X-SourceFiles: =?UTF-8?B?QzpcVXNlcnNcQW5kcmVcRGVza3RvcFxjdXJyZW5jeS1jb252ZXJ0ZXItbWFzdGVyXGN1cnJlbmN5LWNvbnZlcnRlci1tYXN0ZXJcQVNQTkVUTVZDU2VydmljZVxhcGlcY3VycmVuY3k=?=
			X-Powered-By: ASP.NET
			Date: Sun, 26 May 2013 14:39:57 GMT
			Content-Length: 292

			[
			{
			"CurrencyID": 1,
			"Name": "EUR",
			"IsVisible": true,
			"Created": "2013-05-26T16:07:49.987",
			"CreatorUser": null
			},
			{
			"CurrencyID": 2,
			"Name": "USD",
			"IsVisible": true,
			"Created": "2013-05-26T16:07:49.99",
			"CreatorUser": null
			}
			]

	===== Logimine =====		===== Logimine =====

Atsernik: /* TESTI KIRJELDUS 1 - Vastus XML-formaadis */

2013-05-26T14:59:23Z

TESTI KIRJELDUS 1 - Vastus XML-formaadis

← Older revision		Revision as of 17:59, 26 May 2013
Line 286:		Line 286:
	===== TESTI KIRJELDUS 1 - Vastus XML-formaadis =====		===== TESTI KIRJELDUS 1 - Vastus XML-formaadis =====

	HTTP - päis:		HTTP päringu päis:
	GET http://localhost:5189/api/currency HTTP/1.1		GET http://localhost:5189/api/currency HTTP/1.1
	Host: localhost:5189		Host: localhost:5189
	User-Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:20.0) Gecko/20100101 Firefox/20.0		User-Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:20.0) Gecko/20100101 Firefox/20.0
	Accept: text/html,application/xhtml+xml,application/xml;q=0.9,/;q=0.8		Accept: text/html,application/xhtml+xml,application/xml;q=0.9,/;q=0.8
	Accept-Language: en-US,en;q=0.5		Accept-Language: en-US,en;q=0.5
	Accept-Encoding: gzip, deflate		Accept-Encoding: gzip, deflate
	Connection: keep-alive		Connection: keep-alive

			HTTP vastus päringule:
			HTTP/1.1 200 OK
			Cache-Control: no-cache
			Pragma: no-cache
			Content-Type: application/xml; charset=utf-8
			Expires: -1
			Server: Microsoft-IIS/8.0
			X-AspNet-Version: 4.0.30319
			X-SourceFiles: =?UTF-8?B?QzpcVXNlcnNcQW5kcmVcRGVza3RvcFxjdXJyZW5jeS1jb252ZXJ0ZXItbWFzdGVyXGN1cnJlbmN5LWNvbnZlcnRlci1tYXN0ZXJcQVNQTkVUTVZDU2VydmljZVxhcGlcY3VycmVuY3k=?=
			X-Powered-By: ASP.NET
			Date: Sun, 26 May 2013 14:46:47 GMT
			Content-Length: 483

			<ArrayOfCurrency xmlns:i="http://www.w3.org/2001/XMLSchema-instance" xmlns="http://schemas.datacontract.org/2004/07/CurrencyModelsLibrary.Models"><Currency><Created>2013-05-26T16:07:49.987</Created><CreatorUser i:nil="true" /><CurrencyID>1</CurrencyID><IsVisible>true</IsVisible><Name>EUR</Name></Currency><Currency><Created>2013-05-26T16:07:49.99</Created><CreatorUser i:nil="true" /><CurrencyID>2</CurrencyID><IsVisible>true</IsVisible><Name>USD</Name></Currency></ArrayOfCurrency>

	===== TESTI KIRJELDUS 2 - Vastus JSON-formaadis =====		===== TESTI KIRJELDUS 2 - Vastus JSON-formaadis =====

Atsernik: /* TESTI KIRJELDUS 1 - Vastus XML-formaadis */

2013-05-26T14:50:02Z

TESTI KIRJELDUS 1 - Vastus XML-formaadis

← Older revision		Revision as of 17:50, 26 May 2013
Line 286:		Line 286:
	===== TESTI KIRJELDUS 1 - Vastus XML-formaadis =====		===== TESTI KIRJELDUS 1 - Vastus XML-formaadis =====

			HTTP - päis:
			GET http://localhost:5189/api/currency HTTP/1.1
			Host: localhost:5189
			User-Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:20.0) Gecko/20100101 Firefox/20.0
			Accept: text/html,application/xhtml+xml,application/xml;q=0.9,/;q=0.8
			Accept-Language: en-US,en;q=0.5
			Accept-Encoding: gzip, deflate
			Connection: keep-alive

	===== TESTI KIRJELDUS 2 - Vastus JSON-formaadis =====		===== TESTI KIRJELDUS 2 - Vastus JSON-formaadis =====

Atsernik: /* Teenusest endast */

2013-05-26T14:43:49Z

Teenusest endast

← Older revision		Revision as of 17:43, 26 May 2013
Line 283:		Line 283:

	Vaikimisi väljastab teenus andmed xml-formaadis. Kui GET päringu päisesse panna Accept "text/json", siis väljastab JSON formaadis.		Vaikimisi väljastab teenus andmed xml-formaadis. Kui GET päringu päisesse panna Accept "text/json", siis väljastab JSON formaadis.

			===== TESTI KIRJELDUS 1 - Vastus XML-formaadis =====


			===== TESTI KIRJELDUS 2 - Vastus JSON-formaadis =====

	===== Logimine =====		===== Logimine =====

Atsernik: /* Teenusest endast */

2013-05-26T14:42:07Z

Teenusest endast

← Older revision		Revision as of 17:42, 26 May 2013
Line 282:		Line 282:
	Osaliselt on teostatud CurrencyController. HomeController ja ValuesController on jäänud sinna projekti templeidist.		Osaliselt on teostatud CurrencyController. HomeController ja ValuesController on jäänud sinna projekti templeidist.

	Vaikimisi väljastab teenus andmed xml-formaadis.		Vaikimisi väljastab teenus andmed xml-formaadis. Kui GET päringu päisesse panna Accept "text/json", siis väljastab JSON formaadis.

	===== Logimine =====		===== Logimine =====

Atsernik: /* Web API teenus */

2013-05-26T14:38:28Z

Web API teenus

← Older revision		Revision as of 17:38, 26 May 2013
Line 281:		Line 281:

	Osaliselt on teostatud CurrencyController. HomeController ja ValuesController on jäänud sinna projekti templeidist.		Osaliselt on teostatud CurrencyController. HomeController ja ValuesController on jäänud sinna projekti templeidist.

			Vaikimisi väljastab teenus andmed xml-formaadis.

	===== Logimine =====		===== Logimine =====

Atsernik: /* Teenusest endast */

2013-05-26T14:36:33Z

Teenusest endast

← Older revision		Revision as of 17:36, 26 May 2013
Line 279:		Line 279:
	===== Teenusest endast =====		===== Teenusest endast =====
	Web API teenus töötab api/currency väljastab valuutade nimekirja. Paraku ei ole meetodit, mis kehtivad valuuta vahetuse kursid väljastaks. CRUD operatsioonidest on esindatud retsensiooni kirjutamise hetkel R-Read.		Web API teenus töötab api/currency väljastab valuutade nimekirja. Paraku ei ole meetodit, mis kehtivad valuuta vahetuse kursid väljastaks. CRUD operatsioonidest on esindatud retsensiooni kirjutamise hetkel R-Read.

			Osaliselt on teostatud CurrencyController. HomeController ja ValuesController on jäänud sinna projekti templeidist.

	===== Logimine =====		===== Logimine =====

Atsernik: /* Web API teenus */

2013-05-26T14:34:32Z

Web API teenus

← Older revision		Revision as of 17:34, 26 May 2013
Line 275:		Line 275:

	==== Web API teenus ====		==== Web API teenus ====
	~~Ennem~~ veebi käivitamist tuleb käivitada konsooli rakenduse, mis tekitab vajalikud andmed andmebaasi. Web API teenus töötab api/currency väljastab valuutade nimekirja. Paraku ei ole meetodit, mis kehtivad valuuta vahetuse kursid väljastaks. Ülesandes oli nõutud kasutajate audentimist, haldamist, kasutusstastikia pidamist. Valuuta vahetuse kursi kuvamine võib olla avalik teenus, seetõttu sellel nõudel suurt praktilist rakendust pole, aga miinusena mul tuleb see välja tuua :(.		Esimese asjana märkasin ja võibolla abiks ka õppejõule, et ennem veebi käivitamist tuleb käivitada konsooli rakenduse, mis tekitab vajalikud andmed andmebaasi. Seejärel alles saab asuma WEB API testimist.

			===== Teenusest endast =====
			Web API teenus töötab api/currency väljastab valuutade nimekirja. Paraku ei ole meetodit, mis kehtivad valuuta vahetuse kursid väljastaks. CRUD operatsioonidest on esindatud retsensiooni kirjutamise hetkel R-Read.

			===== Logimine =====
			Logimise jaoks oli eeldus loodud. Vajalikud koodiread failides Filters/WebAPIActionLoggingAttribute.cs ja Modules/MyModule1.cs oli välja kommenteeritud. Retsensenseeriad ei hakanud kommentaare eemaldama ning logimise osa testima.

			===== Kasutajate audentimisest ja autoriseerimisest =====
			Ülesandes oli nõutud kasutajate audentimist, haldamist, kasutusstastikia pidamist. Valuuta vahetuse kursi kuvamine võib olla avalik teenus, seetõttu sellel nõudel suurt praktilist rakendust pole, aga miinusena mul tuleb see välja tuua :(.

	==== Kodeerimise stiil ja parimad praktikad ====		==== Kodeerimise stiil ja parimad praktikad ====
	Stiil on puhas, vajalikud ja ebaselged kohad ära kommenteeritud. Ära on märgitud viited kui kasutatud weebi ja õppejõudude materjali abi.		Stiil on puhas, vajalikud ja ebaselged kohad ära kommenteeritud. Ära on märgitud viited kui kasutatud weebi ja õppejõudude materjali abi.
	Meeldib, et lahendusele on lähenetud minimalistliku printsiibiga: Teha mustris võimalikult vähe muudatusi kui võimalik. Iga projekti Referencide nimekirja vaadates ei hakanud midagi kummalist silma, kõik on selliselt nagu peab.		Meeldib, et lahendusele on lähenetud minimalistliku printsiibiga: Teha mustris võimalikult vähe muudatusi kui võimalik. Iga projekti Referencide nimekirja vaadates ei hakanud midagi kummalist silma, kõik on selliselt nagu peab.