std :: moveがstd :: listと一緒に動作しない理由

Question

以下のプログラムをコンパイルできません。

void toSin(std::list<double>&& list) 
{ 
    std::for_each(list.begin(), list.end(), [](double& x) 
    { 
     x = sin(x); 
    }); 
} 

int main() 
{ 
    std::list<double> list; 
    const double pi = 3.141592; 
    const double epsilon = 0.0000001; 
    for (double x = 0.0; x < 2 * pi + epsilon; x = x + pi/16) 
    { 
     list.push_back(x); 
    } 
    // Start thread 
    std::thread th(toSin, std::move(list)); 
    th.join(); 
    return 0; 
} 

私は>エラーC2664取得： 'void (std::list<double,std::allocator<_Ty>> &&)'：あなたのコンパイラがここに間違っているように私は感じる 'std::list<double,std::allocator<_Ty>> &&'

Answer 1

に 'std::list<double,std::allocator<_Ty>>' から引数1を変換することはできませんが。崩壊（コピー）された値の型は、値の参照にバインド可能である必要があります。

とにかくthis quote from the documentation

3を見て）新しいのstd ::スレッドオブジェクトを作成し、実行スレッドに関連付けます。実行の新しいスレッドを使用すると、関数の関数の引数としてコピーされますstd::threadのコンストラクタに引数として渡す

std::invoke(decay_copy(std::forward<Function>(f)), decay_copy(std::forward<Args>(args))...); 

基本的には何の実行を開始します。

std::list変数をrvalue参照ではなく値で受け入れるようにすると、関数が正常に機能することもわかります。あなたの意図は、私はそれを行う方法は、通常、ラムダ

std::list<double> lst; 
auto th = std::thread{[&lst]() { 
    toSin(lst); 
}}; 

である。しかし、あなたが使用することもでき、スレッド関数に変数への参照を渡すことであれば、より

---

ためCorrect usage of rvalue references as parametersを参照してくださいstd::ref同じ効果のために。私は個人的にラムダアプローチがより明確であると感じています。

std::list<double> lst; 
auto th = std::thread{toSin, std::ref(lst)}; 

---

Also as correctly pointed out in the comments、あなたはmutexを防ぐ、または私はあなたがおそらくいくつかを見逃していると思い

auto th = std::thread{[&lst]() { 
    toSin(lst); 
}}; 
th.join(); 

// then iterate and print out 

Answer 2

を終了するスレッドを待つべきあなたのコード内の競合状態を持っている#include 、そのコードはVisual Studio 2015で動作します

#include <algorithm> 
#include <list> 
#include <thread> 
void toSin(std::list<double>&& list) 
{ 
    std::for_each(list.begin(), list.end(), [](double& x) 
    { 
     x = sin(x); 
    }); 
} 

int main() 
{ 
    std::list<double> list; 
    const double pi = 3.141592; 
    const double epsilon = 0.0000001; 
    for (double x = 0.0; x < 2 * pi + epsilon; x = x + pi/16) 
    { 
     list.push_back(x); 
    } 
    // Start thread 
    std::thread th(toSin, std::move(list)); 
    th.join(); 
    return 0; 
}