[es] - Hitno potrebna pomoć oko greške u kodu !

Shadowed
.NET developer

SuperModerator
Član broj: 649
Poruke: 8804
212.200.70.*

Sajt: www.diskusije.net

Profil

Re: Hitno potrebna pomoć oko greške u kodu !

^{27.07.2007. u 03:37}

Nadjoh malo vremena da pogledam detaljnije ovu temu. Imam nekoliko pitanja za korisnika rgdrajko, cisto diskusije radi, posto ima mnogo vise godina iskustva od vecine nas, mozda naucimo nesto od njega. BTW, svaka cast na ulozenom trudu i volji da besplatno uradi aplikaciju.
Radi lakse organizacije i odvaranja, numerisacu...

1. Ako je

Citat:

rgdrajko: Osnovo pravilo microsofta(koje sam negde procitao) tj. Bila Gejtsa:
"Nije bitno kako radi program, bitno je da lepo izgleda."

Da li je onda ovo primer lepog izgleda aplikacije?

2. U kodu nema deklaracije promenljivih. Nigde nema ni jedne Dim kljucne reci (osim ReDim koji je u nekim situacijama bio neophodan). Ako je poznato da:
-Deklaracija promenljivih omogucava manje zauzece memorije jer se ne koristi Variant tip koji je opisni
-Koriscenje unapred deklarisanih promenljivih je brze jer se ne vrsi rad sa Variant tipom kod kojeg je potreban rad u pozadini da bi vrsio rad sa bilo kojim tipom promenljive
-Deklaracija omogucava prilikom pisanja koda olaksanje u vidu IntelliSense-a sto iako tebi mozda nije potrebno olaksace posao onome ko kasnije radi. Kao iskusan programer pretpostavljam da znas da kod obicno koristi i drugi programer ne samo onaj koji ga je prvi pisao. Pogotovo sto je ovo kako si napisao "skolski primer".
-Povecava preglednost koda ako se prilikom deklaracije u komentaru navede i sta predstavlja ta promenljiva

koji je razlog nedeklarisanja promenljivih?

3. Sve klase osim clsSlog nemaju metode vec samo property-e. Zbog cega su onda to klase? Zasto nisu napravljene strukture sa istim property-ima? Strukture tome i sluze, zar ne? Pada mi na pamet samo jedan razlog...

4. U metodi clsSlog.UcitavanjeIzFajla imas brojac kojim odredjujes koji je podatak trenutno ucitan. Zasto je to radjeno pomocu velikog broja If-ova umesto pomocu Select Case?

5. Zasto se vrednosti menjaju u Change dogadjaju tekstbox-ova? Nije neki narociti issue, ali jednostavnije je da se to uradi prilikom snimanja tj. klika na taster za snimanje ili pri promeni sloga. Bilo bi na jednom mestu u kodu i lakse za odrzavanje.

Za sada toliko. Nemam na zalost vremena da detaljnije pogledam. U sustini, sve ovo radi, ali izgleda kao da je pisao neko ko ili radi vrlo sablonski ili je neiskusan. Pretpostavljam da je ovo prvo, s' obzirom na dugogodisnje rgdrajko-vo iskustvo.

Download from Youtube

Otkrivena nova tehnika kloniranja

FBzP

^{27.07.2007. u 03:37}

rgdrajko
Beograd

Član broj: 117734
Poruke: 611
80.93.249.*

Sajt: www.enigmatika.zzl.org

Profil

Re: Hitno potrebna pomoć oko greške u kodu !

^{27.07.2007. u 07:44}

Citat:

Shadowed: Nadjoh malo vremena da pogledam detaljnije ovu temu. Imam nekoliko pitanja za korisnika rgdrajko, cisto diskusije radi, posto ima mnogo vise godina iskustva od vecine nas, mozda naucimo nesto od njega. BTW, svaka cast na ulozenom trudu i volji da besplatno uradi aplikaciju.
Radi lakse organizacije i odvaranja, numerisacu...

1. Ako je
Da li je onda ovo primer lepog izgleda aplikacije?

2. U kodu nema deklaracije promenljivih. Nigde nema ni jedne Dim kljucne reci (osim ReDim koji je u nekim situacijama bio neophodan). Ako je poznato da:
-Deklaracija promenljivih omogucava manje zauzece memorije jer se ne koristi Variant tip koji je opisni
-Koriscenje unapred deklarisanih promenljivih je brze jer se ne vrsi rad sa Variant tipom kod kojeg je potreban rad u pozadini da bi vrsio rad sa bilo kojim tipom promenljive
-Deklaracija omogucava prilikom pisanja koda olaksanje u vidu IntelliSense-a sto iako tebi mozda nije potrebno olaksace posao onome ko kasnije radi. Kao iskusan programer pretpostavljam da znas da kod obicno koristi i drugi programer ne samo onaj koji ga je prvi pisao. Pogotovo sto je ovo kako si napisao "skolski primer".
-Povecava preglednost koda ako se prilikom deklaracije u komentaru navede i sta predstavlja ta promenljiva

