对 struct c++ 使用 new 时出现 Bad_alloc 异常

Question

我正在编写一个查询处理器，它分配大量内存并尝试查找匹配的文档。每当我找到匹配项时，我都会创建一个结构来保存描述文档的两个变量并将其添加到优先级队列中。由于无法知道我会执行此操作多少次，因此我尝试使用 new 动态创建结构。当我从优先级队列中弹出一个结构体时，队列（STL 优先级队列实现）应该调用该对象的析构函数。我的结构代码没有析构函数，因此我假设在这种情况下调用默认析构函数。

但是，我第一次尝试创建 DOC 结构时，出现以下错误：

QueryProcessor.exe 中 0x7c812afb 处未处理的异常： Microsoft C++ 异常：内存位置 0x0012f5dc 处的 std::bad_alloc..

我不明白是什么发生了 - 我是否使用了太多内存以至于堆已满？看来不太可能。好像我以前从未使用过该指针。

所以：首先，我在做什么导致了错误，其次，下面的代码会多次工作吗？我是否需要为创建的每个结构体创建一个单独的指针，或者我可以重复使用相同的临时指针并假设队列将保留指向每个结构体的指针吗？

这是我的代码：

struct DOC{
    int docid;
    double rank;

    public:
        DOC()
        {
            docid = 0;
            rank = 0.0;
        }

        DOC(int num, double ranking)
        {
            docid = num;
            rank = ranking;

        }

        bool operator>( const DOC & d ) const {
           return rank > d.rank;
        }

        bool operator<( const DOC & d ) const {
           return rank < d.rank;
        }
};


//a lot of processing goes on here

        priority_queue<DOC, std::vector<DOC>, std::greater<DOC>> q;

//when a matching document is found, I do this:

rank = calculateRanking(table, num);

    //if the heap is not full, create a DOC struct with the docid and rank and add it to the heap
    if(q.size() < 20)
    {
        doc = new DOC(num, rank);
        q.push(*doc);
        doc = NULL;
    }

    //if the heap is full, but the new rank is greater than the 
    //smallest element in the min heap, remove the current smallest element
    //and add the new one to the heap
    else if(rank > q.top().rank)
    {
        q.pop();

        cout << "pushing doc on to queue" << endl;
        doc = new DOC(num, rank);
        q.push(*doc);
    }

非常感谢，bsg。

Answer 1

为什么要在堆上创建以下结构：

doc = new DOC(num, rank);
q.push(*doc);

首先在堆上创建一个 DOC，然后在队列中存储该对象的副本，随后泄漏动态创建的 DOC< /代码>。

以下就足够了并且不会泄漏：

q.push(DOC(num, rank));

Answer 2

关于异常：类 bad_alloc 有一个成员函数 what()，它返回一个 const char * 到一个包含可读描述的字符串。错误的原因。

编辑：如果您正在考虑存储动态分配的对象，请存储指针而不是对该对象的引用。

Answer 3

当内存不足时，会抛出 std::bad_alloc 。

当您执行 q.pop() 时，您需要释放即将弹出的指针，否则就会泄漏。如果您有很多元素，这可能是您的问题。

Doc *p = q.front();
delete p;
q.pop();

正如其他人提到的，如果您声明您的队列保存 Doc's 而不是 Doc*'s，那么您不必自己管理内存，容器会为您做这件事。

Answer 4

你有内存泄漏。 STL 容器的插入方法存储您传入的类型的副本，因此您无需在堆上分配 doc。

而不是

doc = new DOC(...)
q.push(*doc);
doc = NULL;

执行“

doc(...);
q.push(doc);

或”操作

doc = new DOC(...);
q.push(*doc);
delete doc;
doc = NULL;