Talk:Iread: Difference between revisions
No edit summary |
|||
| Line 1: | Line 1: | ||
=Retsensioon meeskonna Iread veebiteenusele ja klientrakendusele meeskonnalt R0ds= | |||
Valisime retsenseerimiseks selle projekti teil ei ole ühtegi retsensiooni. | |||
'''Veebiteenuse retsensioon''' | '''Veebiteenuse retsensioon''' | ||
Koheselt on näha, et veebiteenuse solutionis on kasutusel 6 projekti. Projektis BLL on hakkavad silma vajalikud failid selle kihi jaoks. Ja positiivse poole pealt on DTO's näha kuidas on ilusti kirjutatud kõik MaxLength, Reqiured ning ka teised atribuudid. Oli ka näha mõningal määrab koodi kommenteerimist, kuigi seda oleks võinud olla igas failis. Service failid oleks võinud olla eraldi Service kaustas, et oleks silmale parem jälgida, aga see otseselt ei häiri. Tore oli näha, et ka Service failidele olid tehtud Interface failid. Me küll täpselt ei tea kas Servicitele eraldi oli vaja teha neid aga vähemalt on pingutatud. | Koheselt on näha, et veebiteenuse solutionis on kasutusel 6 projekti. Projektis BLL on hakkavad silma vajalikud failid selle kihi jaoks. Ja positiivse poole pealt on DTO's näha kuidas on ilusti kirjutatud kõik MaxLength, Reqiured ning ka teised atribuudid. Oli ka näha mõningal määrab koodi kommenteerimist, kuigi seda oleks võinud olla igas failis. Service failid oleks võinud olla eraldi Service kaustas, et oleks silmale parem jälgida, aga see otseselt ei häiri. Tore oli näha, et ka Service failidele olid tehtud Interface failid. Me küll täpselt ei tea kas Servicitele eraldi oli vaja teha neid aga vähemalt on pingutatud. | ||
| Line 15: | Line 18: | ||
Klientrakenduse retsensioon | '''Klientrakenduse retsensioon''' | ||
Revision as of 21:23, 14 June 2015
Retsensioon meeskonna Iread veebiteenusele ja klientrakendusele meeskonnalt R0ds
Valisime retsenseerimiseks selle projekti teil ei ole ühtegi retsensiooni.
Veebiteenuse retsensioon
Koheselt on näha, et veebiteenuse solutionis on kasutusel 6 projekti. Projektis BLL on hakkavad silma vajalikud failid selle kihi jaoks. Ja positiivse poole pealt on DTO's näha kuidas on ilusti kirjutatud kõik MaxLength, Reqiured ning ka teised atribuudid. Oli ka näha mõningal määrab koodi kommenteerimist, kuigi seda oleks võinud olla igas failis. Service failid oleks võinud olla eraldi Service kaustas, et oleks silmale parem jälgida, aga see otseselt ei häiri. Tore oli näha, et ka Service failidele olid tehtud Interface failid. Me küll täpselt ei tea kas Servicitele eraldi oli vaja teha neid aga vähemalt on pingutatud.
Projektis BookLibrary on näha kõik EF jaoks vajaminevad failid. Failides on kirjas kõik vajalikud propertid ning kõik on kommenteeritud väga eeskujulikult. Olemas on 7 olemit, järelikult on täidetud piirang milleks oli vähemalt 6 olemit veebiteenuses. Ka nendes failides on olemas nii MaxLength kui ka Required atribuudid, samuti on mõndades failides kasutusel Range atribuut.
DAL'is on näha õppejõu poolt nõutud Interface, Repository, Factory ja UOW kihid. Näha saab ka Andres Käveri poolt tehtud asju ning mida ta soojalt soovitas kasutada meie projektides. Samuti ei tundu, et oleks tehtud üleliigseid reposid. Interface failides on näha milliseid andmeid tahetakse edasi saata.
Projekt on External API Client on meie jaoks natukene segane. Tundub, et see kiht on vajalik selleks, et raamatute ja autorite andmeid saaks võtta ka välistest andmekogudest(mitte ise sisestatud andmed).
Projekt Identity on puhtalt üle võetud Andres Käveri tehtud näitest.
Viimases projektis nimega minuraamat on olemas kõik vajalik et solution töötaks internetis korralikult. Disin ei ole põhiline selle kodutöö raames ning seega on jäetud see võimalikult minimalistlikuks. Sisselogimine on tehtud väga turvaliseks, selle all mõtlen parooli piiranguid(peavad olema suured tähed, väiksed tähed ja numbrid). Oleks tore kui parooli piirangud oleks kohe registreerimise juures kirjas ja ei pea proovima kas sobib. Aga hea, et kasutajanimi ja emaili aadress ära ei kao kui parool ei vasta nõuetele. On olemas ka kontrollerid, kus on realiseeritud vajalikud meetodid ja samuti on väga palju kommenteerimisi.
Klientrakenduse retsensioon
