C++ におけるスレッド間での変数渡し

Question

C++にてプログラミングをしています。
スレッド間で配列の中身をコピーしたいです．
具体的には
matrix[N][5]というメインスレッドの変数にN個のスレッドから変数を格納したいのですがうまく格納ができていない状態です.
スレッド内のget_five_number により5つの変数が代入されます
それをN回やることでメインスレッドのmatrixの中身が満たされるようにしたいです。
現在のソースではスレッド内の値は格納されているもののメインスレッドの方には値が入っていません。
どのような解決法でもいいので解決したいです。
よろしくお願いいたします。

void
thread_access(int& num,STACK* matrix)
{
     matrix = cc.get_five_number(&num);
}

//行列
matrix[N][5]
for(int i = 0;i < N;i++){
     //muliti_threading
     threads[i] = std::thread(thread_access,std::ref(num[i]),std::ref(matrix[i]));
}

Answer 1

掲示コードを見る限り、マルチスレッド化以前の問題があるように見えます。まずはシングルスレッドで期待通り動くプログラムを記述すべきです。（誤解を招かぬよう；意地悪でそう言っている訳ではなく、どんな熟練者であってもマルチスレッド処理では絶対にこのステップが必要です。）

で、恐らくこんなコードを書きたいのではないでしょうか？一部はこちらで勝手に推測しています。

void thread_access(int& num, STACK* matrix)
{
  // get_five_number関数は5要素配列へのポインタを返す(?)
  // 同時に変数numの値書き換えを行っている(?)
  STACK* result = cc.get_five_number(&num);
  // 5要素をmatrix（＝行列のN行目を指す）へコピー
  std::copy(result, result + 5, matrix);
}

// matrixはSTACK型の2次元配列(?)
matrix[N][5]
for(int i = 0;i < N;i++){
  // 変更点: std::ref(matrix[i]) -> matrix[i]
  //   受け取り側(thread_access)はポインタ型(STACK*)で受けるため、
  //   呼び出し側でのstd::ref()は必要ありません。
  threads[i] = std::thread(thread_access, std::ref(num[i]), matrix[i]);
}

Answer 2

thread_access(int& num,STACK* matrix)
{
matrix = cc.get_five_number(&num);
}

これだと、引数で渡されたmatrixのポインタの値を書き換えているだけなので、matrix配列の中身は変化しません。

cc.get_five_numberメソッドがどのような実装なのか判りませんが、5個の値を格納した配列の先頭ポインタを返すのだと仮定すると、以下のようなコードになります。

STACK* five_number = cc.get_five_number(&num);
for(int i = 0; i < 5; i++)
	matrix[i] = five_number[i];

Comments

[退会済みユーザー]

回答ありがとうございます．
各スレッドが1行ずつの値を担当してメインスレッドでマージしてN行5列の行列を完成させる感じの実装が必要なのでこれを関数内に書けば大丈夫なのでしょうか？？

[catsforepaw]

大丈夫かどうかと問われると、他のソースの中身を知らないのでなんともお答えしにくいのですが、少なくとも行列に何らかの値が設定されると思います。