派生对象/智能指针的容器

Question

可以说我有：

class Base {...};
class Derived1 : public Base {...};
class Derived2 : public Base {...};
class Derived3 : public Base {...};

class Store {
public:
    void Add(const Base&);    //Adds mix of Derived1/2/3
private:
    vector<const Base*> vec;    
    vector<shared_ptr<const Base> > vec_smart;
};

//------------------------------------------------------

void Store::Add(const Base& b){
    vec.push_back(&b);    //got sliced
}

当使用指向 Base 的指针向量时，它被切片。 我想我必须使用智能指针，但我无法让它工作。

我试过类似的东西：

auto tmp = make_shared<const Base> (b);
vec_smart.push_back(shared_ptr<const Base>(b));

但是还是被切了。 我做错了什么？

编辑：我在基类中有 << 运算符，它在派生类中调用虚拟打印。

ostream & operator <<(ostream & os, const Base & x) {
    x.print(os);
    return os;
}

void Base::print(ostream& os) const {
   os << "Base";
}

void Derived1::print(ostream& os) const {
   os << "Derived1";
}

当我打电话

cout << b;

在 Store::Add 它输出正常但是当我在它存储后迭代向量时，我得到的是“Base”

ostream & operator <<(ostream & os, const Store & x) {
    for (auto it = x.vec.begin(); it != x.vec.end(); ++it) {
        os << *(*it) << endl;
    }
    return os;
}

这是一个糟糕的设计吗？

Answer 1

我猜你没有将Base的相关方法声明为virtual 。 尝试这个：

struct Base
{
  virtual void print(std::ostream& os) const { os << "Base"; }
  virtual ~Base() {}
};

struct Derived1 : Base
{
  void print(std::ostream& os) const { os << "Derived1"; }
};

等等。 注意我在这里只使用了struct来节省我多次输入public 。

这是一个工作示例：

#include <vector>
int main()
{
  Base b;
  Derived1 d;
  std::vector<const Base*> v{&b, &d};

  for (auto it = v.cbegin(); it != v.cend(); ++it)
  {
    std::cout << *(*it) << std::endl;
  }

}

输出：

Base
Derived1

你可以在这里看到它的实际效果。

编辑您的代码具有未定义的行为，因为您正在存储指向临时文件的指针：

Store a("aaaaaaa");
a.Add(Derived("abc", 5));
a.Add(Derived("def", 6));

此版本按预期工作：

Store a("aaaaaaa");
Derived d0("abc", 5);
Derived d1("def", 6);
a.Add(d0);
a.Add(d1);

编辑 2 ：来自您的评论：

假设我被迫使用 a.Add(Derived("abc", 5)); 在我的实现中我可以用它做什么？

您可以Add模板：

template <typename T>
void Add(const T& elem)
{
  vec.push_back(new T(elem));
}

或者

template <typename T>
void Add(const T& elem)
{
  vec_smart.push_back(std::make_shared(elem));
}