Talk:Meeskond: Premium

Retsensioon meeskonna Premium projekti HomeLibrary analüüsile

https://wiki.itcollege.ee/index.php/Meeskond:_Premium

Koostaja: Meeskond Scary Spooky Skeletons

On näha, et loodava rakenduse funktsionaalsus on detailideni läbi mõeldud. Loodud on ka kasutajaliidese esialgsed sketch’id, mis ilmselt on olnud ka analüüsi koostamisel suureks abiks (pilt ise on küll kõrvalise vaataja jaoks üsnagi raskesti hoomatav, st tundub olevat arutelu käigus valminud mustand).

Analüüsi loetavust hõlbustaks teksti parem liigendamine (nt bulletpointide kasutamine, oluliste mõistete esile toomine jmt). Palju annaks arusaadavusele juurde ka see, kui lõpus toodud vaadete joonised oleks kajastatud kohe vastavate analüüsi peatükkide juures.

Peamine küsimus, mis analüüsi lugedes tekib, on et kellele on rakendus mõeldud? Kui koduse raamatukogu pidajale (st näiteks kui mina tahan oma olemasolevaid raamatuid hallata ja vahel välja laenata), siis milleks on vajalik kasutajakontode loomine (ja parooliga sisse logimine)? Vaevalt, et te olete plaaninud oma kodu nurka iseteenindusterminali paigutada :) Kui see on mõeldud nö välisele laenutajale (st näiteks mina tahan sõbra „raamatukogust“ omale raamatuid laenutada), siis kuidas mina sellele tarkvarale ligi pääsen? St kas on olemas mingi keskserver, millele ma rakenduse kaudu saan ligi pääseda? Või soovite rakenduse luua veebipõhisena? Arusaamatuks jääb ka see, kust kasutaja oma soovitud raamatu lõpuks füüsiliselt kätte saab.

Kokkuvõtteks, on näha, et olete palju vaeva näinud ja rakenduse üsnagi põhjalikult läbi mõelnud. Kui suudate ära vastata ka ülaltoodud küsimustele, siis võib sellest täitsa tore töö tulla (st midagi, mida saaks ka praktikas kasutada).

PS. Analüüsi lõpus toodud diagramm on vast siiski ERD-diagramm, mitte UML :)

https://en.wikipedia.org/wiki/Entity%E2%80%93relationship_model

https://en.wikipedia.org/wiki/Unified_Modeling_Language

Retsensioon Premium esmasele prototüübile

Business Logic Layer BO-d on paksult täis service´te koode. Meetodid tagastavad alati ühe või mitu BO-d ja kõik need BO-d, mis on tagastatud, on muuhulgas juba laetud. Meetod, mille nimi sisaldab sõna „load“ jätab mulje, justkui tuleks laadimist eraldi teha. (See peaks olema tegelikult servise´is) Näiteks createBook teeb iseenda klooni, aga meetodi nimi ise sellele kuidagi ei vihja.

Rakendus ise Rakenduse üldmulje on väga hea. Tore, et on jõutud tegeleda graafilise poolega. Hetkel on töökorras (st avanevad) Book Lending ja Log. Aga kuna BLL on võrdlemisi tühi, siis ei ole võimalik lõpuni funktsionaalsust testida.

Andmebaasi struktuur Lending ja Person tabelite vahel on kaks sidusust, kuid Lending tabelis ei ole välju, mis seda võimaldaks. Vea parandamiseks soovitame enne diagrammi uuendamist vana diagramm ära kustutada. Wiki analüüsis Lisa 2 kujutatud andmebaasi struktuur ei vasta reaalsele andmebaasi struktuurile?

Lisamõtted Meetodite nimed võiksid olla rohkem ette ennustatavad, ehk siis võiks luua ühtsema nimede süsteemi. Näiteks, kui mul on meetod, mis annab kõik raamatud, siis selle nimi võikski olla „annaKõikRaamatud“. Autoformat´it võiks ka kasutada alati enne Commit´imist (nt üleliigsed tab´id jms) Lisakiitus wireframe joonistuse ja põhjaliku Wiki lehe eest!