Jaatuli - Revision history

Mtammele at 20:51, 11 June 2018

2018-06-11T20:51:11Z

← Older revision		Revision as of 23:51, 11 June 2018
Line 294:		Line 294:
	</xsl:stylesheet>		</xsl:stylesheet>
	</source>		</source>

			=Retsensioonid=
			Retsensioonid meeskond Raamaturiiul kodutööde kohta.

			==API teenused ==



			Äriprojektina võiks idee kasutamist leida, eriti kui mõelda äri poolt edasi näiteks kasutajal oleks võimalik määrata omale lemmik autoreid ning kui mõni selle autori raamat müüki tuleb, saab teavituse.

			Tarkvara projekti ülesehitus on tuttav ja arusaadav, projekt on jagatud eraldi kaheksasse projekti BL – business layer, DAL.App.EF - rakenduse EF data access layer, DAL.EF, DAL.Interfaces, Domain, presentatsiooni projekt WebApp, üldpilt on hea, Helper projekt on veidi õnnetu eraldi projektis ning ilmselt taaskasutust mõnes teises projektis tähtsust ei omaks, seega võiks olla integreeritud mõne olemasoleva projekti juurde.

			Dataannotatsioonid on kasutusel nii domeeni objektide (andmebaasi piirnagud) kui andme ülekande objektides. Osad attribuudid on justkui vales kohas, näiteks Display atribuut domeeni objekti küljes, kuigi õigem koht oleks ilmselt vastav ViewModel.

			Näha on et on otsitud ja proovitud erinevaid lahendusi probleemide lahendamisel, õigesti on tähelepandud, et raamatu info uuendamisel ei ole korrektne lahendus seotud objektide kustutamine ning seejärel dto-st leiduvate seosobjektide taaslisamist. Ehk raamatuinfo uuendamisel kustutakse kõik raamatukategooriad ning seejärel lisatakse uuesti ainult valitud.

			EnumFactory tundub üleliigne ja keerukust lisav, eelkõige kasutatakse factory'id runtime uute objekide loomisel, enumite korral on tegu muutumatute numbri-nime seostega. Sama tulemuse saaks ilmselt lihtsa extensioonina lahendada. Enumite olemusest tingituna nende konverteerimine dtodeks lisaväärtust ei loo, enumid ei saa objektideks areneda.

			Mõned äriloogilised lahendused tahaksid ülevaatamist (analüüsi) ja võibolla uuesti tegemist, näiteks BookAuthorRepository's leidub meetod RemoveByBookId millega kustutakse raamatu järgi autor. Tegu on ilmselt modeleerimise veaga ning ehk on paslik nüüd meenutada Priit Raspeli juhendamisi, näiteks et autor võib eksisteerida ka ilma ühegi raamatuta. Raamatu kaudu ei tohiks autori kustutamine kuidagi sündida.

			InvoiceRepository's on meetod All, millele lisatud mittekohustuslik tagastatavate kirjete arvu piirav parameeter. Sellega võib kaasneda meetodi väärkasutust, kus sõltuvalt andmete hulgast teenus käitub erinevalt. Ennekõike tuleks leida võimalus teenusele filtri kaasa andmist kas klassi või LINQ kujul või kui ikkagi vaja piirata arvuliselt kirjete arvu luua eraldi meetod, näiteks Limit (LINQ pakub sisemiselt võimalust piirata kirjete arvu Take käsuga).

			Kui kategooria kustutamisel on see seotud raamatuga, siis antakse süsteemne Foreign Key viga, vastav kontroll võiks toimuda koodis ning tagastada kliendi poolele kasutatav veakood (nt Forbidden 403)

			Meeskonna wiki lehel toodud andmemudeli kirjelduses on mainitud et transpordimeetodid peidetakse kustamise asemel ära, tegelikkuses on realiseeritud tarnemeetodi reaalne kustutamine baasist DeleteShippingMethod

			ShippingMethod lisamine/muutmine API kaudu on ilmselt ülearune kuna ühtegi kaubavedu (va. "Tulen ise järele") ei ole võimalik kasutusele võtta ilma täiendavate muudatusteta rakenduse äriloogikas.

			BookDiscountService on implenteeritud baas osas, huvitav oleks olnud näha rohkem äriloogikat, näiteks allahindluse tagastamist etteantud päeval ja kliendiga.

			DBContexti kirjeldamisel on keelatud kaskaad kustutamine, võiks lisada selgituse mis eesmärki see täidab.

			Lõpetuseks võib veel märkida, et api koodist ei leidnud kasutajate põhist teenuste kasutamise logide loomist ega säilitamist.



			==XML Retsensioon==



			XML andmefail on korralikult koostatud kuid mõned parandused võiks teha. Olemas on rohkem kui minimaalne 4 loogilist dimensiooni ning on ka vähemalt 3-l dimensioonil olemas teisi atribuute peale ID. Millegi pärast on mõne dimensiooni ID pandud elementi mitte atribuudiks(nt: Author, BookDiscount). CDATA on kasutatud, kuid mitte igal pool näiteks Quality stringi oleks võinud ka CDATA sissse panna. XSD skeemfail valideerub vastu XMLi ning tundub olevat korras peale samade tähelepanekutega, mis olid XMLis.

			XMLi transformatsiooni fail on lihtne kuid toimiv, läbi käiakse kõik raamatud ning lisatakse järjest elemente nende külge. Transformatsiooniga oleks võinud rohkem atribuute panna, mitte neid rohkem elementideks panna. Tehtav XML oleks selle tõttu lühem ja paremini loetav.

			HTMLi transformatsiooni fail on hästi loetav, näidatakse kõiki raamatuid ja andmeid nende kohta ja need on selgesti eraldatud. Kasutatud on tingimusi, et ei tekiks arusaamatusi, tühjasid lünkasid või mõne koha rõhutamiseks(nt: tavahinna läbikriipsutamine soodustuse kehtimisel). Kasutatud on ka palju concat-i, mis teeb loetavuse kohati kergemaks ja faili ise väiksemaks. Võibolla raamatu kirjelduse peale oleks võinud panna väikese pealkiri, et tegu on lühikirjeldusega vms.

			Üldiselt on kõik tehtud korralikult, üheski polnud vigu, mis takistaks valideerimist. Ainukesed vead olid, mille parandamine teeks failide loetavuse paremaks(nt: ühtlane stiil, rohkem atribuutide kasutamist XML-iks transformatsioonis).



			==Klientrakendus==



			Klientrakenduse projekt on arusaadava ülesehitusega ning kood on hästi loetav.

			Klientrakendust on väga lihtne kasutada ning kasutaja leiab väikese vaevaga kõik vajaliku üles. Tavakasutaja saab süsteemis raamatuid vaadata ning otsida neidnii autori, pealkirja kui ka kirjastuse järgi. Raamatuid on võimalik lisada ostukorvi ning seejärel juba tellimus ära täita. Ostu sooritamisel valideeritakse korralikult sisestatud email ning ei ole võimalik sooritada tellimust ilmselgelt vigase emailiga. Probleemina võiks välja tuua selle, et raamatu lisamisel ostukorvi ei anta kliendile mingit tagasisidet ning tekib küsimus, kas toode lisandus ikka ostukorv. Samuti võiks tellimuse peal olla olemas võimalus ka ostukorvis olevate raamatute koguste muutmiseks.

			Administraatori poole pealt tuleb välja osa puudulikku funktsionaalsust. Ei tööta raamatute ja tellimuste uuendamine. Kasutajaliides on selleks küll olemas, aga muudatused ei salvestata. Veidi suurem probleem on uute raamatute lisamine. Kui puudulike andmetega salvestamise nuppu vajutada, siis kaob võimalus formaadi ja köite valimiseks. Kui kõik väljad on õigesti täidetud, siis annab süsteem veateate.
			Koodi vaadates jäi mulje et plaanis oli ka igale raamatule pilt lisada - seda lahendust oleks kindlasti olnud huvitav näha. Kahju, et seda realiseerida ei jõutud.

			Kokkuvõtvalt võib öelda, et suurem Klientrakendus oli visuaalselt ilus ja arusaadav. Tavakasutajale vajalik funktsionaalsusest oli klientrakenduses täiesti olemas ning süsteemis saab edukalt oste sooritada. Administraatori osa vajaks veel veidi parandamist.

