Wie gebe ich den Speicher bei Arrays oder Listen wieder frei?

Hallo,

ich bin jetzt auf einen Fehler in meinen Applikationen gestoßen nachdem 2 Kunden öfter einen Memory Overflow hatten. Alle Protokolle haben nichts ausgesagt.

Jetzt habe ich festgestellt dass es durch Arrays / Lists kommt deren Speicher nicht freigegeben wird.

Hintergrund : Vor langer Zeit habe ich nach einer Möglichkeit gesucht um Bildschirmlisten mit großen Datenmengen sehr schnell anzuzeigen. Etwas Fertiges gab es damals nicht, deshalb habe ich das wie folgt selber gebaut :

Virtuelles ListView, es werden die ersten 30 Zeilen angezeigt, dann wird "Fertig" vorgegaukelt. In einem eigenen Thread wird im Hintergrund ein Array mit den Items gefüllt. Die Zeilen werden nur hergestellt wenn man tatsächlich hinblättert (Lazy Loading).
So wird eine Liste mit über eine Millionen Datensätzen in einer Sekunde angezeigt.
Nur wenn man nach einer Spalte sortiert dauert es länger.

Die ListViewItems speichere ich in einem Array, das ist der Klasse als private definiert ist.

C#-Code:

private ListViewItem[] adr; private void InitList() {     adr = new ListViewItem[anzzeilen];     Thread t = new Thread(ArrayFuellen);     t.Start(); }

Auch nach Beenden der Klasse und Dispose bleibt der Speicher durch das Array belegt. Auch wenn der Thread beendet wird.

Auch ein Umbau auf List<> ergibt keine Änderung.

Alle Bordmittel von .NET schlagen fehl, der Speicher bleibt auf jeden Fall belegt.

Übrigens : Ist .NET 3.5.

Welche Möglichkeiten gibt es den Speichern freizugeben ?

Oder mehrere ListViewItems so zu speichern dass der Speicher wieder freigegeben wird ?

Grüße Bernd

Wenn es noch irgendwo eine Referenz auf dieses Array gibt, wird der Speicher nie freigegeben.

Also suche, wo es noch Referenzen gibt und sorge dafür, dass die nicht erhalten bleiben, wenn aufgeräumt werden soll.
Events sind da ein beliebter Fehler, denn die sorgen zwangsläufig dazu, dass deine Instanz (die wiederum das Array hat) bei der Instanz, die das Event pflegt, referenziert bleibt.

Zitat:

Auch nach Beenden der Klasse und Dispose bleibt der Speicher durch das Array belegt

Eine Klasse kann nicht "beendet" werden. Solange noch Referenzen existieren, räumt der GC nicht auf, völlig egal, was Du in deinem Dispose (was ja auch nur eine dumme Methode ist) tust.
Du könntest im Dispose die Variable mit dem Array auf null setzen, das funktioniert aber nur, wenn es keine weiteren Referenzen gibt. Außerdem ist das vermutlich noch nicht das eigentliche Problem, denn irgendetwas hält eine Referenz auf das Objekt, wo das Array drin ist und diese Stelle musst Du finden.

Zitat:

Welche Möglichkeiten gibt es den Speichern freizugeben ?

Alle Referenzen entfernen, danach räumt der GC von alleine auf.
Wenn der GC nicht aufräumt, gibt's noch Referenzen.


PS:
Und Th69 hat natürlich Recht: Es kann alles noch an der ListView hängen, dann hast Du damit auch eine Referenz, die Du aufräumen musst.
Da sollte es aber reichen, die Liste mit den ListItems zu leeren und/oder die DataSource auf null zu setzen.

Hallo BerndFfm,

die Ursache / Lösung der anderen ist die sauberste Art und Weise das zu handhaben.
Tipp: VS Performance Profiler -> .NET Object Allocation Tracking, dort kannst du die "Wurzeln" betrachten.

Sonst ev. auch  Weak References.

mfG Gü

Nein, noch nicht.

Habe erstmal Quick and Dirty eine Meldung ins Programm eingebaut dass man das Programm neu starten soll wenn mehr als 1000 MB belegt sind.

Ich werde berichten.

Grüße Bernd

Die Anzeige erfolgt mit einem ListView im VirtualMode.

Das ist WinForms.

Grüße Bernd

In ArrayFuellen wird nichts besonderes gemacht, auch keine Events abonniert.

Ich muss das mal genau analysieren wenn ich mehr Zeit habe.

Ich werde berichten.

Grüße Bernd

C#-Code:

...     Liste.VirtualMode = true;     Liste.RetrieveVirtualItem += new System.Windows.Forms.RetrieveVirtualItemEventHandler(Liste_RetrieveVirtualItem);     Liste.VirtualListSize = anzzeilen; ... private ArrayFuellen() {     int m = 0;     foreach(row ...)     {         ListViewItem lvItem = new ListViewItem(inhalt);         for (int i = 1; i < anzspal; i++)         {             lvItem.SubItems.Add(...);         }         adr[m] = lvItem;         m++;     } } private void Liste_RetrieveVirtualItem(object sender, RetrieveVirtualItemEventArgs e) {     if (adr[e.ItemIndex] != null)         e.Item = adr[e.ItemIndex];     else     {         e.Item = new ListViewItem("");         for (int i = 1; i < anzspal; i++) e.Item.SubItems.Add("");     } }

Noch als Info, für die Virtualiserung (hatte das früher auch mal in WPF gebraucht), hatte ich damals das eingesetzt  https://github.com/dotnetprojects/VirtualCollection (stammt nicht von mir, hab es nur geforkt und ein paar dinge gefixt)