変数を使用する関数を並列化する方法

Question

ランダムURLを生成する関数があり、ファイルをダウンロードしようとしています。

void tryURL() 
{ 
    randURL.clear(); 
    for (unsigned short i = 0; i < urlLength; i++) { 
    randURL = randURL + (alphabet[(int)((double)rand()/(RAND_MAX + 1) * alphabetLength)]); 
    } 
    wcout << randURL << endl; 
    HRESULT getImg = URLDownloadToFile(NULL, LPCWSTR((beginURL + randURL + endURL).c_str()), LPCWSTR((randURL + L"/" + endURL).c_str()), 0, NULL); 
    if (SUCCEEDED(getImg)) 
    { 
    wcout << L"Success" << endl; 
    } 
} 

私は通常、この関数を実行すると、それは細かい作業です：

tryURL(); 
0ybOAt 
tryURL(); 
lTPKaR 
tryURL(); 
Ivi05m 
... 

しかし、私はその瞬間に繰り返し回数を、この機能を実行する必要があります。 私はこの試みた：

thread threads[10]; 

for (int i = 0; i < 10; ++i) { 
    threads[i] = thread(tryURL); 
} 

for (int i = 0; i < 10; ++i) { 
    threads[i].join(); 
} 

をそして、それは常に同じ値

0ybOAt0ybOAt 
0ybOAt 

0ybOAt0ybOAt 

0ybOAt 
0ybOAt0ybOAt 
0ybOAt 
0ybOAt 

そして時にはさんが表示されませんENDL私を返します。

どうすれば修正できますか？私はいつも同じ変数randURLを使用しているので壊れていたと思うが、これを避ける方法はわからない。代わりに、同じ変数を使用しての、tryURLリターン URL作る

Answer 1

：次に

// assuming that URLs are strings 
std::string tryURL() { /* ... */ } 

を、URLを返します非同期計算表すためにstd::future<std::string>のベクトルを作成します。

std::vector<std::future<std::string>>> futures; 
futures.reserve(10); 

for (int i = 0; i < 10; ++i) 
{ 
    futures.emplace_back(std::async(std::launch::async, tryURL)); 
} 

を最後に、メインスレッドのURLを使用します。

for(auto& f : futures) 
{ 
    consume(f.get()); 
} 

StackOverflowの上の

Answer 2

他の質問には、いくつかの手がかりを提供します。

• rand() with multiple threads
• iostream thread safety, must cout and cerr be locked separately?

ランド（）問題を修正する簡単な方法は、URLを生成することです。■メインスレッドとパスでそれらをスレッドに追加します。

void tryURL(std::wstring URL) 
... 
// TODO: Set randURL to a random URL 
threads[i] = thread(tryURL, randURL);