Lkivisti: /* Kirjeldus */

2018-06-09T15:33:41Z

Kirjeldus

← Older revision		Revision as of 18:33, 9 June 2018
Line 15:		Line 15:

	*teenus - https://drive.google.com/open?id=1g3VXpNGNSPObUthM-lGL69QoIurW1eE2		*teenus - https://drive.google.com/open?id=1g3VXpNGNSPObUthM-lGL69QoIurW1eE2
			*teenus api logimisega - https://drive.google.com/open?id=1qy2oGpzCbx1LlYRk6M2FGYO9PTubAxBi
	*klient - https://drive.google.com/open?id=1vrR3XxfbraKrzRriLg-HdmRKNGFPwO0c		*klient - https://drive.google.com/open?id=1vrR3XxfbraKrzRriLg-HdmRKNGFPwO0c

Hamaeorg at 15:25, 9 June 2018

2018-06-09T15:25:28Z

← Older revision		Revision as of 18:25, 9 June 2018
Line 14:		Line 14:
	Eesmärk - kiirendada müüki ning tekitada mobiilne kassa. Saab lauast otse tellimusi teha. Tekib vähem vigu tellimuse vastuvõtmisel. Tulevased kliendid saavad ise ehitada endale sobiva liidese meie veebiteenusele. Personaalsed lahendused ning kliendi enda kujundus.		Eesmärk - kiirendada müüki ning tekitada mobiilne kassa. Saab lauast otse tellimusi teha. Tekib vähem vigu tellimuse vastuvõtmisel. Tulevased kliendid saavad ise ehitada endale sobiva liidese meie veebiteenusele. Personaalsed lahendused ning kliendi enda kujundus.

	teenus - https://drive.google.com/open?id=1g3VXpNGNSPObUthM-lGL69QoIurW1eE2		*teenus - https://drive.google.com/open?id=1g3VXpNGNSPObUthM-lGL69QoIurW1eE2
	klient - https://drive.google.com/open?id=1vrR3XxfbraKrzRriLg-HdmRKNGFPwO0c		*klient - https://drive.google.com/open?id=1vrR3XxfbraKrzRriLg-HdmRKNGFPwO0c

	==Funktsionaalsus==		==Funktsionaalsus==

