「C言語」コンマ区切りの数字テキストファイルを読み込み配列に格納したのちに計算する

Question

前提・実現したいこと

C言語でファイルからデータを読み込み配列に格納し合計を計算するプログラムを作っています。
データは、浮動小数点であり、コンマ区切りで記載されています。

発生している問題・エラーメッセージ

ファイルは開きはしたがうまく読み込めてません。

該当のソースコード

#include<stdio.h>
#include<stdlib.h>
int main(){
    /*ファイルオープン*/
    char fname[]="input.txt";
    FILE *fp;
    if((fp=fopen(fname,"r")) == NULL){
        printf("%s is not open\n",fname);
        exit(EXIT_FAILURE);
    }
    /*ファイルのデータを格納*/
    int nx,ny;
    int x=0,y=0;
    fscanf(fp,"%d%d",&nx,&ny);
    float data;
    float **matrix_data;
    printf("nx = %d ny = %d\n",nx,ny);
    while((fscanf(fp,"%[^,]%f",&data)) != EOF){
        printf("ok1\n");
        printf("data = %f",data);
        matrix_data[y][x]=data;
        x++;
        if(x==nx){
            x=0;
            y++;
        }
    }
    /*計算*/
    int i,j;
    float xsum,ysum;
    for(i=0;i<ny;i++){
        for(j=0;j<nx;j++){
            xsum += matrix_data[j][i];
        }
    }
    for(i=0;i<ny;i++){
        for(j=0;j<nx;j++){
            ysum += matrix_data[i][j];
        }
    }
    /*表示*/
    printf("x sum data = %f\ny sum data = %f\n",xsum,ysum);
    return EXIT_SUCCESS;
}

困っていること

実行結果
nx = 50 ny = 50
ok1
data = 0.000000
初めのデータすら読み込めません。
while((fscanf(fp,"%[^,]%f",&data)) != EOF)
この記述の仕方が違うのでしょうか？
コンマ区切りなので%[^,]と書いたのですが、ほかに方法などありますか？
データの間に空白があるからでしょうか？

補足情報（FW/ツールのバージョンなど）

windows10
VSCodeです。
//50*50の浮動小数点数が並んでいます。
//１行目がこのデータの行数
//２行目がこのデータの列数

input.txt
50
50
-0.001494466, -0.002262156,・・・・・0.589552, 0.1961814
・　　　　　　　・
・　　　　　　　・　　　　　　　　　　
・　　　　　　　・　　　　　　　　
-0.002359286, -0.003296237,・・・・・0.2205848, 0.1523751
ここにより詳細な情報を記載してください。

Answer 1

データを読み込む領域を確保せず、matrix_data は未初期化のポインタのままです。
fscanf の書式も間違っています。

次のコードは参考になりますか？

#include <stdio.h>   // fopen, fclose, fscanf, printf, putchar
#include <stdlib.h>  // malloc, free

int main(void)
{
	FILE *fp = fopen("input.txt", "r");
	if (!fp) return 1;
	int m, n;
	if (fscanf(fp, "%d%d", &m, &n) != 2) return 2;
	float **a = malloc(sizeof(float *) * m);
	if (!a) return 3;
	a[0] = malloc(sizeof(float) * m * n);
	if (!a[0]) return 4;
	for (int i = 1; i < m; i++) a[i] = a[i-1] + n;
	for (int i = 0; i < m; i++)
		for (int j = 0; j < n; j++)
			if (fscanf(fp, "%f%*c", &a[i][j]) != 1) return 5;
	fclose(fp);
	for (int i = 0; i < m; i++) {
		for (int j = 0; j < n; j++)
			printf("%16.9f", a[i][j]);
		putchar('\n');
	}
	free(a[0]);
	free(a);
}

fscanf の書式の "%*c" は ',' または '\n' を読み飛ばします。

input.txt

2
4
-0.001494466, -0.002262156, 0.589552, 0.1961814
-0.002359286, -0.003296237, 0.2205848, 0.1523751

実行結果

    -0.001494466    -0.002262156     0.589551985     0.196181402
    -0.002359286    -0.003296237     0.220584795     0.152375102

追記
使っている Cコンパイラが可変長配列(variable length array) をサポートしていれば
ポインタも malloc も free も要りません。
float a[m][n]; だけで済みます。
ただし、自動変数なのであまり大きなサイズの領域は確保できません。

Comments

[tori-a]

ありがとうございます。問題は解決しました。

[kazuma-s]

どのようにして解決したのかを書いてください。
malloc/free を使ったのか、可変長配列を使ったのか、
それとも別のやり方なのか。