C言語で二次元配列のファイル読み込み

Question

C言語初心者です。
C言語でcsvファイルの読み込みを行いたいです。
csvファイルは上部の画像です。
このcsvファイルから各行の最後の数字のみを抜き出し、ファイルに書き出すということをしたいです。
現在その途中なのですが、以下のコードを試すと実行結果がdata[0][0]のみ数値が表示され、data[0][1]やその他の配列は全て０となってしまいます。どなたか教えていただきたいです、、。
また、配列に数値が入った後、どのようにして各行の最後の数値を抜き出せば良いかご教授よろしくお願いします、、。

#include <stdio.h>
#define BUF_SIZE 256
int main(int argc, char *argv[])
{
    FILE *fp;
    char buf[BUF_SIZE];
    int line = 0;
    if ((fp = fopen("Book1.csv", "r")) == NULL) {
        return -1;
    }
    while (fgets(buf, BUF_SIZE, fp) != NULL) {
        line++;
    }

    rewind(fp); // ★ または fseek(fp, 0, SEEK_SET);

    int data[10][8];
    for(int i=0;i<10;i++){
      for(int j=0;j<8;j++){
	fscanf(fp,"%d ",&data[i][j]);
      }
    }
    printf("%d\n",data[0][0]);
    fclose(fp);
    
    return 0;
}

Answer 1

1行目は 2個、6行目は 5個。10行目は 4個ですね。
1行 8個を想定して読み込んではいけないのではありませんか？

fscanf の書式を "%d," に変更するだけではダメでしょう。

追記
fgets と strtol を使った例です。他にもやり方はあります。

#include <stdio.h>   // fopen, fclose, fgets, puts, printf, putchar
#include <stdlib.h>  // strtol

#define BUF_SIZE   256
#define LINE_SIZE   20
#define FIELD_SIZE  16

int main(void)
{
	FILE *fp = fopen("Book1.csv", "r");
    if (fp == NULL) return 1;

    char buf[BUF_SIZE];
    int line;
    for (line = 0; fgets(buf, BUF_SIZE, fp); line++) ;
	if (line > LINE_SIZE) { puts("too many lines"); return 2; }

    rewind(fp);

    int data[LINE_SIZE][FIELD_SIZE], count[LINE_SIZE];
	for (int i = 0; fgets(buf, BUF_SIZE, fp); i++) {
		char *p = buf, *sep;
		int j = 0;
		while (j < FIELD_SIZE) {
			data[i][j] = strtol(p, &sep, 10);
			if (sep == p) break;
			if (++j >= FIELD_SIZE) { puts("too many fields"); return 3; }
			if (*sep != ',') break;
			p = sep + 1;
		}
		count[i] = j;
	}
    fclose(fp);

	for (int i = 0; i < line; i++) {
		for (int j = 0; j < count[i]; j++)
			printf("%10d", data[i][j]);
		putchar('\n');
	}
    return 0;
}

Comments

[iamisa]

各行の数が違うので、どのように考えばいいのかわからず、１行８個を想定してしまっていました。このような場合どのように考えれば良いのでしょうか？、、

[iamisa]

ありがとうございます！！！とても参考になりました！

Answer 2

CSVファイルという事は、コンマ区切りだと思いますが、

fscanf(fp,"%d ",&data[i][j]);

これではだめですね。

C言語のコードを書くなら、デバッグできる環境を整えましょう。
Eclipseや、WindowsならVisualStudioなど。
コードの任意の場所で実行を止め、変数のナカミを見ることができます。そこから１行づつ実行して、コードの流れを見れるようになります
そうすれば、アテズッポでコードを書かなくて済むようになります。

Comments

[iamisa]

確かに、コンマ区切りなのをわすれていました、、。ありがとうございます。

[y_waiwai]

他の方の回答にもありますが、この修正だけでは無理ですね
やるなら、１行づつ読み込んで、行単位で処理する、とかの工夫がいりそうです

[iamisa]

1行ずつしか無理なんですね、、！ありがとうございました！