Problem sa Memory Usage applikacije

ok, nismo se razumeli ali nije ni vazno jer sam se snasao za pocetak. Za ubuduce, kad profilujes aplikaciju to radis u okviru jedne sesije sa vise snapshotova, ne zasebna sesija za svaki snapshot jer onda ne mozes da poredis snapshote i otezana je analiza jer ne moze da se eliminise ono sto je vec postojalo (delta nije dobar). Isto kad sam rekao gen0 snapshot to je znacilo da i prvi i drugi snapshot budu preko onog dugmeta sa Gen0 (pored ovog koje pritiskas), ne da kreiras poseban gen0 snapshot, to dugme samo znaci da prvo uradi gen0 kolekciju i onda tek da uradi snapshot, sto vodi ka smanjenju suma a i velicine session fajla.

Elem,

tvoja muka lezi u krypton buttonu defintiivno, pronasao sam 8000 instanci dugmica, oni sami po sebi nisu toliki problem (managed data je manje od 2 mb) ali posto za svaki ucitavas sliku onda to vuce unamanaged GDI resurse koje medjutim nisi ukljucio u analizu pa ne mogu da vidim da li je to zasluzno za raspad sistema.
Sad treba da uradis sledece. Ubaci dispose u igru, pre nego ti stari dugmici odu iz scope-a disposuj ih (vidim u logu da nisu disposovani) i uradi jos jednu sesiju sa dva snapshota (posle starta, i posle maltertiranja) i daj mi fajl, ali sad podesi ove opcije u session wizardu:

- dispose tracker - YES
- real time - NO
- instance data - NO
- heap utilization - YES
- unmanaged resource tracker - YES

to ce malo i ograniciti sesson file a dace informacije koje trebaju, znaci oba snapshota u okviru iste sesije, please.

Potpis

Sloba je za 12 godina promenio antropološki kod srpskog naroda. On je od jednog
naroda koji je bio veseo, pomalo površan, od jednog naroda koji je bio znatiželjan, koji
je voleo da vidi, da putuje, da upozna,
od naroda koji je bio kosmopolitski napravio narod koji je namršten, mrzovoljan,
sumnjicav, zaplašen, narod koji se stalno nešto žali, kome je stalno neko kriv - Z.Đinđić

Pozdrav,

nazalost ima jedan mali problem sa tim dispose-om:Ako pogledas code koji sam na pocetku teme post-o,
primjetit ces sledece:



Kao sto vidis, dinamicki se dodaje handler za svaki button. Eh sada, nisam bas siguran ako da uklonim handler za btn(i), jer nemam to "i".
Dispose bih radio na izlazu sa forme, ali nazalost,
iako nisam probo, siguran sam da ce compiler javiti error jer ne zna sta je to "i".
Iskreno ni ja ne znam kako da ga dispose, jer ako stavim ovako, past ce...



Kako da ga dispose???

Vrlo prosto, tamo gde ih inace izbacujes iz scope-a, a to je tamo gde pozivas TableLayoutPanel3.Controls.Clear(), samo sto mene ne interesuje removehandler, Eventhandler instanca koju kreiras sa AddHandler je ciklicna (pripada button-u na koji pokazuje), tako da kad cleanup pokupi button pokupice i taj event handler, ako bas hocs da budes koser mozes da dodas i removehandler, ali mislim da to nije presudno, vazniji je dispose.

samo sto izgleda na osnovu logova da kryptonbutton dodatno kreira evente u nekim svojim internim strukturama koje zive duze od dugmeta i da te reference sprecavaju cleanup nakon sto nullujes svoj pointer, sto pretpostavljam da ce button.dispose da resi. Posto ta event invocation lista pripada svom posebnom GC rootpath-u ne mogu da znam dal nesto drzi objekat u zivotu legalno ili bagovito pre nego sto uradis dispose, verovao ili ne tvoj jedan button pripada 22 razlicitih rootpath-ova, tj nalazi se u 22 razlicite GC strukture, sto je malo mnogo bar po nekom mom iskustvu, treba sve to pocistiti (al to nije tvoj problem, vec onog ko je pisao krypton). Tek kad button uradi svoj cleanup mogu da vidim da li je neki root-path preostao ili su svi nullovani i objekat je otisao na kolekciju.