koji je razlog nedeklarisanja promenljivih?

3. Sve klase osim clsSlog nemaju metode vec samo property-e. Zbog cega su onda to klase? Zasto nisu napravljene strukture sa istim property-ima? Strukture tome i sluze, zar ne? Pada mi na pamet samo jedan razlog...

4. U metodi clsSlog.UcitavanjeIzFajla imas brojac kojim odredjujes koji je podatak trenutno ucitan. Zasto je to radjeno pomocu velikog broja If-ova umesto pomocu Select Case?

5. Zasto se vrednosti menjaju u Change dogadjaju tekstbox-ova? Nije neki narociti issue, ali jednostavnije je da se to uradi prilikom snimanja tj. klika na taster za snimanje ili pri promeni sloga. Bilo bi na jednom mestu u kodu i lakse za odrzavanje.

Za sada toliko. Nemam na zalost vremena da detaljnije pogledam. U sustini, sve ovo radi, ali izgleda kao da je pisao neko ko ili radi vrlo sablonski ili je neiskusan. Pretpostavljam da je ovo prvo, s' obzirom na dugogodisnje rgdrajko-vo iskustvo.

OK, kazu, napad je najbolja odbrana, zar ne?

Odgovori:
1. Ne, to je radjeno na brzinu.
2. Ne, to je radjeno na brzinu.
3. Ne, to je radjeno na brzinu, ideja je bilo da sve budi klase, za rne?
4. Ne, to je radjeno na brzinu i za tako mali broj ispitivanja to ne utice bitno na brzinu, zar ne?
5. To je radjeno na brzinu i izgleda sa namerom da se neki isprovociraju, tako ispada, zar ne?

Drago mi je da ovo sve u sustini radi, s obzirom na dugogodisnje neiskusvo i sablonsko pisanje, zar ne.

www.enigmatika.zzl.org

^{27.07.2007. u 07:44}

Shadowed
.NET developer

SuperModerator
Član broj: 649
Poruke: 8804
212.200.70.*

Sajt: www.diskusije.net

Profil

Re: Hitno potrebna pomoć oko greške u kodu !

^{27.07.2007. u 09:01}

1. OK
2. OK, mada sam mislio da sa vremenom to jednostavno postane navika.
3. Paa, OK, posto je tvoja ideja, ima smisla da to bude razlog ali je u tom slucaju losa ideja jer ne pokazuje dobro koriscenje klasa (ili nije bila ideja i da bude prikazano dobro koriscenje klasa?). Sto se tice "na brzinu" podjednako je komplikovano napisati strukture i ovakve klase.
4. Ne, ne utice bitno na brzinu izvrsavanja, cak je verovatno i manje vise isto. Ali ako je radjeno na brzinu valjda je logicnije da bude onaj kod koji je 3-4 puta kraci (nije valjda potrebno neko razmisljanje da bi video koji je nacin bolji u takvoj situaciji?).
5. OK, na brzinu bi bio OK razlog. Sto se provociranja tice, ne znam ja tvoje razloge, ako si zbog toga tako radio onda je zbog toga, ti znas :). Ja sam pitao jednostavno iz razloga sto je malo losiji dizajn koda (ono o cemu obicno brinu iskusniji programeri).

S' obzirom na ovaj post sve sto je napravljeno da radi - radi, ali mnogo toga nedostaje a ja nigde nisam rekao da si neiskusan, naprotiv uzeo sam tvoje 20ogodisnje iskustvo koje nigde ne propustas da pomenes kao aksiomu pa procenio ovaj kod kao sablonski pisan (tj. rekao da tako izgleda).

Napad? Zasto napad? Ako imas iskustvo i posto ovaj forum sluzi za razmenu znanja i posto si vec napisao program (znaci ucestvujes u diskusiji) ne vidim zasto je postavljanje pitanja o tome kako je nesto uradjeno napadanje. Pogotovo sto sam sve lepo i argumentovano pitao i naveo i razloge zasto je koje pitanje postavljeno. Aj', OK, prvo pitanje mozda moze da se pritumaci kao provokacija, al' stvarno mi bilo smesno videti takva dva posta jedan posle drugog :)

Download from Youtube

Otkrivena nova tehnika kloniranja

FBzP

^{27.07.2007. u 09:01}