Talk:Iread: Difference between revisions

From ICO wiki
Jump to navigationJump to search
No edit summary
Line 19: Line 19:


'''Klientrakenduse retsensioon'''
'''Klientrakenduse retsensioon'''
Klientrakendust tööle pannes tekib kohe probleeme. Kui täidan kõik väljad ära tuleb error, millega öeldakse "One of the fields is probably not valid.". See ei ole just kõige parem veateade, mida kasutajale edasi anda. Proovisin teha kasutajat mitmete andmetega kuid alati on sama veateade. Kuigi probleemi lahendiks oli see, et tuleb panna enne API tööle ja seejärel klient. Seega oleks võinud selle algse veateate asemel anda midagi mis annab teada, et API ei tööta.
Klientrakenduses saab vaadata enda niiöelda lemmikute listi lisatud raamatuid ja neid seal kõikvõimalikul moel järjestada, on ka võimalus raamatuid sealt eemaldada. Olemas on ka funktsioon, millega saab andmebaasist raamatuid juurde otsida ja neid lisada lemmikute listi. Otsing töötab väga hästi. Häirib see, et rakenduses ei saa akna suurust muuta vaid peab kerima vasakule/paremale.
Failide poole pealt on näha, et oli kasu eelmise aasta C# ainest, kus õpetati kirjutama korralikku koodi mitte spagetti(on olemas ViewModel ja Model kaustad). Kui API solutionis olite väga agarad oma koodi kommenteerima, siis tundub, et klientrakenduses sellele enam nii suurt tähelepanu ei ole pööratud.

Revision as of 21:42, 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

Klientrakendust tööle pannes tekib kohe probleeme. Kui täidan kõik väljad ära tuleb error, millega öeldakse "One of the fields is probably not valid.". See ei ole just kõige parem veateade, mida kasutajale edasi anda. Proovisin teha kasutajat mitmete andmetega kuid alati on sama veateade. Kuigi probleemi lahendiks oli see, et tuleb panna enne API tööle ja seejärel klient. Seega oleks võinud selle algse veateate asemel anda midagi mis annab teada, et API ei tööta.

Klientrakenduses saab vaadata enda niiöelda lemmikute listi lisatud raamatuid ja neid seal kõikvõimalikul moel järjestada, on ka võimalus raamatuid sealt eemaldada. Olemas on ka funktsioon, millega saab andmebaasist raamatuid juurde otsida ja neid lisada lemmikute listi. Otsing töötab väga hästi. Häirib see, et rakenduses ei saa akna suurust muuta vaid peab kerima vasakule/paremale.

Failide poole pealt on näha, et oli kasu eelmise aasta C# ainest, kus õpetati kirjutama korralikku koodi mitte spagetti(on olemas ViewModel ja Model kaustad). Kui API solutionis olite väga agarad oma koodi kommenteerima, siis tundub, et klientrakenduses sellele enam nii suurt tähelepanu ei ole pööratud.