Talk:Meeskond "PE"
Graafiline pool selge, võibolla natuke liiga kokku pressitud. Ruumi ju on, oleks võinud natuke avaram olla.
Otsing tegemisel võiks olla kasutatud LIKE meetodit, kuna hetkel toimib otsing ainult siis, kui sisestatud on täpne kasutajanimi.
Statistika osa on suht kasutu minumeelest. See võiks pigem kuvada statistikat konkreetse kliendi kohta. Või pigem oleks hädavajalik kuvada antud auto ja kliendi ajalugu, et näha, mis on sellel autol varem remonditud.
Lahtrite valideerimine toimib ja mingit errorit baasi sisestamisel ei õnnestunud saada.
Isiklikult pooldan vormidele kuvatavaid veateateid, kui avanevaid message boxe, mida tuleb ekstra sulgema hakata aga see on isiklik eelistus.
Kuna vormil on ka juba võla lahter, siis tasuks kaaluda sinna ka mingisuguse arve genereerimise osa tekitada.
Üldiselt on rakendus suhteliselt väike ja lihtne, kuid oleks siiski loetavuse huvides võinud kasutada eraldi klassi näiteks sql päringute tarbeks.
Wikis võiks olla natuke rohkem infot rakenduse kasutuse kohta.
Kokkuvõtteks: tehtud lahendus toimis veatult, viimistlus korralik aga rakendusena on tast sellisel kujul vähe kasu.
Kommenteeris, Priit Siimo D22
Arvustus: Karel Niine
Programmi idee paistab olema päris hea, isegi tavalisele autoomanikule mitte ainult väiksesele autotöökojale, kus siis võimalus hoida informatsiooni, mis on autoga või muude motoriseeritud vahenditega toimunud, ning sinna kirja panna ka kulud. Ja kui omad mitut mootorsõidukit siis veel parem, sest kes ikka mäletab millal oma zigullil õli sai viimati vahetatud ja palju see maksis ;) Kahjuks aga käivitamisel kurdab, et ei saa andmebaasiga ühendust, kuna connectioni teekond mis on sortusus ei kõlba. Seda ühenduse teadet taob pidevalt programmis ette. Sellisest kohast nagu “+ Directory.GetCurrentDirectory() + @"..\..\..\db.accdb;" ta seda andmebaasi ei leidnud Samas andmebaasi fail, kui selline, on täitsa sortsu kaustas olemas.
Form1.cs -is on piisavalt korralikult koodi kommenteeritud ning enamasti kõik ilusasti emakeeles.
Meetodide juures oleks võinud kasutada kommentaare stiilis:
/// <summary> /// Komentaar /// </summary> /// <param name="p"></param> /// <param name="koord"></param> /// <returns></returns>
mis lihtsustaks teistel osapooltel asja edasi arendada.
Form2.cs -is kahjuks kommentaare napib.
Koodist paistab veel et programm annab ka kasutajale korralikku tagasisidet valede käikude kohta ja seda järjekordselt emakeeles.
Ausalt on välja toodud info selle kohta, milline osa koodist on internetist saadud. Kuna tõesti jalgratas ei ole mõtet uuesti leiutada, ja kui kood on veel opensource.
Kuna aga kahjuks seda asja päris käima ei saanud, siis kasutaja poolses osas kõike funktsioone läbi proovida ei saanud. Kokkuvõtteks võib öelda, et üldmulje jäi hea, ning loodan et see teekonna probleem peale seda arvustust parandatakse.
LISA 16.12.2010 - Tuli teade et lastud välja uus versioon, nüüd asi toimib, ehk siis näeb ka andemid.
Raske öelda miks ei õnnestunud käivitada programmi Visual Studios 10, sest teistel arvutitel ta surepäraselt käivitab. Igal juhul lisame meie programmi build version ehk exe fail. Saate alla laadida samast kohast
Lugupidamisega
PE
Kuna olen ise auto ja mootoratta huviline siis pakkus programmi kirjeldus kohe huvi. Mõte on hea ja töötavast programmist on kasu nii väikesele/keskmisele töökojale kui ka eraisikule. Kasutaja liides on hea lihtne, kuid kuna programm andmebaasiga ühendust ei saa siis on praegu temast vähe kasu. Esialgu ei ole õnnestunud ka koodi selliselt muuta, et programm andmebaasiga suhtlema hakkaks. Kui tegijatel tulevikus aega see probleem lahendada on see täitsa ok programm.
Reigo Riives
Esimene käivitamine ja kohe kena aken ees, kus saan sisestada oma auto andmed, margi valimisel on kena valik juba ees ja kogu protsess läks lihtsalt. Otsisin, lisasin, redigeerisin ja vaatasin statistikat- töötab :) Accessi kasutamine pole sellise lihtsa programmi puhul miinuseks. Loomulikult on siin ruumi programmi täiendamiseks (näiteks kasvõi auto ajaloost ülevaate näitamisega), kuid põhipoint on täidetud. Kokkuvõttes on vist nii, et kui kasutajal programmi kasutamisel küsimusi ei teki, siis on programm hästi tehtud.
Jako
Erinevalt eelpoolkõnelejatest, sain katsetada programmi koos andmebaasiga ja kohe tekkis selleteemaline küsimus. Küsitav oli minu jaoks Omanike tabel - uue auto lisamisel võiks saada valida juba sisestatud omanike andmete hulgast sobiva ja kui omanik pole eelnevalt tabelis registreeritud, siis lisada tabelisse rida. Praeguse lahenduse korral võiks omaniku andmed sama edukalt olla autodega ühes tabelis. Aga põhimõtteliselt asi töötab, nupud teevad seda, mida lubavad ja statistika ehk võlgnike arv ja võla summa arvutatakse ka õigesti :)
Mari Lehis, DK 21
Sain allalaadida tõesti töötava programmi, andmebaasi seadistustega ei pidanud vaeva nägema. Programm on ise suhteliselt lihtsakoeline, aga täidab enamjaolt oma eesmärki. Mõningad kontrollid on jäänud küll lisamata, näiteks saab auto aastaarvu ja omaniku telefoninumbri väljale kirjutada suvalisi tähemärke - andmebaas ei hakka samuti selle peale virisema, vaid salvestab antud väärtused kenasti ära. On olemas ka üks raport aga see võiks sisaldada oluliselt rohkem andmeid nt. võlas olevate autode ja nende omanike kohta jne. Positiivse poole pealt tooks välja, et kommentaare oli lisatud enamike meetodite juurde. Küll aga soovitaks mõne pikaks veninud meetodist teha mitu väiksemat meetodit, mis parandab loetavust ja hilisemat debugimist/täiendamist.
Lauri Õunmaa DK21
Idee oli hea ning kirjeldus, mis tiimide listis antud on tekitas pisut suuremaid ootuseid, kui valmis programm täita suutis. Nimelt kirjelduse põhjal oli tekkinud ootus leida auto töökodadele mõeldud programm, kus saab näha autode tehtud tööde ajalugu, kuid antud juhul näeb vaid läbi käinud autosid.
Hetke lahendus jättis mulle pooliku mulje nii programmi enda kui ka andmebaasi struktuuri poole pealt(see vast andestatav, kuna ei ole teil veel Raspli andmebaaside aluseid vast olnud ega ka info süsteemi projekti ainet). Lisada võiks vähemalt autoga toimunud hooldustööde ajaloo. Arvestada tuleks ka sellega, et auto omanikud ei ole alati eraisikud ning ühel omanikul võib mitu autot olla (ei ole ilus samu andmeid mitmekordselt baasis pidada).
Andmebaasina sellisel asjal ei soovitaks kasutada accessi, kuna kui tegu on vähegi tegusama ettevõttega kasvab baas kiiresti ja access muutub suhteliselt kiiresti aeglaseks. Paar aastat tagasi sai üht sarnast accessi peale ehitatud lao/kliendihaldus programmi kohatud, mis peale paar aastat kasutuses olekut oli vaja parajat kannatust sealt andmete saamiseks.
Iseenesest tehtu on suhteliselt hästi loetava koodiga ning arusaadavalt dokumenteeritud.
Ühe kommentaari ütleks ka eelnevale retsensendile andmebaasi mitte nurisemise kohta. Nimelt, andmebaas ei peakski nurisema telefoninumbri üle nagu andmebaaside aluste aines teada saate. Õige on sisestada selliseid andmeid mille põhjal mingeid arvutusi ei tehta varchar'ina, sellised andmed ongi näiteks telefoninumber, isikukood, jne.
Mikk Mähar
Idee on hea. Samas kui asi juba Accessi peale on ehitatud, siis vabalt võikski kogu see lahendus, kasutajaliides ja kõik muu, olla Accessi peal tehtud... Teisest küljest, kui on vajadus viia asi üle mingile muule baasile, siis on mugavam, kui kasutajaliides on juba universaalsena olemas.
Teostus on minimalistlik, sisaldab ilmselt elementaarset vajalikku funktsionaalsust. Näiteks remontide ajalugu võiks olla eraldi tabelis, praegu luhtsalt kogu info ühes andmebaasiväljas jne. Statistika osa võiks ehk olla mingis eraldi aknas ja põhjalikum, hetkel lihtsalt messagebox vähese infoga.
Kood on arusaadav ja piisavalt kommenteeritud.
-- Siim Liivand AK41
Väga kasulik programm mootorsõiduki omanikule, ka enesele, et oleks ülevaade enda autoga toimunust olemas. Meeldib programmi lihtsus. Midagi halba programmi kohta ei oskagi kohe öelda, sest kõik mis on olemas, see ka töötab ja puudu sellise lihtsa programmi kohta pole midagi. Mis puudutab andmebaasi osa, siis see oleks võinud siiski olla juba automaatselt integreeritud, ebamugav on mitu asja ühe programmi tarvis installida.
Wiki on natuke tühi, ei ole kirjeldatud ära tööjaotus. Üldiselt väärib see programm originaalsuse poolest lisapunkte. Hakka või isegi kasutama
Allar Märk D-21
Autopäeviku rakendust teha, on tegelikult väärt idee. Antud programm sobiks kasutamiseks ideaalselt üksiküritajatele, kes oma garaažis teiste autosi parandavad. Mõnekümne kliendiga, on sellise rakenduse haldamine veel mugav. Kui aga minna suuremasse firmasse, kus kliendibaas on väga lai, jääb selline rakendus natuke kesiseks. Funktsionaalsuse poole pealt pole väga viga, vajalikud lahtrid on olemas ning informatsioonist puudu ei jää, kui teatud kliendi ja tema auto kohta ülevaadet tahta. Lihtsa windows forms põhjale on tehtud töötav rakendus mis täidab oma eesmärgi ning on üpris lihtne kasutada. Nuriseda võib selle üle, et visuaalne pool peaaegu puudub. Kõik on küll loogiliselt paigutatud, aga tõdeda tuleb, et kasutajasõbralikkust lisab väljanägemine. Samuti võiks statistika näidata ka võlgnike täpselt, võlgnevuste kogusumma polegi nii oluline. Rakenduse resolutsioon peaks olema suurem, samtuti nupud ning väljad. Mina ise oleks teinud selle programmi WPF-is, kuna see keskond vähe arenenum ning on suuremad võimalused visuaalset poolt arendada. Kokkuvõttes hea ideega keskpärane rakendus, mis sooviks veidi arendamist ning "lihvimist".
Sten Vaisma A22
Teha autopäevik on väga hea idee, sest sealt on nähtav kogu auto ajalugu ning see hõlbustab omanikul või remondimeestel leida auto tüüpvigu. Rakenduse kujundus on klassikaline, kuid kõik elemendid on paigutatud väikesele alale. Otsing töötab täpsete päringute puhul, kuid rakenduse koostamisel pole arvestatud juhtusi, kus ei teata täpseid auto või omaniku andmeid. Lahtrite täitmise kontroll on olemas, kuid puudub kontroll selle üle, kas tegu on numbri või tähega. (Näiteks aasta puhul. ) Autode lisamise, muutmise ja kustutamise funktsioonid töötavad korralikult. Statistikalehele oleks võinud lisada rohkem statistilisi elemente, nagu palju on autosi andmebaasis. Koodi oleks võinud jagada klassidesse ning kommentaare juurde lisada. Wiki on veidi tühi – lisada oleks saanud tööjaotuse ning klasside kirjeldused.
Andres Mill A22