Dakle ubaci sledeci kod umesto TableLayoutPanel3.Controls.Clear()



ovo predvidja da u TableLayoutPanel3 drzis samo dugmice....

Potpis

Sloba je za 12 godina promenio antropološki kod srpskog naroda. On je od jednog
naroda koji je bio veseo, pomalo površan, od jednog naroda koji je bio znatiželjan, koji
je voleo da vidi, da putuje, da upozna,
od naroda koji je bio kosmopolitski napravio narod koji je namršten, mrzovoljan,
sumnjicav, zaplašen, narod koji se stalno nešto žali, kome je stalno neko kriv - Z.Đinđić

Uf! Ajd' ovako, array ti i ne treba ...



Da pocistis za sobom, recimo ovako (ideju si mogao dobiti već iz posta sa početka teme):



Pozdrav

Edit, malo zakasnih ...

Pozz,

evo napravio sam dva snapshota Gen0 sa dispose kao sto ste opisali.

http://rs428.rapidshare.com/files/244345831/Session_4.rar

Nazalost nista se nije promijenilo.

Da li bi recmi pomoglo ako bi ja negdje upload demo aplikacije pa da pogledate?

Nije se nista promenilo zato sto nista nisi dispose-ovao, pogledaj sliku i videces da je dispose kolona prazna, sto znaci da nijedan od dinamicki kreiranih dugmica nije dispose-ovan; da nemas jos negde TableLayoutPanel3.Controls.Clear()? Ta kolekcija ne sme da se prazni sa Clear() bez da se elementi disposuju prvo, to je obavezno u tvom slucaju jer na nivou aplikacije postoji root instanca klase KryptonManager koja drzi zivu instancu svih krypton kontrola na formi (koliko vidim zbog skinovanja) i mislim da je to rootpath koji sprecava autoamtsku finalizaciju buttona (na slici desno).


Dalje, tebe definitivno slike umlacuju, na kraju imas 16800 image objekta cije su bitmape lockovane u memoriju, tebi na kraju ne ponestane virtuelnog adresnog prostare vec mesta u fizickoj memoriji za smestanje slika (onih iz \program files\@Caffee\@Caffee\slike, npr balantine2.jpg, campari.jpg prednjace). To ce definitivno dispose da resi ako krypton nije bagovit.

Potpis

Sloba je za 12 godina promenio antropološki kod srpskog naroda. On je od jednog
naroda koji je bio veseo, pomalo površan, od jednog naroda koji je bio znatiželjan, koji
je voleo da vidi, da putuje, da upozna,
od naroda koji je bio kosmopolitski napravio narod koji je namršten, mrzovoljan,
sumnjicav, zaplašen, narod koji se stalno nešto žali, kome je stalno neko kriv - Z.Đinđić

Pazi kad me nije mrzelo da skinem Krypton Suite ... i da malo pogledam.
Prvo sto sam uocio je da sam te pogresno savetovao oko dispose buttona iz panela.

Naime, to ne radi kako treba, jer iterator prodje samo preko pola objekata u kolekciji,
sto se moze videti iz sledeceg:



Sto ce reci, iteracija kroz kolekciju i promena broja elemenata iste, se ne zavrsava sretno.
Previd, sa moje strane, ali i dobar primer kako nesto sto na prvi pogled radi dobro u stvari i ne radi.

No, da bi stvarno prosli kroz kolekciju kontrola u panelu treba nam nesto na primer ovako:


I sad konacno imamo metod koji stvarno radi dispose ... kako treba

