Vermeiden Copykonstruktor während bevöl Mitglied Vektor

Question

Ich habe eine Klasse A:

class A { 
    int value1; 
    int value2; 
    std::string text; 

public: 
    A(int value1, int value2, std::string text) 
    : value1(value1), value2(value2), text(text) { } 
}; 

und einige "Container" Klasse B:

class B { 
    std::vector<A> objects; 
    ... 

public: 
    ... 
    void addObject(A a) { 
    objects.push_back(a); 
    } 
}; 

Und Code:

B b; 
A a(2, 5, "test"); 
b.addObject(a); 
//I no longer need a from now on 

Mein Problem ist, wie man B::addObject(A a) optimiert, um jedes Kopieren zu vermeiden. Was ich erreichen will, ist ein neues Objekt vom Typ A zu B.objects über B Methode hinzufügen.

Answer 1

Es gibt viele Antworten auf diese Frage. Sie könnten nur Ihre B Funktion move() ändern und move() in dieser Funktion:

void addObject(A a) { objects.push_back(std::move(a)); } 

b.addObject(std::move(a)); // two moves 

Sie Überlastungen von addObject hinzufügen könnte, die von const lvalue Bezug nehmen und durch rvalue Referenz

void addObject(A const& a) { objects.push_back(a); } 
void addObject(A&& a) { objects.push_back(std::move(a)); } 

b.addObject(std::move(a)); // one move 

Sie könnten ein hinzufügen Funktionsvorlage für addObject die emplaces:

template <class... Args> 
void addObject(Args&&... args) { objects.emplace_back(std::forward<Args>(args)...); } 

b.addObject(2, 5, "test"); // zero moves 

---

So oder so, ist es eine völlig unnötige Kopie hier:

A(int value1, int value2, std::string text) 
: value1(value1), value2(value2), text(text) { } 

Sie wollen:

A(int value1, int value2, std::string text) 
: value1(value1), value2(value2), text(std::move(text)) { } 

Answer 2

Sie können den Parameter Typen addObject ändern und emplace_back, verwenden Sie ein Objekt von A zu vermeiden konstruieren, die später nicht verwendet und in die vector kopieren. z.B.

void addObject(int value1, int value2, std::string text) { 
    objects.emplace_back(value1, value2, text); 
} 

und

B b; 
b.addObject(2, 5, "test"); // construct the element directly into the vector without constructing A