我有一个基类 BaseObject 和两个派生类 DerivedObject1 和 DerivedObject2。它们共享共同的行为和方法，但 DerivedObject1 有一个额外的方法。我的主类 MyClass 存储(在 std::vector 中)这些类实例的 boost::shared_ptr。 MyClass需要为所有的BaseObject调用commonMethod()，有时需要为所有的DerivedObject1调用additionalMethod()。

class BaseObject
{
  virtual void commonMethod();
}

Class DerivedObject1 : public BaseObject
{
  void commonMethod();
  void additionalMethod();
}

Class DerivedObject2 : public BaseObject
{
  void commonMethod();
}

在 MyClass 中有两个 vector 有什么缺点，一个存储 DerivedObject1 和 DerivedObject2 的所有指针，另一个 vector 仅存储 DerivedObject1 的指针？这意味着我会将所有 DerivedObject1 指针都设置两次。但我认为对不同方法的调用至少会很清楚。

class MyClass
{
  typedef std::vector<std::shared_ptr<BaseObject>> BaseObjectVector;
  typedef std::vector<std::shared_ptr<DerivedObject1>> DerivedObject1Vector;
  BaseObjectVector everything;
  DerivedObject1Vector only_derived1;

  void doSomething()
  {
    for (BaseObjectVector::iterator iter = everything.begin(); iter != everything.end(); ++iter)
    {
      (*iter)->commonMethod();
    }
  }

  void doSomethingForDerivedObject1()
  {
    for (DerivedObject1Vector::iterator iter = only_derived1.begin(); iter != only_derived1.end(); ++iter)
    {
      (*iter)->additionalMethod();
    }
  }
}

我可以想到其他方法来做到这一点，主要是为 DerivedObject1 设置一个 vector ，为 DerivedObject2 使用一个 vector ，但是要调用 commonMethod()，我将不得不迭代这两个 vector 。我原来的解决方案对我来说似乎是最好的，除了一些指针被存储了两次。这有什么缺点？

最佳答案

有趣的问题。 我们在维护不知道是谁写的遗留代码时，有时会遇到这样的情况。

Are there any disadvantages of having two vectors in MyClass … ?

我认为没有机械(或性能)缺点。 如果我们在发布截止日期前苦苦挣扎，我们别无选择，只能选择这种简单的方式。 但是，将相同的 vector 存储两次实际上会降低可维护性，我们应该考虑在未来改进它。

• std::dynamic_pointer_cast(或 boost::dynamic_pointer_cast)[Demo]

如果你需要dynamic_pointer_cast来实现像@Caleth这样的功能的push_back/remove, 从一开始就删除 only_derived1 并不情愿地在 doSomethingForDerivedObject1() 中只应用一个 dynamic_pointer_cast 怎么样？ 它将使 MyClass 更简单。如果以后定义DerivedObject3，所需的修改不会复杂。

void MyClass::doSomethingForDerivedObject1()
{
    for (const auto& obj_i : everything)
    {
      if (auto derived1 = std::dynamic_pointer_cast<DerivedObject1>(obj_i))
        {
           derived1->additionalMethod();
        }
    }
}

void MyClass::doSomethingForDerivedObject3()
{
    for (const auto& obj_i : everything)
    {
      if (auto derived3 = std::dynamic_pointer_cast<DerivedObject3>(obj_i))
        {
           derived3->additionalMethod();
        }
    }
}

• @sagi 提出的虚拟方法[Demo]

声明虚函数BaseObject::additionalMethod()并实现

void DerivedObject2::additionalMethod()
{ 
  /* nothing to do */
}

然后您可以再次删除 only_derived1。 在此方法中，仅当定义了 DerivedObject3 时，您才必须实现 DerivedObject3::additionalMethod()。

但是，虽然这取决于你的构造函数或setter代码，如果也会出现下面的情况

everything;    ->derived2
only_derived1; ->derived1

这个方法还是不够的。

• 更改设计 [Demo]

理想情况下，我们不应该使用公共(public)继承来实现“IS-ALMOST-A”关系中的对象，如 Herb Sutter 所说。 BaseObject、DerivedObject1 和 DerivedObject2 之间的关系如下所示。 由于我不知道您的应用程序的完整代码，所以我可能是错的，但考虑将 DerivedObject1::additionalMethod() 提取为另一个类或函数指针并将其 vector 放入 MyClass 中是值得的作为私有(private)成员(member)。

关于c++ - 将相同共享指针的拷贝存储在不同的 vector 中是一种好习惯吗？，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/52627324/