別のスレッドで実行されている関数が奇妙な出力を生成する

Question

こんにちは、私はC++でマルチスレッド化するのが初めてです。私は別のスレッドで関数を実行するためにC++ 11で利用可能なスレッドクラスを使用していますが、何とか関数から取得している出力は非常に扱いにくいです。これは、異なるスレッドが同じ変数を同時に実行しようとしている可能性があり、競合を引き起こす可能性があるからです。正しい出力が得られるようにコードを修正する方法を提案してください。私は何をしようとしているのかのサンプルコードを投稿しています。これは元のコードではありませんが、元のコードが長すぎて投稿できないため、元のコードの流れが表示されますが、問題は同じです。

#include<iostream> 
    #include<thread>  

    using namespace std; 

    typedef struct { 
     int thread_id; 
     char *message; 
    }threadData; 

    int display(threadData *tData){ 
     threadData *my_data; 
     my_data = (threadData *) tData; 

     cout << "Thread ID: " << my_data -> thread_id << endl; 
     cout << "Message: " << my_data -> message << endl; 

     return 0; 
    } 

    int main(){ 

     threadData *data; 

     data = (threadData *)malloc(sizeof(threadData)); 
     data->thread_id = 12; 
     data->message = "This is the message"; 
     for (int i = 0; i<10; i++) 
     { 
      std::thread t1(display, data);  
      t1.detach(); 
     } 
     return 0; 
    } 

出力：スレッドとして

Thread ID: 12 
    Message: This is the messageThread ID: 
    12 
    Message: This is the message 
    Thread ID: 12 
    Message: This is the message 
    Thread ID: 12 
    Message: This is the message 
    Thread ID: 12 
    Message: This is the message 

Answer 1

は、共有リソースへのアクセスを保護する必要が 、最初に実行されるguarunteeしません。 これを行う最も簡単な方法は、ミューテックスを使用する方法です。

std::mutex g_i_mutex; // protects g_i 

typedef struct { 
    int thread_id; 
    string message; 
}threadData; 

int display(threadData *tData) 
{ 
std::lock_guard<std::mutex> lock(g_i_mutex); 
threadData *my_data; 
my_data = (threadData *) tData; 

cout << "Thread ID: " << my_data -> thread_id << endl; 
cout << "Message: " << my_data -> message << endl; 

return 0; 

}

出力：

スレッドID：12 メッセージ：このメッセージ スレッドIDである：12 メッセージ：これは、メッセージ スレッドIDである：12 メッセージ：これはメッセージです スレッドID：12 メッセージ：これはメッセージです

スレッディングの概念について少しお読みになることをお勧めします。その背後にある概念は単純ではなく、単に準備が整ったソリューションを手に入れても長期的にはあなたを助けません。

Answer 2

私が読んだのは、ループが10回実行されると予想されますが、主な機能ではすべてのスレッドが完了するのを待っていなかったためです実行するチャンスを持つスレッド。 'main'は、すべてのスレッドが作業を完了するのを待つためにしばらくスリープ状態にする必要があります。

ここで競合状態は見られませんでした。なぜなら、すべてのスレッドがただ読んでいて、誰もthreadDataに書き込んでいないからです。