AW: VirtualSourceTree selektierte Nodes löschen?
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. |
AW: VirtualSourceTree selektierte Nodes löschen?
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.
|
AW: VirtualSourceTree selektierte Nodes löschen?
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. |
AW: VirtualSourceTree selektierte Nodes löschen?
Zitat:
Delphi-Quellcode:
ist nebenbei bemerkt keine sehr schnelle Operation und kann bei extrem vielen Nodes deine Performance schon ziemlich verschlechtern. Hat es irgendeinen Grund, warum du die Objekte überhaupt in einer
IndexOf
Delphi-Quellcode:
ablegst? Ich persönlich gehe immer so vor, dass ich beim Einfügen einer Node mein Datenobjekt/Datenrecord mit
TObjectList
Delphi-Quellcode:
erstelle und zuweise. Im
Create
Delphi-Quellcode:
hole ich mir dann mit
OnFreeNode
Delphi-Quellcode:
den direkten Zeiger auf das Objekt und gebe es dort entsprechend mit
GetNodeData
Delphi-Quellcode:
wieder frei.
Free
Oft bietet es sich sogar an mit Records zu arbeiten. Dort musst du zu Begin einmal
Delphi-Quellcode:
setzen. In diesem Falle verwaltet der VST den Speicher tatsächlich vollkommen von alleine. Lediglich ein
NodeDataSize := SizeOf(TDataRecord)
Delphi-Quellcode:
im
Finalize
Delphi-Quellcode:
ist teilweise erforderlich (wenn dein Records Strings, Interfaces oder andere managed types enthält).
OnFreeNode
|
AW: VirtualSourceTree selektierte Nodes löschen?
Wir drehen uns im Kreis.
|
AW: VirtualSourceTree selektierte Nodes löschen?
Zitat:
Was ist denn jetzt richtiger? Mir ist nur wichtig, dass ich immer vollen Zugriff auf alle Daten habe. Daher dachte ich, wäre eine ObjectList nicht schlecht. Zitat:
Zitat:
Delphi-Quellcode:
Und so erzeuge ich meine Nodes
// ObjectList mit OwnObjects auf FALSE
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.Delete(idx); Data.Free; end;
Delphi-Quellcode:
type
TVSTData = class(TObject) Col1, Col2, Col3, Col4, Col5, sDescription: string; Icon: Byte; end; TVSTDataClasses = TObjectList<TVSTData>; var aVSTDataClasses: TVSTDataClasses; ... procedure TForm2.Button1Click(Sender: TObject); var aVSTData: TVSTData; i: Integer; begin if not Assigned(aVSTDataClasses) then aVSTDataClasses := TVSTDataClasses.Create(False); for i := 0 to 5 do begin aVSTData := TVSTData.Create; aVSTData.Col1 := IntToStr(i); aVSTData.Col2 := IntToStr(Random(1000)); aVSTData.Col3 := IntToStr(Random(1000)); aVSTData.Col4 := IntToStr(Random(1000)); aVSTData.Col5 := IntToStr(Random(1000)); aVSTData.Icon := Random(ImageList2.Count - 1); if i mod 3 = 0 then aVSTData.sDescription := 'Das ist ein ganz langer String - ok so lang ist er nicht!' else aVSTData.sDescription := ''; aVSTDataClasses.Add(aVSTData); end; VST.RootNodeCount := aVSTDataClasses.Count; end; |
AW: VirtualSourceTree selektierte Nodes löschen?
Zitat:
Zitat:
|
AW: VirtualSourceTree selektierte Nodes löschen?
Ehrlich gesagt - ich weiß es nicht ;)
Am liebsten würde ich die Daten direkt im Node speichern aber das macht man ja wohl auch nicht. Pointer-Gefummel möchte ich nicht mehr. |
AW: VirtualSourceTree selektierte Nodes löschen?
Zitat:
Delphi-Quellcode:
und einmal residiert der Zeiger in der
GetNodeData
Delphi-Quellcode:
. Dein
TObjectList
Delphi-Quellcode:
bezieht sich ja aber trotzdem auf das konkrete Objekt.
Free
Delphi-Quellcode:
macht das Gleiche wie
Data := Node.GetData<TVSTData>;
... Data.Free;
Delphi-Quellcode:
aVSTDataClasses[aVSTDataClasses.IndexOf(Data)].Free
|
AW: VirtualSourceTree selektierte Nodes löschen?
Dann ist das also doch so wie ich vermutet habe.
Deine Methode mit den Records, hat die irgendwelche Vorteile gegenüber einer TObjectList mit einer Klasseninstanze pro Node/Datensatz? Am Rande. Wenn ich OwnObjects auf False habe, dann kann auch auch genau so gut eine TList nehmen und mit die paar Byates overhead sparen :P |
Alle Zeitangaben in WEZ +1. Es ist jetzt 06:59 Uhr. |
Powered by vBulletin® Copyright ©2000 - 2024, Jelsoft Enterprises Ltd.
LinkBacks Enabled by vBSEO © 2011, Crawlability, Inc.
Delphi-PRAXiS (c) 2002 - 2023 by Daniel R. Wolf, 2024 by Thomas Breitkreuz