Talk:Meeskond "PE"

From ICO wiki
Revision as of 21:46, 6 December 2010 by Kniine (talk | contribs)
Jump to navigationJump to search

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.