C++ 11 std :: reverse（）のコンビニエンスラッパー

Question

std::reverse()のコンビニエンスラッパーを実装するには、これはC++ 11のrvalue-referencesとmove semanticsを使用する正しい方法ですか？

template <class BIDirContainer> inline BIDirContainer&& reverse(BIDirContainer a) { 
    std::reverse(begin(a), end(a)); 
    return std::move(a); 
} 

コードは、私のテストケースで動作しますが、私はその程度の性能が不明午前：私はここ&&を使用するか、それがunneccesaryである必要がありますか？

Answer 1

私はそれを行うための正しい方法は、あなたの関数から値で返すことだと思います。

template <class BIDirContainer> inline BIDirContainer reverse(BIDirContainer a) { 
    std::reverse(begin(a), end(a)); 
    return a; 
} 

し、それが1つを持っていない場合BIDirContainerに移動コンストラクタを与えます。そして、表現のこの種：

BIDirContainer x = ...; 
BIDirContainer backwards{reverse(x)}; 

はbackwardsにあなたのreverse機能が一時的にaの内容を移動する必要があります。あなたが逆のコピーを作成したい場合は

Answer 2

（rvalue）参照で返信すると、aはローカルオブジェクトなので、ダングリングリファレンスが返されます。価値によって戻り、すべてが「うまくいく」べきです。

Answer 3

ですから、パラメータを変更したい場合は...

template <class BIDirContainer> 
inline BIDirContainer& reverse(BIDirContainer& a) 
{ 
    std::reverse(begin(a), end(a)); 
    return a; 
} 

は：

template <class BIDirContainer> 
inline BIDirContainer reverse(BIDirContainer a) 
{ 
    std::reverse(begin(a), end(a)); 
    return a; 
} 

関数の戻り値は、すでに右辺値（あります具体的には「x値」）。移動セマンティクスを持つ関数に渡すと移動されますが、上記の名前付き戻り値最適化（NRVO）resultがインプレースで構築される場合もあります（移動セマンティクスよりも優れています）。