Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Clear a TList or a TObjectList

I'm a bit puzzled of what to use for storing objects in a list. Until now I have used TList and freed each item in a loop. Then I discovered TObjectList that do this automatically from Free. Then I saw this from the doc of TList.Clear:

Call Clear to empty the Items array and set the Count to 0. Clear also frees the memory used to store the Items array and sets the Capacity to 0.

So it is basically the same. So

for TList

mylist.Clear;
myList.Free;

is the same as for TObjectList?

myList.Free;

Can TObjectList only be used for items as classes or can I store records?

like image 934
Roland Bengtsson Avatar asked May 02 '12 08:05

Roland Bengtsson


4 Answers

1. TList won't free the elements, with both Clear or Free.

aList.Clear;

Will just set aList.Count := 0 without freeing the aList.Items[] elements. So you'll leak memory. You'll need an explicit free as such:

for i := 0 to aList.Count-1 do
  TObject(aList[i]).Free;

But this is what TObjectList does... :)

About TObjectList, it is worth saying that TObjectList.Destroy is calling Clear.

So

aObjectList.Clear;
aObjectList.Free;

is exactly the same as

aObjectList.Free;

2. To store a list of records, you can use a dynamic array.

You'll get all TList methods (and more) with our dynamic array wrapper. That is, Add / Delete / Clear / Count / IndexOf / Find...

It has built-in serialization features (in binary or JSON), automated sorting and comparison (using RTTI) which are not existing with a TList/TObjectList. From Delphi 5 and later.

With more modern version of Delphi, you may use generics to handle the dynamic array, if you do not want to use a third-party library.

like image 193
Arnaud Bouchez Avatar answered Oct 24 '22 12:10

Arnaud Bouchez


It's not the same TList.Clear only frees the memory allocated to store the pointers, not objects they are pointing to. To avoid memory leaks you need to free the memory yourself - as you have been doing - or use TObjectList.

To answer the second question, TObjectList doesn't support storing records. You need to use TList (or something else) in that case.

like image 34
Ondrej Kelle Avatar answered Oct 24 '22 11:10

Ondrej Kelle


Read what the documentation is saying more carefully:

Clear also frees the memory used to store the Items array

Only the memory for the array itself is freed, not the memory used by individual elements inside the array.

like image 27
Remy Lebeau Avatar answered Oct 24 '22 11:10

Remy Lebeau


If you are using a recent version of Delphi I suggest that you use a generic list.

Right now you probably need to do a lot of casting when you use the objects from the list. With a generic list you don't have to do that anymore.

For instance if you have:

TMyObject = class(TObject);

Then you make the list like this:

TMyObjectList = TObjectList<TMyObject>;

There is an article in the Embarcadero Wiki:

http://docwiki.embarcadero.com/CodeExamples/XE8/en/Generics_Collections_TObjectList_(Delphi)

like image 22
Birger Avatar answered Oct 24 '22 12:10

Birger