Probaj ovako.

I da, ovoliki broj slika je stvarno problematican, ako je bas neophodno da ih bude ovoliko,
potrebno je optimalnije resenje.

Pozdrav i javi kako ide dalje

@mmix

Hvala na analizi, sad sam gledao bez slika i niej toliko strasno. Ali moj problem je da jednostavno mroam imati sliku na buttonu.
Dali mozes da mi obajsnis koji je najbolji nacin odnosno gdje da cuvam te objekte(slike)? Da li mozda u bazi(koristim Sql Server 2005)?
Trenutno drzim na disku(znam da nije najbolja solucija, znao sam to i raniej ali se ni u snu nisam nadao da ce me toliko kostat)


@vbbojan

Hvala i tebi. To sto se samo pola kontrola dispose, sam i ja jutros primjetio. Nazalost nisam stigao napisati jer sam morao krenuti na put.
Sa novim kodom sam takodjer probao, radi dobro, memorija takodjer radi kako ocekujem. Medjutim kada u tablelayoutu radi dispose, sve mi treperi.Sto mi se nikako svidja. Imas li neki prijedlog? Probao sam da suspendujem layout pa da resume ali ne ide bas....

Nemoj ici kroz kolekciju sa ovom metodom:



idi sa ovom(C# ali mislim da ces se snaci):


Razlog, modifikuje se kolekcija kada uradis Dispose() ali to uradi tiho i ne upozorava te.

Pokusao, i opet isto. Treperi i dalje...

@Igor, u pravu si, i ja sam zaboravio na ovaj idiotizam Control objekta, ako su vec toliko prekrsili sopstvenu semantiku onda su bar mogli da stave neki fleg u Clear metod pa da ti parent disposeuje kontrole ako hoces. Ima tu mnogo nelogicnosti.

@CallMeSaMaster:

slike mozes i da kesiras pa da kloniras po potrebi, dobices na brzini sto ne moras svaki put da ih ucitavas sa diska. Ali ti i dalje to ne reseava tvoj problem, dal ces klonirati bitmapu ili ces je ucitati sa diska ili ces je ucitati iz SQL-a opet zauzima memoriju jer GDI radi sa bitmapama u fizickoj memoriji, ako ne disposujes bitmap objekat GDI handle ce ostati zajedno sa bitmapom. Dispose ce osigurati da se u dispose lancu oslobodi i slika iz GDI-a.
form ti treperi zato sto dispose button-a poziva remove u TableLayoutPanel koji posle svake iteracije poziva repaint. Treba da koristis suspend layout na nivou forme, pre nego udjes u dispose petlju i da uradis resume na kraju kad je TableLayoutPanel ispraznjen i onda napunjen sa novim dugmicima (u prevodu, na pocetku i kraju btnClick metoda)

Potpis

Sloba je za 12 godina promenio antropološki kod srpskog naroda. On je od jednog
naroda koji je bio veseo, pomalo površan, od jednog naroda koji je bio znatiželjan, koji
je voleo da vidi, da putuje, da upozna,
od naroda koji je bio kosmopolitski napravio narod koji je namršten, mrzovoljan,
sumnjicav, zaplašen, narod koji se stalno nešto žali, kome je stalno neko kriv - Z.Đinđić

Ok, sada radi to sa dispose, jer mi se memorija vrti oko jedne vrijednosti. Tako da tacno vidim da kada uradim dipose da se ista i oslobadja.

Nazalost, treperenje i dalje ostaje, da li suspend/resume bio na pocetku i karju ili bilo kako....

Ne mora da se radi za celu formu, mozes i na nivou samog panela.
Evo ti ideja, isto bi trebalo da uradis i kad dodajes kontrole u panel.

@vbbojan

Pokusao sve ali ne pomaze....Razmisljam da zamjenim tu kontrolu ali enmam bas puno izbora...
Stvarno ne kontam zasto se ovo desava...