Delete QMap value as a pointer

Question

I have a QMap with key QString and with the value myClass. I've looked through the Qt documentation and decided to define value as pointer to myClass object. But, I'm puzzled with issue of how to remove both key and (pointer object allocated by new operator) value? In case, if I used remove() method, would that be safe in terms of memory release of dynamically allocated object?

Is this a good alternative?

myClass *type = types.take("foo");
delete type;

no that wont delete your instance of myClass and will result in a memory leak — Florian Groetzner, Nov 12 '13 at 17:35
You should use one of the Qt smart pointers, which will call `delete` for you. Every time it's needed, even if an exception occurs. — MSalters, Nov 13 '13 at 13:11

score 0 · Accepted Answer · answered Nov 12 '13 at 18:21

0

As Qt documentation says about QMap::take() function

Removes the item with the key key from the map and returns the value associated with it.

So that your code will remove the key from the map and delete the allocated memory for myClass object.

answered Nov 12 '13 at 18:21

vahancho

20,808
3
47
55

1

Key, as being stored as a value, will be destroyed as soon as the execution will go out of the scope. – vahancho Nov 12 '13 at 18:28

Delete QMap value as a pointer

1 Answers1