[ pgavrilo @ 15.02.2008. 16:35 ] @
Ja sam napokon napravio svoj PRVI program koji ima namjensku korist.
Program je trebao mom tati.
Evo okačiću fajl pa recite da li valja;).
Svaka kritika dobro dođe.
[ toxi_programer @ 15.02.2008. 17:50 ] @
-Tabulatori nisu podešeni kako treba.
-Ne treba ti maximize button ako se već ništa ne dešava kada resize-uješ formu.
-Ne može da se snimi u fajl ono što si uneo.

[ MarkoBalkan @ 15.02.2008. 19:13 ] @
Citat:
toxi_programer: -Tabulatori nisu podešeni kako treba.
-Ne treba ti maximize button ako se već ništa ne dešava kada resize-uješ formu.
-Ne može da se snimi u fajl ono što si uneo.

;)


da i ne.gdje se vidi popis svih?
nije baš lijepo posloženo itd..
[ Predrag Glumac @ 16.02.2008. 00:19 ] @
Petica za trud Letimicno sam pogledao kod i ne svidja mi se VB-ovski stil pisanja koda. Prvo sto mi bode oci je nedostatak access modifiers: sve clanice klase si deklarisao sa prostim Dim. Osmisli klasu i shodno tome poljima dodeli private, public, protected modifiere ..... U ovom primeru cak ni nemas potrebe da IO Stream writer i reader deklarises globalno na nivou klase, jer tvoje metode otvore stream pisu/citaju i zatvore ga, znaci deklarisi ih na nivou metode.

Vidi se da si poceo da se ucis na VB-u, uzmi i pogledaj osnove OOP-a.
Na dobrom si putu, samo nastavi i uci
[ Shadowed @ 16.02.2008. 00:49 ] @
Sad opet ja.. Hteo si reci, vidi se VB6 stil pisanja? :) (nisam otvarao)
[ mmix @ 16.02.2008. 07:56 ] @
Citat:
Shadowed: Sad opet ja.. Hteo si reci, vidi se VB6 stil pisanja? (nisam otvarao)

Nije, forme su OOP, samo sto elementi klasa nisu al dobro, to manje vise, evo sta sam ja zametio:

Code:

        w = New IO.StreamWriter("...\SacuvaniFajlovi\" & ImeFajla & ".namep")
        For i = 0 To 0.5
            w.WriteLine(ImePorodice)
        Next
        w.Close()


For petlja je ovde toatlno suvisna, ako ces da snimis jedan podatak u stream, nema potrebe da ga stavljas u For petlju za koju znas da ce uraditi samo jedan step.

Code:

        w1 = New IO.StreamWriter("...\SacuvaniFajlovi\" & ImeFajla & ".lsti")
        For i = 0 To lst.Items.Count + 1
            w1.WriteLine(lst1.Items.Item(i))
        Next


Kad prolazis For petljom kroz zero-based niz, ides od 0 do Count-1, ne do Count+1.

Nije mi bas jasna logika ucitavaca, tj Try/Catch konstrukcija, ucitavas podatke iz streamova, pa ako pukne onda u catch bloku ih snimis nazad u stream? Sta si zapravo hteo da postignes ovde?

Cemu sluzi Form2 koji nasledjuje Form1? I zasto kroz nasledjivanje?

toliko za pocetak, ja sam malo kritican, al ne daj se obeshrabriti, doteracemo mi ovo
[ pgavrilo @ 19.02.2008. 13:31 ] @
Izvinite što kasnim sa odgovorima počela škola pa se mora učiti.
Što se tiče programa ja sam ga počeo pisati dok sam učio još VB.
To jeste nisam znao za module klasa. Pa čak i sad imam probrema sa klasama
i objektno orjetisanim programiranjem.
Ja dok sam pravio taj program meni je na pamet padalo bilo šta.
Čak i neke gluposti poput i ovog koda:
Code:
        w1 = New IO.StreamWriter("...\SacuvaniFajlovi\" & ImeFajla & ".lsti")
        For i = 0 To lst.Items.Count + 1
            w1.WriteLine(lst1.Items.Item(i))
        Next

U prvom obrascu Form1 je bilo For i = 0 To lst.Items.Count - 1
ali u drugom obrascu tako nije išlo.
Neznam ni ja sam šta sam sve stavljao i radio pa kad sam stavio For i = 0 To lst.Items.Count + 1
umjesto For i = 0 To lst.Items.Count - 1 onda je sačuvalo kao treba.
Iskreno kod treba doterati!!!
Citat:
toxi_programer: .
-Ne može da se snimi u fajl ono što si uneo.

Kako kod tebe ne može kad kod mene može???
Nadam se da ćemi neko pomoći u ispravljanju grešaka.
POZ
P.S
I naravno u doterivanju programa