私はstd :: move_ptrを移動コンストラクタで移動する必要がありますか？

Question

考慮する： - 一つは実際移動shared_ptr、および他だけコピーshared_ptrにstd::moveを使用する上記のコードで

#include <cstdlib> 
#include <memory> 
#include <string> 
#include <vector> 
#include <algorithm> 
#include <iterator> 
using namespace std; 

class Gizmo 
{ 
public: 
    Gizmo() : foo_(shared_ptr<string>(new string("bar"))) {}; 
    Gizmo(Gizmo&& rhs); // Implemented Below 

private: 
    shared_ptr<string> foo_; 
}; 

/* 
// doesn't use std::move 
Gizmo::Gizmo(Gizmo&& rhs) 
: foo_(rhs.foo_) 
{ 
} 
*/ 


// Does use std::move 
Gizmo::Gizmo(Gizmo&& rhs) 
: foo_(std::move(rhs.foo_)) 
{ 
} 

int main() 
{ 
    typedef vector<Gizmo> Gizmos; 
    Gizmos gizmos; 
    generate_n(back_inserter(gizmos), 10000, []() -> Gizmo 
    { 
     Gizmo ret; 
     return ret; 
    }); 

    random_shuffle(gizmos.begin(), gizmos.end()); 

} 

をGizmo::Gizmo(Gizmo&&)の2つのバージョンがあります。

両方のバージョンが表面上で動作するようです。 1つの違い（私が見ることができる唯一の違い）はmoveバージョンであり、shared_ptrの参照カウントは一時的に増加しますが、一時的に増加します。

私は通常moveのshared_ptrに進みますが、私のコードではっきりしていて一貫性があります。私はここで考慮を欠いていますか？ テクニカルの理由で、1つのバージョンを他のバージョンよりも優先したいですか？

Answer 1

ここでの主な問題は、shared_ptrの余分なアトミック・インクリメントとデクリメントによるパフォーマンスの小さな違いではありませんが、移動を実行しない限り、操作のセマンティクスは矛盾しています。

shared_ptrの参照カウントは一時的なものに過ぎないと仮定していますが、そのような保証はありません。移動元のソースオブジェクトは一時的なものになる可能性がありますが、存続期間が非常に長くなる可能性もあります。それはないshared_ptrからを移動することで、その場合、あなたはまだ、移動のソースと所有権を共有し維持し、もしされている、（std::move(var)を言う）右辺値参照にキャストされた名前の変数可能性があり宛先shared_ptrのスコープが小さければ、指示されたオブジェクトの有効期間が不必要に延長されます。

Answer 2

moveの使用が好ましい：それは参照カウントの余分な原子増分および減分を必要としないので、コピーよりも効率的でなければならない。

Answer 3

私はJames McNellisの答えをアップしました。私は彼の答えについてコメントしたいと思いますが、私のコメントはコメント形式に収まりません。だから私はここに入れている。

shared_ptrのコピーと1つのコピーのパフォーマンスの影響を測定する楽しい方法は、vector<shared_ptr<T>>のようなものを使用して、それらの全体を移動またはコピーして時間を計ることです。ほとんどのコンパイラでは、言語モード（-std = C++ 03や-std = C++ 11など）を指定することによって、移動セマンティクスをオン/オフする方法があります。ここで

は、私はちょうど-O3でテストコードです：

#include <chrono> 
#include <memory> 
#include <vector> 
#include <iostream> 

int main() 
{ 
    std::vector<std::shared_ptr<int> > v(10000, std::shared_ptr<int>(new int(3))); 
    typedef std::chrono::high_resolution_clock Clock; 
    typedef Clock::time_point time_point; 
    typedef std::chrono::duration<double, std::micro> us; 
    time_point t0 = Clock::now(); 
    v.erase(v.begin()); 
    time_point t1 = Clock::now(); 
    std::cout << us(t1-t0).count() << "\u00B5s\n"; 
} 

打ち鳴らすの使用/ libcの++と-std = C++ 03で、これは私のために出力します。

への切り替え

195.368µs 

- std = C++ 11私は得る：

16.422µs 

あなたの走行距離は異なる場合があります。