Talk:Meeskond "PE"
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 begin_of_the_skype_highlighting end_of_the_skype_highlighting begin_of_the_skype_highlighting end_of_the_skype_highlighting begin_of_the_skype_highlighting end_of_the_skype_highlighting 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
Rakenduse Arvustus
- Wikileht - oleks võinud olla süstematiseeritum ja põhjalikum. Wikilehel on olemas logi. Võiks aga olla ka programmi lühikirjeldus eraldi, samuti rollijaotus - muudaks lihtsamini hõlmatavaks, mis programmiga tegu on.
- Idee - idee on hea, antud aine raames originaalne ning samuti vajalik - tihti on vaja meelde tuletada, millal viimati midagi sai parandatud/lisatud vms.
- Programm - käivitades programmi, sain veateate - "Andmebaasi ei leitud või tekkis programmi viga!". Programm käivitus sellegipoolest, ei tööta mitte ükski andmebaasiga seotud nupp. Iga nupuvajutusega antakse kasutajale infot, mida ta parajasti teeb. Näiteks vajutades nuppu "Lõpp", küsitakse, kas kasutaja on kindel, et soovib programmist lahkuda. Samuti antakse kasutajale teavet iga vea puhul - proovides kasutada otsingut, teavitatakse, et andmebaasi ei leitud või on viga päringus. Antud juhul on tegu andmebaasi puudumisega. Statistika nupu vajutamisel tekib unhandled viga.
Kuna sellisel viisil oli suhteliselt tüütu programmi testida, küsisin programmi proovida ühelt eelpool arvustajalt, kel andmebaas töötas. Selgus, et kõik nupud töötavad ning vigu ei teki.
- Programmi funktsionaalsus - idee on hea ja programm töötab korralikult, kuid selleks, et programmis oleks rohkem kasu, oleks vaja kindlasti seda kõvasti täiendada. Näiteks otsing - võiks olla võimalus otsida ka mitte täieliku koodi järgi vms. Andmebaas saaks olla optimaalsem. Nt mitme auto omanikud võiksid andmebaasis olla sees ühe kirjena, antud süsteem seda aga ei võimalda. Programmi praktiliselt rakendades oleks vaja palju lisavälju jne juurde. Samas antud projekti eesmärk oli natuke teine.
- Kood - Form1.cs - tegevusi on kommenteeritud - välja oli toodud isegi lõik, mis oli võetud internetist. Samas oli kood raskesti loetav pikkuse tõttu - oleks võinud kasutada rohkem jagamist klassidesse ning neid kommenteerida. Kasutada oleks saanud ka /// kommentaare - sel juhul on kommentaar näha kohe kursorit objekti vms peal hoides.
- Kokkuvõtvalt - programmi idee on hea ja teostus samuti hea - väga lihtne ning töötab vigadeta. Samuti on disain piisav ja arusaadav ilma üleliigsete vidinateta. Vea puhul antakse enamikel juhtudel selgitus. Andmebaas võiks olla optimaalsem, otsingus võiks saada leida ka osalise sõna järgi. Kõik muu on projekti raames väga hästi, kasutajasõbralikult ja lihtsalt tehtud.
Miina Sikk, D21
