Talk:Meeskond:vertigo: Difference between revisions
No edit summary |
|||
| Line 32: | Line 32: | ||
</pre> | </pre> | ||
Muidu väga hea töö mis väärib maksimumpunkte. | Muidu väga hea töö mis väärib maksimumpunkte. | ||
==Veebiteenuse retsensioon meeskonna Taandarendajad poolt== | |||
Lisaks Identity kasutamisele on meeskond loonud 6 mudelit veel juurde. String tüüpi atribuutidel on ära määratud ka maksimaalsed pikkused, aga pole lisatud veateateid. See võib tekitada pärast andmete sisestamisel arusaamatusi, et miks ei õnnestu mingit stringi sisestada. Muidu on kõik atribuudid nimetatud korrektselt. | |||
WebAppEFContexti failis on näha, kuidas luuakse mängu jaoks andmebaas. Hetkel tundub, et kasutajatel on võimalus ainult mängu mängida, mitte uusi küsimusi postitada. See võiks muidu tore funktsionaalsus olla, mida kunagi juurde lisada. Artworkide filelocationi atribuudist jääb silma, et meeskond on pildid ise kuhugi üles laadinud, et vältida imelike olukordade teket, kus näiteks google-ist leitav pilt on ära kustutatud vahepeal. Kasutusele on võetud interface-d, repod ja UoW, nagu nõutud oli. Ühestki interface-ist ainult ei õnnestu mul leida custom meetodeid. | |||
Meeskond on loonud ka BLL projekti. Iga mudeli jaoks on eraldi DTO loodud, mis pole seotud virtuaalsete listidega. Kahju on näha, et taaskord toimuvad ainult ALL päringud ja ei ole mingit sorteerimist enne. Väikse andmebaasi puhul sellega probleemi ei teki, aga suurema projekti juures muudaks see päringute tegemise väga aeglaseks. Näiteks GetAllWorkNamesByCategory meetodi esialgse päringu võiks teha ära DAL-is, kus oleks võimalik peale DbSeti kohe sorteerida Where-iga. Hea on näha muidu, et meetoditele on ka kommentaarid lisatud. | |||
Web API-s on kasutatud ninjecti nagu nõutud. Siin tundub, et on juba rohkem vaeva nähtud. Kasutatud on ka admini teenuse turvamiseks, mis on suur pluss. Igas meetodis tagastatakse ka BLL-is loodud DTO-sid. | |||
Kokkuvõttes on loodud veebiteenus väga hea. Sellest andmebaasi päringute filtreerimisest võib hetkel mööda vaadata, sest hetkel pole seal palju kirjeid. Kõige rohkem meeldis mulle see, et BLL-i oli kasutatud otstarbekalt, mitte ei loodud seda lihtsalt sellepärast, et ta olemas oleks. | |||
Revision as of 11:21, 30 May 2015
Meeskonna Valar Morghulis retsensioon meeskonnale Vertigo
Meeskonnale Vertigo xml-faili sisuks on IMBD andmete sarnane filmiandmete nimekiri. Kindlasti võib see kasutamist leida ka antud aine järgnevate osade juures.
Main.xml - vastab igati nõuetele. Rohket CDATA kasutamist mina veaks ei pea erinevalt kuna filmide kirjeldused, kompaniide nimed jm võivad sisaldada keelatud märke.
Main.xsd- ga on palju vaeva nähtud, peale atribuudi id tüübi "unsignedInt"-iks muutmise on määratud ka andmetüüpide miinimumpikkusi ja (vajaduse korral) ka nende maksimaalset esinemiste arvu.
Actors.xslt - on kasutatud if-testi sorteerimaks näitlejaid atribuudi väärtuse järgi kahte gruppi.
ShorterMovies.xslt - on loovalt kasutatud if-testis XSLT funktsioone position() ja last(), et tekitada komadega eristatud nimekiri. Kui antud koht koodis oleks kirjutatud kujul
<xsl:if test="position() != last()">, </xsl:if>
mitte
<xsl:if test="position() != last()"> , </xsl:if>
siis oleks saanud vältida ka ebavajalikku tühikut enne koma.
SmallVersionXML.xslt - on valitud alamhulk Main.xml-is olevatest andmetest.
HTML transformatsioonidel (Actors.xslt ja ShorterMovies.xslt) on vaid üks viga. Neil pole deklareeritud dokumenditüüpi. Kui kasutada W3C HTML validaatorit http://validator.w3.org/check tuleb see selgelt välja.
Line 1, Column 1: no document type declaration; implying "<!DOCTYPE HTML SYSTEM>"
Sellest saaks üle lisades xslt-failidesse vahetult enne <html> elemendi algust järgmise rea
<xsl:text disable-output-escaping='yes'><!DOCTYPE html></xsl:text>
Muidu väga hea töö mis väärib maksimumpunkte.
Veebiteenuse retsensioon meeskonna Taandarendajad poolt
Lisaks Identity kasutamisele on meeskond loonud 6 mudelit veel juurde. String tüüpi atribuutidel on ära määratud ka maksimaalsed pikkused, aga pole lisatud veateateid. See võib tekitada pärast andmete sisestamisel arusaamatusi, et miks ei õnnestu mingit stringi sisestada. Muidu on kõik atribuudid nimetatud korrektselt.
WebAppEFContexti failis on näha, kuidas luuakse mängu jaoks andmebaas. Hetkel tundub, et kasutajatel on võimalus ainult mängu mängida, mitte uusi küsimusi postitada. See võiks muidu tore funktsionaalsus olla, mida kunagi juurde lisada. Artworkide filelocationi atribuudist jääb silma, et meeskond on pildid ise kuhugi üles laadinud, et vältida imelike olukordade teket, kus näiteks google-ist leitav pilt on ära kustutatud vahepeal. Kasutusele on võetud interface-d, repod ja UoW, nagu nõutud oli. Ühestki interface-ist ainult ei õnnestu mul leida custom meetodeid.
Meeskond on loonud ka BLL projekti. Iga mudeli jaoks on eraldi DTO loodud, mis pole seotud virtuaalsete listidega. Kahju on näha, et taaskord toimuvad ainult ALL päringud ja ei ole mingit sorteerimist enne. Väikse andmebaasi puhul sellega probleemi ei teki, aga suurema projekti juures muudaks see päringute tegemise väga aeglaseks. Näiteks GetAllWorkNamesByCategory meetodi esialgse päringu võiks teha ära DAL-is, kus oleks võimalik peale DbSeti kohe sorteerida Where-iga. Hea on näha muidu, et meetoditele on ka kommentaarid lisatud.
Web API-s on kasutatud ninjecti nagu nõutud. Siin tundub, et on juba rohkem vaeva nähtud. Kasutatud on ka admini teenuse turvamiseks, mis on suur pluss. Igas meetodis tagastatakse ka BLL-is loodud DTO-sid.
Kokkuvõttes on loodud veebiteenus väga hea. Sellest andmebaasi päringute filtreerimisest võib hetkel mööda vaadata, sest hetkel pole seal palju kirjeid. Kõige rohkem meeldis mulle see, et BLL-i oli kasutatud otstarbekalt, mitte ei loodud seda lihtsalt sellepärast, et ta olemas oleks.
