Talk:Meeskond "Pohl": Difference between revisions

From ICO wiki
Jump to navigationJump to search
Lkermas (talk | contribs)
No edit summary
Mtoompuu (talk | contribs)
→‎Aitähh , Seda!: new section
 
Line 52: Line 52:


Stringid pole arveFormi aknas numbriväljadel lubatud juba pool päeva (updated kell 21.04, 5.01.11)
Stringid pole arveFormi aknas numbriväljadel lubatud juba pool päeva (updated kell 21.04, 5.01.11)
== Aitähh , Seda! ==
Parandasin vead ära.
Mailis

Latest revision as of 15:44, 6 January 2011

Kasulik ärirakendus igale ettevõtjale - hea idee. Naljaks roosa kujundus. Lihtne installeerida. Mugavad aknad klientide juurdelisamiseks: eraldi firmade ja üksikisikute jaoks. Meeldis, et arvelisamise aknas on klientide otsingusüsteem esitähtede järgi. Programm on piisavalt kommenteeritud, arveFormi klass on kuidagi liiga pikk... Edasiarendamiseks võiks mõelda, kuidas tabelitesse saaks ridu juurde lisada, ilma akent eelnevalt sulgemata. Kõik asjad päris "1-2-3!" ei tööta veel, aga ideel on jumet. Kui crm-ilikkust lisada, võiks saada lihtsa ja praktilise rakenduse, kuna senini proovitud analoogsetest (crm) tarkvaradest on pea kõik liiga laialivalguvad, liiga kõiketegevad ja suuremahulised. Arvete genereerimise võimalus näiteks üksikettevõtjale, kes väga raamatupidaja ei ole, aga ometi kiiresti arveid kirjutada soovib :) ja paberid pärast välja prindib ning raamatupidajale toimetab. (Lelet Hollo)


Rakendus

Lihtne installeerida, kuid millegi pärast võttis aega. Võib Olla minu ühenduse probleem oli korra kes tea. Igatahes nutikas ärirakendus ettevõtjatele. Kuid kujundus oleks võinud parem olla. Veidikene rohekm proffesionaalne. Liiga lihtlabane. Tobe oli see , et kastidesse võis sisestada ükskõik mida, et oleks võinud kontroll olla, et kui on number kast siis ainutl numbrid ja kui stringi kast siis ainult stringid. Ja emaili valideerimine oleks võinud kah olla. Tabelisse lisades oleks võinud vähem infot näidata, pole vaja ju kasutajal näha mis ID-ga ta on jne. Ebavajalikku liiga palju.

Lähtekood

Kood ise on korralik ja hästi kommenteeritud. Kõik on kenasti arusaadav.

Wiki

Wikis kõik vajalik info olemas.


Madis Kõosaar D21


Seda Sahradyan D21

Idee on päris hea. Igale ettevõttele kindlasti vajalik rakendus. Rakenduse paigaldamine oli kiire ja mugav. Rakendus oli üsna lihtsalt mõistetav ja arusaadav. Ei olnud keeruline asju sisestada. Kahjuks siin positiivsed asjad lõppevad. Lahtritesse saab sisestada ükskõik mida ja lahtird saab ka tühjaks jätta, kõik sobib. Mingit kontrolli ei ole, mida sisestatakse. Ühesõnaga saab luua kliente, firmasid, arveid, millel ei ole mingit infot. Andmebaasiga mina ei saanud ühendust, seega kustusid kõik minu sisestatud firmad, kliendid, arved kohe ära kui ma vastava akna kinni panin. Arve lisamise vaade ei ole oma käitumiselt kuigi kasutajasõbralik. Kui lisad arve tabelisse ja kogemata vajutad selle peale, enne kui see andmebaasi lisatakse, ei muutub kogu aken üsna kasutuks, ei saa enam kuhugi vajutada ja peab kogu akna kinni panema. Ja kui tagasi minna, siis arve on loomulikult kadunud. Kui ma lõpuks jõudsin arve ridade täitmise juurde, siis jooksis mul programm kokku ja andis üsna arusaamatu errori esmapilgul. Ainult lähemal uurimisel sain aru, et üks lahter sai valesti täidetud. Ka lahtirite õieti täitmisel ei saanud mina arvele rida lisatud viie korraga. Kujundus ei olnud just kõige asjalikum, kuid kujundus ei olnudki ülesande eesmärgiks. Üks asi oleks võinud kujunduse juures teisiti olla. Oleks võinud teha nii, et saab korraga avada ainult ühe sama tüüpi akna või kogu rakendus töötaks ühes aknas. Oleks võinud selle navigeerimise asja natukene teisiti teha. Praegu saab avada aknaid nii palju kui arvuti neid laadida jõuab. Mis on veel huvitav on see, et saab selle "peamenüü" kinni panna, kuid ülejäänud aknad jäävad ikka lahti. Naljaks on see ka, et kui programm käima panna, siis ta laeb ennast 7 korda arvuti protsessidesse ja sööb kokku üle 300mb mälu, mis on natukene naljakas nii väikese programmi kohta. Kui liigsed 6 tükki kinni panna, siis programm ikka töötab edasi.

Kood on üsna loetav ja korrektne, kommentaare oleks võinud olla rohkem, kuid kood on nendetagi loetav. Wikis on kõik oluline olemas.


Huvitav lähenemine projektile - kaks erinevat lahendust, mis on teineteisest erinevad. Mina uurisin lähemalt Windows Forms varianti.

Kood on ilusti jagatud klassidesse ning kommenteeritud.

Rakenduse töötamise ajal suuremaid tõrkeid ei tekkinud. Kujundus oli konservatiivne, mis selliste rakenduste puhul on eeldatav.

Ainuke probleem tekkis kliendi lisamisega, sest see oli pandud vormi varjatud kujul. Selleks, et klienti lisada, tuli vajutada kirjale Lisa klient. Sellist lahendust ma eriti heaks ei pea. Kuid avanenud vaatepilt oli hea.

Olemas olid kõik andmebaasis olevad kliendid. Lisaks sai juurde lisada uusi kliente ning seadete alt lisada uusi linnu või riike. Arvel on kõik vajalik olemas - kliendi ja müüja andmed.

Positiivne on see, et arveid saab salvestada kettale ja kuvada html vormingus, ehk ettevõtjal jäävad kõik arved alles ning ta saab neid vajadusel printida. Hea lahendus on veel see, et on tehtud alg html fail, ehk siis seda muutes muutuvad ka genereeritavad arved.

Wikis oli samuti kõik vajalik olemas.

Arvan, et tiim on projektile seatud nõuded üldjoontes täitnud.


stringe ei luba numbriväljadele

Stringid pole arveFormi aknas numbriväljadel lubatud juba pool päeva (updated kell 21.04, 5.01.11)

Aitähh , Seda!

Parandasin vead ära.

Mailis