@Alexandar.
Svojevremeno sam ja bio kritikovan od strane nekih moderatora zbog mog ne imenovanja promjenjivih
i procedura, po pravilu, odnosno kako je preporuceno da se imenuju, a ti sada branis svoje metode koje
u ovom slucaju odstupaju od pravila.
"Reset" sam zapazio jer mi je bilo cudno otkud ta komanda kada ne baratas s fajlovima, a Sub RGB tek sada
kad si napisao. U ovom slucaju, slazem se da nece smetati, ali gledano uopsteno, to moze izazvati probleme.
To bi bilo slicno kao drzanje dvije istoimene funkcije u dva razlicita modula, pa sad pri pozivanju jos moras
da pozivas ime modula pa jos i funkciju. Ja mislim da je to bespotrebno komplikovanje.
A kad si spomenuo FSO on se ne moze usporediti sa starim komandama "Open" za pristup file-ovima,
jer je osakacen, odnosno ne dovrsen za sve nacine pristupa file-ovima koje ima komanda Open.
Vidio sam da je u VB.Net komanda Open izbacena, ali nisam gledao da li je FSO nadogradjen za nedostatke
koje je imao, odnosno mozda je zamijenjen nekom drugom klasom, al to nam sigurno @Shadowed moze reci.
Kad sam se malo detaljnije upustio u razgledanje koda, primijetio sam da cesto koristis dva ugnijezdena "IIF-a".
Koliko ja znam, bukvalno: jedan IIf je jednak:
Code:
If...Then
'......
Else
'......
End If
Pa si ti stoga ustvari napravio:
Code:
If ... Then
'....
Else
If ... Then
'...
Else
'...
End If
End If
Sto je gledajuci sa Krasnopis strane odlicno, ali sa procesorske dosta komplikovano.
Mislim da bi mnogo jednostavnije i citkije bilo koristenje If..Then ElseIf.. Then
Code:
If Cyan < 0 Then
'......
ElseIf Cyan > 100 Then
'......
End If
ili ako bas hoces da skratis kod onda dva If... Then
Code:
If Cyan < 0 Then Cyan = 0
If Cyan > 100 Then Cyan = 100
Nasaosam jos jednu greskicu. Naime Dialog-Box ne pamti Custom Boje. Gledao sam u API-Guide i skontao
da je greska u tome sto "lpCustColors" u tipu "CHOOSECOLOR" mora definisati kao string, a ne kao long, sto
ce kasnije promijeniti i operacije na njemu.
Sve u svemu, da se knjizevnim rijecnikom izrazim, klasa je vrlo zivopisno napisana i sa velikom paletom rijeci,
to jest programerskim rijecnikom receno, korisceno je skoro sve sto je ugradjeno u VB: Petlje, IIf, Nizovi, Type, Case,
Enumeracije, Property Get Let, Proceduralno pisanje.... i jos na sve to sve je ubaceno u Klasu na radost shadoweda.
Sto se mene tice polozio si Diplomski rad ;)
Mala zamjerka je teska citljivost zbog velikog proceduralnog razlaganja koda, al posto je sve u
klasi to je manje-vise nebitno, osim za nas cjepidlake koji se trudimo da nadjemo dlaku u jajetu :)
Citat:
PS. Off.. Vidjeste li ljudi sta bi sa Hamiltona? Ko se kladio na Kimija dobro je proso.
Ero S Onoga Sveta