Deallocate memory to singleton class which access database ?

[J.Hilk]

@KroMignon IIRC range based for loops is a c++11 feature in itself 🤔

never mind, I think need a break!

[KroMignon]

@J-Hilk said in Deallocate memory to singleton class which access database ?:

IIRC range based for loops is a c++11 feature in itself 🤔

Are you sure? I don't believe that std::map<>::iterator support is comming with C++11.

[jsulm]

@KroMignon I think @J-Hilk misunderstood your code: you are not using range based loop

[Qt embedded developer]

@KroMignon Here inside for loop i need to write

for(std::map<int,SqliteManagerBGSync*>::iterator it = sqliteManagerMap::begin(); it != sqliteManagerMap::end(); ++it)
{
delete it;
}

or what i need to write to deallocate memory ?

[jsulm]

@Qt-embedded-developer said in Deallocate memory to singleton class which access database ?:

delete it;

delete it->second;

of course

[KroMignon]

@Qt-embedded-developer said in Deallocate memory to singleton class which access database ?:

or what i need to write to deallocate ?

Please, take time to read std::map<> documentation.
You are using a map iterator, so it is a container object which gives you access to key and value.

delete it do not make sense!

You want to delete the pointer hold in the value part: delete it->second

[Qt embedded developer]

This post is deleted!

[Qt embedded developer]

@KroMignon Hi when i put sqliteManagerMap::begin() and sqliteManagerMap::end() i am getting error .

like below :

for(std::map<int,SqliteManagerBGSync *>::iterator it = sqliteManagerMap::begin(); it != sqliteManagerMap::end(); ++it)

but when i put sqliteManagerMap.begin() then i am getting no error.

[jsulm]

@Qt-embedded-developer said in Deallocate memory to singleton class which access database ?:

sqliteManagerMap::begin() and sqliteManagerMap::end()

As already explained in your other thread: begin() and end() are non-static methods, so you call them with . not :: !
Please learn C++ and read documentation!

[KroMignon]

@Qt-embedded-developer said in Deallocate memory to singleton class which access database ?:

for(std::map<int,SqliteManagerBGSync *>::iterator it = sqliteManagerMap::begin(); it != sqliteManagerMap::end(); ++it)

Sorry my fault, as @jsulm already told, this are not static class methods. You want to call class instance methode:

for(std::map<int,SqliteManagerBGSync *>::iterator it = sqliteManagerMap.begin(); it != sqliteManagerMap.end(); ++it)

Was a long day yesterday!