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

---

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

---

Programm tundub kohati olevat veidi toores ja normaalse funktsionaalsuse saavutamiseks oleks vaja rohkem testida ning vead välja lihvida. Raamatu otsimise osa töötab, kuid väljade järjekord on vale. Kui tab'ga edasi liikuda, siis hüppab kursor esimesest lahtrist neljandasse ning seejärel juba nimekirja. Olles terve nimekirja läbi jooksnud, läheb kursot lõpuks ka teise ja kolmandasse lahtrisse. Kui laenutus on sisestatud võiks programm sellest kuidagi märku anda. Hetkel tuleb minna "Aktiivsete laenutuste" alla, vajutada uuendus nuppu ning peale seda näed viimati lisatud sisestust. Kui tähtaeg jätta valimata jookseb programm laenutuse lisamisel kurja "String was not recognized as a valid DateTime." erroriga kokku ja see pole ainuke selline koht. Lugeja lisamine töötab ilusti. Viimaseks väljaks raamatu otsimise juures on laenutused, kuid seal ei ole ühtegi märget ka nende raamatute puhul, mis on välja laenutatud. Raamatute lisamisel õnnestus ühe raamatu lisamine. Teise raamatu lisamisel hakkas programm andma erroreid ning rohkem raamatuid lisada ei õnnestunudki. Tundub olevat mingi probleem primary key inkrementeerimisel, sest uue raamatu ID'ks tekkis 0 (milleks üldse seda ID välja kuvatakse?). Ei leidnud kohta, kus raamatu tagastamise puhul märget teha. On näha, et laenutuste nimekirjas on ka "Aegunud staatust", kuid kuidas süsteem toimib. Väljaandja võiks saada märkida, et raamat on tagastatud. Kokkuvõtteks, kui "bugid" kõrvale jätta, on tegemist asjaliku, kena välimusega ja loogilise ülesehitusega tarkvaraga.

Ulvar Petmanson, AK22

---

Pavel Kodotšigov

AK22

Esimene käivitamine jääb hea muljet, programm töötab otsib, lisab ja kõik teeb mis on ette nähtud. Aga mul on mõned soovitusi ja täiendusi. Minu arvates, esimene tab ehk "Meie Raamu" on absoluutselt mõtetu järjehoidja, sest töötamise ajal mina kahtlen, et ta on vajab kunagi. Kui autoril on see nii meeldis, siis saab see lihstalt ajutiselt panna ilmuvaks.

Programmi tegijatel on mitte päris hea tehtud sisestamise kontroll. Näiteks sisestan ID väljas täht (näiteks A) valin - Lisa laenutus ja kohe saan RunTime Error.

Programm pole mugav selles, et kohe ei näe raamatu žanri, aasta jne. - tuleb hiirega liikuma, et vaadata vastavad väljad.

Programmis kohe saab redigeerida "dataView" objektis, mis ei ole päris hea, sest on võimalust andmeid juhustuslikult parandada või kustutada. Mitte mingeid teadet sellest ei ilmu.

Programm on kommenteeritud, vaid natukene nappus.

Kokkuvõttes programm jäi head muljet ja vaieldamatult ta leiaks tulevikkus oma kasutajat!

20.12.2010

---

Programmi välimus ja kasutajaliides on sümpaatne. Wiki on kergelt minimalistlik, kuid tehtud tööst vähemasti saab ülevaate.

Raamatu lisamisega baasi on mingi kala... Tekib duplikaatne primaarvõti (numbriga 0). Võiks saada ka hulgi laenutada, märgistades mitu raamatut korraga. Laenutamisel võiks olla ette antud mingi vaikimisi kuupäev, praegu kuupäeva mitte valides tekib viga. Andmete valideerimist muidu ikka tehakse (lugeja ja raamatu andmete sisestamisel). Laenutamisel ei kontrollita, kas raamat on juba äkki väljas (kuigi iseenesest ei saaks vajadust laenutada sel juhul üldse tekkida, nii et vahet pole).

Kood on piisavalt kommenteeritud, liigendatud ja loetav. Hetkel sisaldab tarkvara veel päris palju vigu, mis kasutamist tegelikult ei võimaldagi, kuid ideepoolest on asi korralik.

-- Siim Liivand Ak41