Talk:Meeskond "Ying Yang"
Tiim on lahendusega üksjagu vaeva näinud ja kodutöö võiks lugeda hästi sooritatuks. Vaadates logi, siis on arendatud stabiilselt (mitte "viimasel minutil") ja töö on valminud enne tähtaega! Kood on kenasti struktureeritud (igale objektile oma klass) - kommentaare võiks ehk natuke rohkem olla. Baasi struktuur on loogiline (vaate tekitamine laenutuste jaoks ei olnud ilmselt väga lihtne. Eriti arvestades, et see on esimene aine ITK kaugõppes, kus andmebaasidega kokku puututakse) Teades kui keeruline esmakordselt sellise asja tegemine olla võib (vähemalt meil (KRTK) oli:), siis üldmulje on hea. Natuke nurisemist siiski:
- nupp "lisa laenutus" annab NullReferenceExceptioni kui ühtegi raamatut ei ole valitud
- raamatute otsingu Gridis võiks muutmise ära keelata. Baasi küll ei salvestata ja midagi katki ei lähe, ent võib tekitada segadust
- väljalaenutamise vormi kujundusel on natuke arenguruumi :)
- logimise ja raportite nõue jäi vist kahe silma vahele (mis ei ole küll peamine ja jäi ka meie lahendusel poolikuks)
Wiki leht võiks olla natuke põhjalikum. Codeplexi kasutatakse küll lõpliku koodi hostimiseks, ent täitsa tore oleks vaadata ka vahepealseid committe blogi kõrval :)
Ivo Lõhmus, AK31, 8.12.2010
Kena ja loogiline lahendus. Tõesti tuleb meeskonda kiita kiire töö eest! Kui leida ka puudusi, siis saab sisestada uute raamatute all mittenumbrilisi aastaarve - vormilt sisestades sellist aastaarvu lihtsalt ei salvestata, mitte ei hoiatata kasutajat. Kui aga tabelis käsitsi aastaarvu kohale mõni sõna kirjutada, siis see ka salvestub. Samuti ei leidnud ma kohta, kust laenutusi lõpetada saaks. Ühte raamatut saab korraga välja laenutada mitmele kasutajale ja laenutustähtaja saab panna minevikku. Ideaalis võiks olla veel olemas raamatu-vaade, kus on näha antud raamatu ajalugu. Ja logisid samuti ei leidnud. Kuigi nüüd sai üles loetud mitu puudust, siis tegelikult on raamistik selge ja lihtne ning koduraamatukogu haldamiseks annaks seda kindlasti kasutada.
Anu Kuusmaa, DK21, 12.12.2010
