堆损坏试图将元素添加到现有的 std::vector

Question

我有一个 std::vector 包含 BoundingBox 类的元素（std::vector<BoundingBox>，我称之为 BB_Array）。 首先，我使用将在此处简化的函数创建此向量：

BB_Array* Detector::generateCandidateRegions(BB_Array* candidates){
    BB_Array* result = new BB_Array(); // without new here, i get segmentation fault
    BB_Array tempResult;

    // apply tests to candidates and add the good ones to tempResult ... 
    // ... using tempResult.push_back((*candidates)[i]);

    for (int i=0; i < tempResult.size(); i++){
        // apply more tests to tempResult[i] and, if necessary, add it...
        // ... using result->push_back(maxCandidate); 
    }

    return result;
}

此功能有效，根据 valgrind 没有内存泄漏。 现在，该函数需要应用一次（为了提高性能），我需要一个函数来添加元素。这就是我遇到问题的地方。我现在的做法是：

BB_Array* Detector::addCandidateRegions(BB_Array* candidates) {
    BB_Array* result = new BB_Array();
    BB_Array sparseDetections;

    // apply tests and add stuff to sparseDetections using... 
    // ... sparseDetections.push_back((*candidates)[i]);

    for (int i=0; i < sparseDetections.size(); i++){
        // add things to result using result->push_back()
    }

    return result;
}

第二个函数为我提供了我需要添加到之前创建的向量的候选对象：

BB_Array *newCandidates = addCandidateRegions(...);
if (newCandidates->size() > 0){
   denseCandidates->insert(denseCandidates->end(), newCandidates->begin(), newCandidates->end());
   delete newCandidates;
}

现在，这会导致堆损坏，并且程序在大约 500 张图像后崩溃。那么我做错了什么？有更好的方法吗？

我还有一个函数可以在之后从向量中删除元素，但我想我可以在完成添加部分后弄清楚。

编辑：忘记输入错误信息：

*** Error in `./C_Arnoud_Calibrated_Detector': malloc(): smallbin double linked list corrupted: 0x0000000001f4eed0 ***
Aborted (core dumped)

不会每次都在同一个迭代中发生，有时我会遇到分段错误。

编辑 2：我今天修好了。没有更多的堆问题。我可能很累并且在一个特定的场景中使用了错误的索引，所以在成千上万的迭代中有时会发生意想不到的事情并且它破坏了一切。感谢大家的建议，如果您使用对象检测器，现在可以安全使用 =)。
https://github.com/CArnoud/C_Arnoud_Calibrated_Detector

Answer 1

首先，您是否使用比您更老的编译器？

是的 -> 停止这样做。你为什么这么讨厌自己？

否 -> 好消息，你的教授“教”给你的关于 C++ 的一切都是错误和错误的。编译器非常擅长返回值优化。让你的函数返回值，它们将被移动而不是复制，这基本上是免费的，并停止在应用程序代码中这种新/删除/原始指针的疯狂。

BB_Array Detector::generateCandidateRegions(BB_Array& candidates){
    BB_Array result; // Use values.
    BB_Array tempResult;

    // Do whatever the heck you want.

    for (int i=0; i < tempResult.size(); i++){
        // apply more tests to tempResult[i] and, if necessary, add it...
        // ... using result.push_back(maxCandidate); 
    }

    return result;
}

BB_Array newCandidates = addCandidateRegions(...);
if (newCandidates.size() > 0){
   denseCandidates.insert(denseCandidates.end(), newCandidates.begin(),
                          newCandidates.end());
}

轻松的内存管理，轻松的生活。