Hamaeorg at 15:24, 9 June 2018

2018-06-09T15:24:59Z

← Older revision		Revision as of 18:24, 9 June 2018
Line 14:		Line 14:
	Eesmärk - kiirendada müüki ning tekitada mobiilne kassa. Saab lauast otse tellimusi teha. Tekib vähem vigu tellimuse vastuvõtmisel. Tulevased kliendid saavad ise ehitada endale sobiva liidese meie veebiteenusele. Personaalsed lahendused ning kliendi enda kujundus.		Eesmärk - kiirendada müüki ning tekitada mobiilne kassa. Saab lauast otse tellimusi teha. Tekib vähem vigu tellimuse vastuvõtmisel. Tulevased kliendid saavad ise ehitada endale sobiva liidese meie veebiteenusele. Personaalsed lahendused ning kliendi enda kujundus.

	~~kood~~ - https://drive.google.com/open?id=1g3VXpNGNSPObUthM-lGL69QoIurW1eE2		teenus - https://drive.google.com/open?id=1g3VXpNGNSPObUthM-lGL69QoIurW1eE2
			klient - https://drive.google.com/open?id=1vrR3XxfbraKrzRriLg-HdmRKNGFPwO0c

	==Funktsionaalsus==		==Funktsionaalsus==

Hamaeorg at 14:16, 9 June 2018

2018-06-09T14:16:13Z

← Older revision		Revision as of 17:16, 9 June 2018
Line 14:		Line 14:
	Eesmärk - kiirendada müüki ning tekitada mobiilne kassa. Saab lauast otse tellimusi teha. Tekib vähem vigu tellimuse vastuvõtmisel. Tulevased kliendid saavad ise ehitada endale sobiva liidese meie veebiteenusele. Personaalsed lahendused ning kliendi enda kujundus.		Eesmärk - kiirendada müüki ning tekitada mobiilne kassa. Saab lauast otse tellimusi teha. Tekib vähem vigu tellimuse vastuvõtmisel. Tulevased kliendid saavad ise ehitada endale sobiva liidese meie veebiteenusele. Personaalsed lahendused ning kliendi enda kujundus.

	https://drive.google.com/open?id=1g3VXpNGNSPObUthM-lGL69QoIurW1eE2		kood - https://drive.google.com/open?id=1g3VXpNGNSPObUthM-lGL69QoIurW1eE2

	==Funktsionaalsus==		==Funktsionaalsus==

Hamaeorg at 14:16, 9 June 2018

2018-06-09T14:16:01Z

