C++ファイルを読み込み空白で区切って数値をとり配列に代入した時に各行の値が同じになることの回避法

Question

C++においてファイルから数値を読み取り読み取った値を行列にして表示したいです。

ファイル内には数字が空白で区切られて入っています。
ファイルの各行を行列の各行になるよう代入したいのですが、今のプログラムだと全ての行に同じ配列が入ってしまいます。
ファイルの各行を行列の各行になるようにするには、どのように修正したらよいでしょうか？

分かる方いらっしゃいましたら、ご回答よろしくお願いします。

リンク内容
ちなみにこちらの質問を参考にしました。

c++
コード
#define _USE_MATH_DEFINES
#include<stdio.h>
#include<math.h>
#include <stdlib.h>
#include <string.h>
#define MAX 4096
int main(void){
FILE *fp ;
char buf[MAX]={0};
int **h1; //パリティ検査行列
int *tmp1; //パリティ検査行列に使う
int i,j;

h1=(int **)malloc(1280 * sizeof(int *));
tmp1=(int *)malloc(1280*1920 * sizeof(int ));
for(i=0;i<1280;i++){
  h1[i]=tmp1+1920;
}

if((fp=fopen("LDPC.txt","r"))!=NULL){
    /*ファイルの終わりまで繰り返し読み込む*/
  i=0;
  while(fgets(buf,MAX,fp)!=NULL){
printf("%s,",buf);
h1[i][0]=atoi(strtok(buf," "));
for(j=1;j<1920;j++){
  
  h1[i][j]=atoi(strtok(NULL," "));
}
i++;

  }
}
fclose(fp);

for(i=0;i<1280;i++){
  for(j=0;j<1920;j++){
printf("%d,",h1[i][j]);
  }
  printf("\n");
  }

free(h1);
free(tmp1);
return 0;

}

Answer 1

ソース全部は見てないけど

/* 間違いだと思う */
h1[i]=tmp1+1920;

/* 正解例の１つかもしれない */
h1[i]=tmp1+1920*i;

C言語の範囲内でやるなら、読むデータが全て数値の場合は

/* aはint型 */
fscanf(fp, "%d", &a);

を(1280*1920)回繰り返すような方法でもいいんじゃないかなと思います。
数値の個数が不定とか間違ってる場合も考慮するなら、fscanfの戻り値もチェックする。
ある行は個数が多くて別の行は個数が少ない場合なんかはズレてしまいますが。

Answer 2

#include <iostream>
#include <vector>
#include <string>
#include <sstream>
#include <fstream>

int main(void){
  std::fstream in("LDPC.txt");
  std::string line;
  std::vector<std::vector<int>> h;

  while (std::getline(in, line)) {
    int value;
    std::stringstream ss(line);

    h.push_back(std::vector<int>());

    while (ss >> value) {
      h[h.size()-1].push_back(value);
    }
  }

  for(auto i=0; i<h.size(); i++){
    for(auto j=0;j<h[i].size()-1; j++){
      std::cout << h[i][j] << ",";
    }
    std::cout << h[i][h[i].size()-1] << std::endl;
  }
  return 0;
}