c++ - 复制后在子类中丢失指针

Question

我有 2 个类:

class CCandidate {
public:
    float score;
    BitSet documents;
    std::vector<std::vector<int> > phrases;
    int cardinality;

    /** For merging. */
    CCandidate()
    : score(0.0), documents(1), phrases(1), cardinality(0) {}

    /** */
    CCandidate(
    std::vector<int>& phraseIndices, BitSet& documents,
    int cardinality, float score) {

    this->phrases.reserve(1);
    this->phrases.push_back(phraseIndices);

    this->documents = documents;
    this->score = score;
    this->cardinality = cardinality;
    }
};

class PCandidate {
public:
    CCandidate * topics;
    float coverage;

    bool selected;
    bool mostGeneral;
    bool mostSpecific;

    PCandidate(CCandidate * c, float coverage)
    : topics(c), coverage(coverage), 
      selected(true), mostGeneral(true), mostSpecific(true) {}
};

在另一个使用这些类的类中，我有这样的东西:

// ...
std::vector<std::shared_ptr<PCandidate> > phrases(mergeList.size());

for (size_t i = 0; i < mergeList.size(); i++) {
    CCandidate * cc = baseTopics.at(mergeList.get(i));
    std::wcout << cc->toString() << std::endl;
    float coverage = cc->cardinality / result->cardinality;
    std::wcout << "coverage=" << coverage << std::endl;
    phrases.push_back(std::make_shared<PCandidate>(PCandidate(cc, coverage)));

    std::for_each(phrases.begin(), phrases.end(),
        [&](const std::shared_ptr<PCandidate>& pc) { 
            std::wcout << pc->toString() << " "; }); // error
    }
}

anotherMethod(phrases);

// ...

CCandidate cc 一切正常(目前在这个版本中它是一个原始指针)，我可以打印它的内容(方法 toString() 未在此处复制)并且一切正常。然后，我使用 make_shared 构建了 PCandidate 对象，将其插入短语 vector 中，当尝试访问该“短语” vector 以向我显示 Pcandidate 的内容时，主题集群出现了段错误。

我不能做类似的事情

std::wcout << ptr->topics->phrases.size() << std::endl

其中 ptr 是指向 PCandidate 的指针。 topics 是指向包含短语 vector 的 CCandidate 的指针。

它会给我

==10013== Invalid read of size 8

查看 CCandidate 中短语 vector 的大小。

我有点迷茫，因为我现在不知道在哪里可以找到问题，从昨天开始就坐在这里。这可能是一个该死的初学者错误。是否缺少缺少的复制构造函数/赋值运算符？如果是，它们应该是什么样子？例如复制整个短语 vector ，像深拷贝？我认为到目前为止默认的复制/分配应该没问题。

如果有人能告诉我错误或如何解决这个问题，那就太好了!提前感谢您的宝贵时间!

Answer 1

您最初使用 mergeList.size() NULL 共享指针填充您的 phrases vector ，然后推送 real 在 之后。

std::vector<std::shared_ptr<PCandidate> > phrases(mergeList.size());

因此 vector 中的第一个 mergeList.size() 指针为 NULL。失去初始大小。

std::vector<std::shared_ptr<PCandidate> > phrases;

如果你想保留容量，你可以，但最终共享指针仍然必须通过它们的引用计数算法。我会跳过它，只执行上述操作。