Talk:Meeskond: .njet

Retsensioon tiimi .njet lõpptootele tiimilt XYZ

• .njet tiimi lõpptooote analüüsi sissejuhatuseks peame kohe mainima, et hetkel ei saa rakenduses kontot luua. Debugimisel selgus, et konto loomisel jõutakse meetodini “GetUserByName” ja sealt edasi enam ei liiguta, sest kontot luues ei ole veel andmebaasis sellise nimega kasutajat. Seega ei saanud me pärast esimest kahte akent (LoginWindow ja NewUserWindow) teiste akende funktsionaalsust rakendust käivitades kontrollida. Seetõttu kommenteerime edaspidi koodi ainult visuaalse vaatlemise ning vaateid Visual Studio eelvaadete põhjal.
• Koodi pole väga palju kommenteeritud; hetkel on olemas ainult töökommentaarid, mis oleksid abiks koodi edasisel kirjutamisel, aga selgitusi selle kohta, mida üks või teine meetod teeb, ei ole. Samas on tore, et mujalt võetud koodile on ilusti viidatud linkide näol.
• Solution on tervikuna hästi liigendatud. Meetodite ja klasside nimetamisel on kasutatud läbi projektide ühtset stiili, mis muudab koodi loetavaks ja arusaadavaks.
• Vaated kasutavad databindingut vaatemudelitega ning ei sõltu code-behindist.
• Üldiselt tundub kood korralik. Ilmselt on meetodid funktsionaalsed ja hetkel ei saa luua kontot lihtsalt sellepärast, et vaatemudelis on kusagil õigele meetodile viitamise bug sees.

Kommentaarid WPF-i visuaalile:

• Sisselogimisaknal on tervitussõnumi alumine ja parem osa ära lõigatud, kuigi XAMLi eelvaates on see täies mahus näha.
• Silmale pakub ilu, et rakendusel on läbiv visuaalne teema sinise ja valge gradientefektiga.

Kokkuvõttes suudame me samastuda tiimiga .njet, kuna ka meie tegime semestri alguses väga optimistlikke plaane, millest kõiki ei suutnud täita. Lisaks sellele kaotas .njet ühe oma tiimiliikmetest ja eelduslikult 33% töövõimest, mistõttu on selge, et koodi täiustamiseks oleks neil veel aega vaja.