C++, wie zweideutig bewegen Bauer zu vermeiden

Question

Ich habe eine Klasse mit ein paar Felder, assignment c-tor und move c-tor:

class A{ 
    std::vector<int> numbers; 
    int k; 


public: 

    A(std::vector<int> &&numbers, const int k): 
     numbers(numbers), // fast 
     k(k) 
    { 
     // logic 
    } 

    A(const std::vector<int> &numbers, const int k): 
     A(std::move(std::vector<int>(numbers)), k) // copy-and-move vector 
    { 
     // empty 
    } 
}; 

Ich möchte logic halten in einem c-tor und es von anderen nennen. Auch ich möchte schnell move-semantics unterstützen. Und ich muss explizit copy-and-move Argumente in der assignment c-tor.

Gibt es eine Möglichkeit, solche verschachtelte Konstruktion zu vermeiden und alle Vorteile, die ich oben aufgeführt habe, zu behalten?

Answer 1

Sie könnten einen Konstruktor auf die andere übertragen:

struct A 
{ 
    A(const std::vector<int> & v) : A(std::vector<int>(v)) {} 

    A(std::vector<int> && v) 
    : v_(std::move(v)) 
    { 
     // logic 
    } 

    // ... 
}; 

Der bewegliche Konstruktor jetzt so schnell ist, wie es sein kann, und die Kopier Konstruktor Kosten einen Zug, als wenn Sie beide Konstrukteure buchstabieren. Wenn Sie bereit sind, einen zusätzlichen Schritt zu zahlen, obwohl, könnte man genauso gut haben nur einen einzigen Konstruktor:

struct A 
{ 
    A(std::vector<int> v) 
    : v_(std::move(v)) 
    { 
     // logic 
    } 
}; 

Die Alternative den gemeinsamen Code in eine Funktion zu setzen und rufen, dass von beiden Konstrukteure.