のstd ::アトミックとstd :: lock_guardまたは唯一のstd :: lock_guard

Question

私は多くのスレッドで関数fooラン（std::thread t([&]() { foo(a); }）持っている：私は競合状態に対してint x、int yとstd::vector<> vecを保護する必要が

void foo(int a) 
{ 
    if (x && y != a) 
    { 
     ++x; 
     y = a; 
     vec.push_back(std::chrono::high_resolution_clock::now()); 
    } 
} 

を。 最適なソリューションは何ですか？

std::atomicとしてxとyを定義しvecにプッシュする前std::lock_guard(mutex)を使用します。

またはstd::lock_guard全身ifについては？

それとももっと良い解決策ですか？

Answer 1

これは、スレッドが残りの時間に実行している処理（xとyが使用されている）に多少依存しますが、一般的な経験則として、ロックを少なくするほど効果的です。

Answer 2

これ以上の文脈がなければ、確かめるのは難しいですが、yを使用して変更しているようですので、割り当て後までifより前にロックする必要があります。それはstd::atomicで行うことはできません。あなたがコードのその領域をミューテックスしているとすれば、xをアトミックにすることから得られる利点はほとんどありません。しかし、あなたは私たちにそれから何かを読んだことを示さなかったので、言うことは難しいです。