Reply to vector von abstrakter Klasse - wie implementieren? on Sat, 26 Sep 2015 19:54:03 GMT

Ramanujan — Sat, 26 Sep 2015 19:54:03 GMT

Ich hab eine abstrakte Klasse X und eine weitere Klasse Y, die einen std::vector mit Instanzen von X besitzt. Darüber hinaus, soll es in Y eine Methode, um weitere Instanzen zu X hinzuzufügen.

Mein bisheriger Ansatz war:

class Y {
public:
    std::vector> v;

    void add(X* obj) {
        v.emplace_back(obj);
    }
}

Ich finde den Ansatz nicht so gut, weil sowas wie

Y y;
y.add(new X_subclass());

so aussieht, als hätte man vielleicht ein delete vergessen.

Gibt es bessere Möglichkeiten, mein Vorhaben zu implementieren?

Reply to vector von abstrakter Klasse - wie implementieren? on Fri, 02 Oct 2015 06:32:48 GMT

theta — Fri, 02 Oct 2015 06:32:48 GMT

#include 
#include 
#include 

class X
{
public:
  virtual ~X() {}
};

class X_subclass
  : public X
{};

class Y
{
  std::vector> v;

public:
  void add(X* obj)
  {
    v.emplace_back(obj);
  }

  void add(std::unique_ptr obj)
  {
    v.emplace_back(std::move(obj));
  }

  template 
  void add(Args&& ...args)
  {
    v.emplace_back(std::unique_ptr(new T(std::forward(args)...)));
  }
};

int main()
{
  Y y;
  y.add(new X_subclass());
  y.add(std::unique_ptr(new X_subclass()));
  y.add(std::make_unique()); // mit C++14 oder selbstgemachtem make_unique<..>(..)
  y.add();
}

// edit
Beispiel vervollständigt und Variante mit Variadic-Templates hinzugefügt. Der Ansatz mit Variadic-Templates entspricht im Wesentlichen einem selbstgemachten und versteckten std::make_unique<..>(..).

// edit1
Formatierung bereinigt

// edit2
Bei std::forward<..> das fehlende Template-Argument ergänzt