为什么 itr = vector.erase(itr) 在循环中导致内存损坏？我在这里做错了吗？答案

【问题标题】：why itr = vector.erase(itr) in a loop causing memory curruption? Am I doing something wrong here?为什么 itr = vector.erase(itr) 在循环中导致内存损坏？我在这里做错了吗？
【发布时间】：2013-11-12 04:36:27
【问题描述】：

以下代码在调用vector.erase(itr) 时偶尔转储内核。您能否指出我在这里做错了什么？

    std::vector<Ntfy>::iterator itr = NtfyVector.begin();
    for ( ; itr != NtfyVector.end(); )
    {
            if (certaion condition) {
                    itr = NtfyVector.erase(itr);
                    continue;
            } else {
                    itr++;
            }
    }

以下是 GDB 跟踪：

(gdb) 在哪里

#0 0x000000000048cea5 in __gnu_cxx::new_allocator::destroy (this=0x7f0890, __p=0x7adbbf0) at /usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../ ../include/c++/4.1.2/ext/new_allocator.h:107

#1 0x000000000048da34 in std::vector >::erase (this=0x7f0890, __position={_M_current = 0x7adbc40}) 在/usr/lib/gcc/x86_64-redhat-linux/4.1.2/../../../../include/c++/4.1.2/bits/vector.tcc:115

我正在使用 g++ (GCC) 4.4.7 20120313 (Red Hat 4.4.7-3)。

完整代码如下：

void NtfyHandle (void) {

std::vector<Ntfy>::iterator itr = NtfyVector.begin();
for ( ; itr != NtfyVector.end(); )
{
    PF_BOOL bRetry = PF_FALSE;
    switch (itr->GetOperation()) {
        case PF_SERVICE_ADD:
            {
                bRetry = LISwithState(itr->GetServiceName(), itr->GetNewState(), itr->GetInvokeId());
            }
            break;
        case PF_SERVICE_REM:
            {
                liPsPlatform->SendInvokeResponse(itr->GetInvokeId(), PF_STATUS_SUCCESS);
            }
            break;
        default: break;
    }
    if (PF_FALSE == bRetry) {
        // Delete the notification from the vector since retry isn't required
        itr = NtfyVector.erase(itr);
        continue;
    } else {
        itr++;
    }
}

}

【问题讨论】：

什么是haNtfyIter？你为什么不使用std::remove_if？你为什么不显示你的实际代码？
抱歉错字。这是一个实际的代码。为了简单和精确，我尝试替换奇怪的变量名称和不相关的代码。
使用remove_erase_idiom
不，这不是实际代码。您没有声明 if (certaion condition)。如果你有一段代码不工作，而且你不知道是什么导致它不工作，那么你怎么知道哪些部分是不相关的？我们感谢将代码减少到最小示例的努力，但请确保该减少的示例仍然存在问题，否则我们只是在追逐鬼魂。 sscce.org
这里是完整的代码：

标签： c++ g++ gdb

【解决方案1】：

似乎与给定循环无关，但 Ntfy::~Ntfy 析构函数可能有问题？顺便说一下，'erase-remove' 习惯用法可以让代码更整洁。

【讨论】：

谢谢！我会尝试“擦除删除”成语。