2

我有一个带有共享指针的向量：

std::vector<std::shared_ptr<DescriptorsNs::Descriptor> > mDescriptorList;

现在我想要一个 getter 函数，它返回一个带有这些对象副本的向量：

void CatUpdater::getDescriptorList(std::vector<Descriptor*>& descriptorList) const
{
    descriptorList.clear();
    for (auto it = mDescriptorList.begin(); it != mDescriptorList.end(); it++)
    {
        descriptorList.push_back(*it);
    }
}

现在的问题是 Descriptor 是一个抽象类，当我尝试将它们添加到向量时，它正确地指出：

/usr/include/c++/4.7/ext/new_allocator.h|110|error: cannot allocate an object of abstract type ‘DescriptorsNs::Descriptor’|

我可以通过尝试std::dynamic_pointer_cast所有类型的派生类来解决这个问题，但应该有一种更简单的方法来实现这一点。

谁能告诉我将原始对象复制到返回向量的更好方法？

4

2 回答 2

4

你不能让 Descriptor 对象，因为它是抽象的。

但是您可以制作指向这些对象的指针的深层副本。

为此，您的类描述符缺少clone虚拟方法！

将它添加到您的类并在您的派生中正确实现。

class Descriptor {
 ...
  virtual Descriptor* clone() const = 0;
};


class SomeDescriptor : public Descriptor {
 ...
  virtual Descriptor* clone() const { return new SomeDescriptor (*this); }
};

并在将一个向量深拷贝到另一个向量时使用 clone()：

void CatUpdater::getDescriptorList(std::vector<std::shared_ptr<DescriptorsNs::Descriptor> >& descriptorList) const
{
    descriptorList.clear();
    descriptorList.reserve(mDescriptorList.size());
    for (auto it = mDescriptorList.begin(); it != mDescriptorList.end(); it++)
    {
        descriptorList.push_back((*it)->clone());
    }
}
于 2012-09-20T15:33:24.443 回答
1

我猜当你使用shared_ptrs 存储你的对象时，它们要么是大对象，不应该被复制，要么它们代表不应该被复制的东西的单个实例。

在这种情况下，您最好只填充一个 const 指针向量，这将阻止调用者修改指向的对象。

typedef std::shared_ptr<const Descriptor> ConstSharedPtr;

void CatUpdater::getDescriptorList(
                    std::vector<ConstSharedPtr>& descriptorList
                    ) const
{
    descriptorList.clear();

    for (auto it = mDescriptorList.begin(); it != mDescriptorList.end(); it++)
    {
        descriptorList.push_back(it);
    }
}
于 2012-09-21T08:06:14.797 回答