抽象类的 shared_ptr 向量到副本向量

Question

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

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 来解决这个问题，但应该有更简单的方法来实现这一点。

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

Answer 1

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());
    }
}

Answer 2

我猜测，当您使用 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);
    }
}