← Older revision		Revision as of 17:16, 9 June 2018
Line 13:		Line 13:

	Eesmärk - kiirendada müüki ning tekitada mobiilne kassa. Saab lauast otse tellimusi teha. Tekib vähem vigu tellimuse vastuvõtmisel. Tulevased kliendid saavad ise ehitada endale sobiva liidese meie veebiteenusele. Personaalsed lahendused ning kliendi enda kujundus.		Eesmärk - kiirendada müüki ning tekitada mobiilne kassa. Saab lauast otse tellimusi teha. Tekib vähem vigu tellimuse vastuvõtmisel. Tulevased kliendid saavad ise ehitada endale sobiva liidese meie veebiteenusele. Personaalsed lahendused ning kliendi enda kujundus.

			https://drive.google.com/open?id=1g3VXpNGNSPObUthM-lGL69QoIurW1eE2

	==Funktsionaalsus==		==Funktsionaalsus==

Lkivisti: /* XML teisendus XSLT */

2018-06-09T08:56:45Z

XML teisendus XSLT

← Older revision		Revision as of 11:56, 9 June 2018
Line 230:		Line 230:
	<source lang="xml">		<source lang="xml">
	<?xml version="1.0" encoding="utf-8"?>		<?xml version="1.0" encoding="utf-8"?>
	<xsl:stylesheet version="1.0" xmlns:xsl="http://www.w3.org/1999/XSL/Transform"		<xsl:stylesheet version="1.0" xmlns:xsl="http://www.w3.org/1999/XSL/Transform" xmlns:msxsl="urn:schemas-microsoft-com:xslt" exclude-result-prefixes="msxsl">
	xmlns:msxsl="urn:schemas-microsoft-com:xslt" exclude-result-prefixes="msxsl"
	>
	<xsl:output method="xml" indent="yes"/>		<xsl:output method="xml" indent="yes"/>

	<xsl:template match="/Bills">		<xsl:template match="/Bills">
	<xsl:element name="Bills">		<xsl:element name="Bills">

Lkivisti: /* XML HTMLi (XSLT) */

2018-06-09T08:56:22Z

XML HTMLi (XSLT)

← Older revision		Revision as of 11:56, 9 June 2018
Line 177:		Line 177:
	<source lang="xml">		<source lang="xml">
	<?xml version="1.0" encoding="utf-8"?>		<?xml version="1.0" encoding="utf-8"?>
	<xsl:stylesheet version="1.0" xmlns:xsl="http://www.w3.org/1999/XSL/Transform"		<xsl:stylesheet version="1.0" xmlns:xsl="http://www.w3.org/1999/XSL/Transform" xmlns:msxsl="urn:schemas-microsoft-com:xslt" exclude-result-prefixes="msxsl">
	xmlns:msxsl="urn:schemas-microsoft-com:xslt" exclude-result-prefixes="msxsl"
	>
	<xsl:output method="html" indent="yes"/>		<xsl:output method="html" indent="yes"/>

	<xsl:template match="/Bills">		<xsl:template match="/Bills">
	<html>		<html>

Lkivisti: /* XML teisendus XSLT */

2018-06-09T08:55:51Z

XML teisendus XSLT

← Older revision		Revision as of 11:55, 9 June 2018
Line 231:		Line 231:

	=== XML teisendus XSLT ===		=== XML teisendus XSLT ===
			<source lang="xml">
	<?xml version="1.0" encoding="utf-8"?>		<?xml version="1.0" encoding="utf-8"?>
	<xsl:stylesheet version="1.0" xmlns:xsl="http://www.w3.org/1999/XSL/Transform"		<xsl:stylesheet version="1.0" xmlns:xsl="http://www.w3.org/1999/XSL/Transform"
Line 294:		Line 295:
	</xsl:template>		</xsl:template>
	</xsl:stylesheet>		</xsl:stylesheet>
			</source>

Lkivisti: /* XML HTMLi (XSLT) */

2018-06-09T08:55:31Z

XML HTMLi (XSLT)

← Older revision		Revision as of 11:55, 9 June 2018
Line 175:		Line 175:

	=== XML HTMLi (XSLT) ===		=== XML HTMLi (XSLT) ===
			<source lang="xml">
	<?xml version="1.0" encoding="utf-8"?>		<?xml version="1.0" encoding="utf-8"?>
	<xsl:stylesheet version="1.0" xmlns:xsl="http://www.w3.org/1999/XSL/Transform"		<xsl:stylesheet version="1.0" xmlns:xsl="http://www.w3.org/1999/XSL/Transform"
Line 227:		Line 228:
	</xsl:template>		</xsl:template>
	</xsl:stylesheet>		</xsl:stylesheet>
			</source>

	=== XML teisendus XSLT ===		=== XML teisendus XSLT ===