Warum hat Add-Funktion keine Auswirkungen in C++ 11 Thread?

Question

Ich versuche, die C++ 11-Thread und folgenden Code müssen lernen:

#include <iostream> 
#include <thread> 
#include <vector> 
#include <mutex> 
#include <algorithm> 

void add(int& i){ 
    std::mutex some_mutex; 
    // std::cout << " I am " << std::endl; 
    std::lock_guard<std::mutex> guard(some_mutex); 
    i++; 
} 


int main(){ 
    int i = 0; 
    std::vector<std::thread> vec_threads; 

    for(int i = 0; i < 10; i++){ 
     vec_threads.push_back(std::thread(add,std::ref(i))); 
    } 

    std::for_each(vec_threads.begin(),vec_threads.end(), 
      std::mem_fn(&std::thread::join)); 
    std::cout<< " i = " << i << std::endl; 
return 0; 
} 

Ich habe eine vector erstellt, die von jedem Thread std::thread und ich rufe die Add-Funktion hält und übergeben i durch ref. Nach dem, was ich angenommen habe, dass der Thread (das Hinzufügen von i = i+1) tun würde, spiegelt das Endergebnis nicht, was ich wollte.

---

Ausgang: i = 0

erwartete Ausgabe: i = 10

Answer 1

Mutex zwischen Threads gemeinsam genutzt werden muss, das richtige Ergebnis zu bekommen. Und i im Schatten von Loop-Variable, ersetzen Sie es durch j.

#include <iostream> 
#include <thread> 
#include <vector> 
#include <mutex> 
#include <algorithm> 

void add(int& i, std::mutex &some_mutex){ 
    // std::cout << " I am " << std::endl; 
    std::lock_guard<std::mutex> guard(some_mutex); 
    i++; 
} 


int main(){ 
    int i = 0; 
    std::vector<std::thread> vec_threads; 
    std::mutex some_mutex; 

    for(int j = 0; j < 10; j++){ 
     vec_threads.push_back(std::thread(add,std::ref(i), std::ref(some_mutex))); 
    } 

    std::for_each(vec_threads.begin(),vec_threads.end(), 
      std::mem_fn(&std::thread::join)); 
    std::cout<< " i = " << i << std::endl; 
    return 0; 
}