C++删除地图中的指针

Question

使用地图中的指针删除堆项目

我有这张地图:

map<string, Plaats*> plaatsen;

在一个函数中,我将这个地方添加到这个地图:

Plaats * fromPlace = new Plaats(from);
Plaats * toPlace = new Plaats(to);
auto insertedFrom = plaatsen.insert(pair<string,Plaats*>(from,fromPlace));
auto insertedTo = plaatsen.insert(pair<string,Plaats*>(to,toPlace));
//delete from or to if they are not inserted
if(!insertedFrom.second){
    delete fromPlace;
}
if(!insertedTo.second){
    delete toPlace;
}

如果元素添加到我的地图中,我需要在析构函数中删除它.

KortstePad::~KortstePad(){
    //delete every item in plaatsen
    for(pair<string,Plaats*> place : plaatsen){
        //Plaats *p = place.second;
        delete place.second;
        place.second = nullptr;
    }

    for(pair<string,Plaats*> place : plaatsen){
        Plaats *p = place.second;
        cout << (p == nullptr) << endl;
    }
}

看来我的代码没有删除我的地方,因为这是程序的输出:

0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0

为什么这段代码没有设置指向nullptr的指针？我是否将局部变量设置为nullptr？

Answer 1

您应该在循环中使用对值类型的元素的引用,而不是使用副本.例如

for ( pair<const string,Plaats*> &place : plaatsen){
    //Plaats *p = place.second;
    delete place.second;
    place.second = nullptr;
}

还要考虑到密钥应具有限定符const pair<const string,Plaats*>.