Talk:Meeskond "Ying Yang": Difference between revisions

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


Anu Kuusmaa, DK21, 12.12.2010
Anu Kuusmaa, DK21, 12.12.2010
----
Võrreldes sellega, mis meil siin muude projektidena leida võis, on see ikka üks kena asi küll. Muidugi on natuke masendav, et üks terane eelkõneleja on käinud ja hunniku pisipuudusi avastanud, mis tähendab, et hulgi tuleks kontrolle juurde kirjutada. Minu arvates on peamine saavutus suhteliselt kobeda ja funktsioneeriva rakenduse tööle saamine.
Koodi oli ka päris mõnus lugeda. Üldse on see üks tore projekt algajatele Linq huvilistele vaadata.
Wikit lugedes paraku meeskonnaliikmete panus otseselt ei selgu. Aga teinekord ongi nii, et juhtoinas vedib ees ning lambakari lohiseb järele...
Avaldan kiitust!!!
Anneli Rasu, DK21, 15.12.2010

Revision as of 20:36, 15 December 2010

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


Võrreldes sellega, mis meil siin muude projektidena leida võis, on see ikka üks kena asi küll. Muidugi on natuke masendav, et üks terane eelkõneleja on käinud ja hunniku pisipuudusi avastanud, mis tähendab, et hulgi tuleks kontrolle juurde kirjutada. Minu arvates on peamine saavutus suhteliselt kobeda ja funktsioneeriva rakenduse tööle saamine.

Koodi oli ka päris mõnus lugeda. Üldse on see üks tore projekt algajatele Linq huvilistele vaadata. Wikit lugedes paraku meeskonnaliikmete panus otseselt ei selgu. Aga teinekord ongi nii, et juhtoinas vedib ees ning lambakari lohiseb järele... Avaldan kiitust!!!

Anneli Rasu, DK21, 15.12.2010