Dlaczego funkcja dodawania nie ma wpływu w wątku C++ 11?

Question

Próbuję nauczyć się C++ 11 nitkę i mam następujący kod:

#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; 
} 

Stworzyłem vector która posiada std::thread i wywołać funkcję Dodaj z każdego wątku i przekazać I przez ref. Po tym, co założyłem, że wątek zrobi (dodanie i = i+1), wynik końcowy nie odzwierciedla tego, co chciałem.

---

wyjście: i = 0

oczekiwany wynik: I = 10

Answer 1

Mutex musi być dzielone pomiędzy wątkami, aby uzyskać prawidłowy wynik. I i jest zacieniony przez zmienną pętli, zastąp ją 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; 
}