VirtualSourceTree selektierte Nodes löschen?

[madas]

Zitat von SneakyBagels:

Ich muss im fertigen Code nicht nur den Eintrag löschen sondern noch ein paar mehr Dinge machen (z.B. ggf. den ObjectList-Eintrag löschen)
Ich möchte neben dem VST noch eine ObjectListe führen, welche die Daten hält.

Bei der Objectliste OwnsObjects auf false setzen und im OnFreeNode vom VST das Object der Liste freigeben.
Bei DeleteNode wird dann OnFreeNode aufgerufen.

[Uwe Raabe]

Ohne das jetzt probiert zu haben, wäre diese vereinfachte Form deines ursprünglichen Ansatzes nicht vollkommen ausreichend?

Delphi-Quellcode:

  Node := VST.GetFirst;
  while Assigned(Node) do begin
    NodeTmp := VST.GetNextSibling(Node);
    if VST.Selected[Node] then begin
      VST.DeleteNode(Node);
    end;
    Node := NodeTmp;
  end;

[SneakyBagels]

(Siehe Edit)

Ungefähr so?
=> ObjectList OwnObjects auf Fale
=> Änderung von Uwe eingebaut

und

Delphi-Quellcode:

procedure TForm2.VSTFreeNode(Sender: TBaseVirtualTree; Node: PVirtualNode);
var
 Data: TVSTData;
 idx: Integer;
begin
 Data := Node.GetData<TVSTData>;

 idx := aVSTDataClasses.IndexOf(Data);
 if idx > -1 then
  aVSTDataClasses.Items[idx].Free;

 // Data.Free;
end;

Danach wird mir noch immer "Count: N ausgegeben" (N = die Anzahl der Nodes vor dem Löschen)
ShowMessage('Count: ' + IntToStr(aVSTDataClasses.Count));


Edit
Dummheit muss bestraft werden. aVSTDataClasses.Delete(idx); und nicht aVSTDataClasses.Items[idx].Free; .

[Aviator]

Auch wenn das aVSTDataClasses.Items[idx].Free; an der Stelle nicht passt hier noch ein kleiner Tipp:

Du kannst das .Items getrost weglassen und einfach aVSTDataClasses[idx].Free; schreiben. Items ist die Standardeigenschaft einer Liste. Somit ist das überflüssig.

Und die Vorteile die du davon hast sind:

• Keine unnötige Schreibarbeit
• Übersichtlicherer SourceCode
• Du kannst so eine Zeile ganz schnell mal für die Benutzung mit einem Array verwenden (auch wenn das jetzt vielleicht nicht so häufig vorkommt)

[DeddyH]

Zitat von SneakyBagels:

Dummheit muss bestraft werden. aVSTDataClasses.Delete(idx); und nicht aVSTDataClasses.Items[idx].Free; .

Und wo wird jetzt die Instanz wieder freigegeben?

[SneakyBagels]

Das weiß ich nicht aber beim Schließen der Demo habe ich keinerlei Memory-leaks.
Ich gehe einfach mal davon aus, dass weil ja jetzt das VST die Speicherverwaltung übernimmt, hier die Instanz beim Aufruf von Data.Free freigegeben wird.

[DeddyH]

Das ist aber ganz schlecht, wenn Du das nicht weißt. Da können schon kleinere Änderungen am Code entweder zu MemoryLeaks oder zu AVs führen. Ich spreche da aus Erfahrung.

[SneakyBagels]

Wie oben geschrieben gehe ich davon aus, dass das VST das jetzt übernimmt, da die ObjectList ja die Daten nicht mehr erbt.
Wenn dem nicht so ist, kannst du mich gerne erschlagen.
Aber selbst wenn ich im OnFreeNode aVSTDataClasses.Delete(idx); weglasse bekomme ich keine Speicherlecks.