Eigene ControlLibrary - Memoryleak
- VB.NET
- .NET (FX) 4.5–4.8
Es gibt 76 Antworten in diesem Thema. Der letzte Beitrag () ist von Daniel Baumert.
-
-
-
@nafets, ja das wäre nett
-
Tu mal folgendes (so Dein Studio es erlaubt - Express wohl nicht):Daniel Baumert schrieb:
Objekt.Dispose()
Projekt -> Eigenschaften -> Codeanalyse ==> Codeanalyse beim Erstellen aktivieren
da werden genau solche Sachen gefunden.Falls Du diesen Code kopierst, achte auf die C&P-Bremse.
Jede einzelne Zeile Deines Programms, die Du nicht explizit getestet hast, ist falsch
Ein guter .NET-Snippetkonverter (der ist verfügbar).
Programmierfragen über PN / Konversation werden ignoriert! -
-
FailDaniel Baumert schrieb:
Ok, und bei //Objekte disposen muss ich jetzt für jedes Objekt
Objekt.Dispose() hinschreiben?
Das kann doch nicht sein. In Vb.net ist das doch auch nicht so..
Imo sollte man, wenn man sich ans Entwickeln eigener Controls wagt, wirklich so leidlich sattelfest sein, etwa ein gründliches Buch durchgelesen haben, zB den Löffelmann oder den Kühnel/Leibbrand:
Entwickler-Ressourcen und Tools, Bücher und WebCasts
sind beide umsonst.
Da wird auch der Dispose-Pattern ordentlich erklärt (zumindest vom Löffelmann2005 weiß ich das genau).
Edit - Korrektur: Im Kühnel wird Resourcenbereinigung leider auch nicht durchgenommen.
-
-
-
Ich empfand Dispose schon immer als Fehldesign, aber das ist halt so. Falls IDisposeable richtig implementiert ist, müsste man nichtmal Dispose() aufrufen. Dispose dient nur dazu Objekte eher freizugeben, falls der Finalizer erst später durchlaufen wird. Ggf. müsste null zuweisen schon reichen, bin mir aber nicht sicher.
Ob das im FW überall korrekt gemacht ist, kann ich auch nicht sagen und wage es zu bezweifeln. Deswegen gilt: Immer Dispose() aufrufen, sobald das Objekt unnötig wird. -
Mach dich kundig. Wo, habe ich ja genannt.Daniel Baumert schrieb:
Wieso?
Beschränke dich in diesem wichtigen Punkt nicht auf aus dem Internet aufgeschnappten Halbwissen. -
@Daniel Baumert So, ich hab mir mal ein Control rausgepickt und genauer angesehen:
AppleCheckBox.
Klick mal auf eine solche in der Form und gib ihr einen anderen Font, Size allein genügt schon.
Wie Du siehst, ist es schon nützlich, die Datei zu zerpflücken und alles genau nachzusehen.
Dann findest Du auch das eine (oder andere) Control, was sich blöd verhält.
Allerdings muss das identifiziert werden, einfach mal in der Form rumklicken reicht da nicht.Falls Du diesen Code kopierst, achte auf die C&P-Bremse.
Jede einzelne Zeile Deines Programms, die Du nicht explizit getestet hast, ist falsch
Ein guter .NET-Snippetkonverter (der ist verfügbar).
Programmierfragen über PN / Konversation werden ignoriert! -
-
-
So etwa:Daniel Baumert schrieb:
Dispose
Falls Du diesen Code kopierst, achte auf die C&P-Bremse.
Jede einzelne Zeile Deines Programms, die Du nicht explizit getestet hast, ist falsch
Ein guter .NET-Snippetkonverter (der ist verfügbar).
Programmierfragen über PN / Konversation werden ignoriert! -
Na, wenn ich es in Vb.net schon jahre falsch gemacht habe..
Und jetzt soll ich es aufeinmal richtig machen und dann noch in C# , womit ich gerade beginne.
Edit: Ja, so habe ich es doch nun.. Aber
ich hab ein Problem, da ich nicht genau weiß, wie
ich manche objekte klassenglobal machen soll.
Moment eben. -
Falls Du diesen Code kopierst, achte auf die C&P-Bremse.
Jede einzelne Zeile Deines Programms, die Du nicht explizit getestet hast, ist falsch
Ein guter .NET-Snippetkonverter (der ist verfügbar).
Programmierfragen über PN / Konversation werden ignoriert! -
Okay und zwar habe ich Folgendes vor:
C#-Quellcode
- public MangoEForm()
- {
- this.SetStyle(ControlStyles.AllPaintingInWmPaint | ControlStyles.OptimizedDoubleBuffer | ControlStyles.UserPaint | ControlStyles.ResizeRedraw | ControlStyles.SupportsTransparentBackColor | ControlStyles.FixedHeight, true);
- this.UpdateStyles();
- BackColor = Color.FromArgb(25, 25, 25);
- this.B = new Bitmap(Width, Height);
- this.G = Graphics.FromImage(B);
- }
- private Bitmap B;
- private Graphics G;
- protected override void OnSizeChanged(EventArgs e)
- {
- this.B = new Bitmap(Width, Height);
- this.G = Graphics.FromImage(B);
- base.OnSizeChanged(e);
- }
Fehler:
Ungültiger Parameter. Codezeile:
Weiß jemand, wie man das lösen kann? Der Konstruktor wird doch eigentlich zuerst aufgerufen.
Dachte eben, dass man lieber klassenglobale Variablen erstellt und eben auf diese
Zeichnet. -
Du hättest was verstanden.Daniel Baumert schrieb:
Dachte eben
Ich habe diePensundBrushesin Verdacht.
Falls Du diesen Code kopierst, achte auf die C&P-Bremse.
Jede einzelne Zeile Deines Programms, die Du nicht explizit getestet hast, ist falsch
Ein guter .NET-Snippetkonverter (der ist verfügbar).
Programmierfragen über PN / Konversation werden ignoriert! -
- "klassenglobalität" gibt es nicht. zumindest dein Beispiel enthält nichts, was damit in Verbindung zu bringen wäre, egal wie frei man "klassenglobal" auszulegen bereit ist.
Grundlagen: Fachbegriffe - Der Fehler sagt dir doch, was zu tun ist: Überprüfe die Parameter.
- "klassenglobalität" gibt es nicht. zumindest dein Beispiel enthält nichts, was damit in Verbindung zu bringen wäre, egal wie frei man "klassenglobal" auszulegen bereit ist.
-
Okay, also ich hab jetzt die relevanten Controlklassen bearbeitet
mit euren Tipps und der RAM Verbrauch ist enorm gesunken, man bekommt
den auch nicht mehr auf 150, maximal 26(da ich nicht alle Klassen verbessert habe bis jetzt).
Dennoch wird noch nicht richtig gezeichnet auf der einen Tabpage.
Vielleicht mag es sich jemand nochmal ansehen.
-
Ähnliche Themen
-
8 Benutzer haben hier geschrieben
- Gast (35)
- RodFromGermany (24)
- ErfinderDesRades (7)
- Gonger96 (4)
- Trade (3)
- nafets (2)
- Luki-Progger (1)
- Counterbug (1)
