C言語で任意のcsvファイルを配列に格納したい

Question

実現したいこと

タイトルにある通り、C言語で任意のcsvファイル(ただしデータは1行)を配列に格納したいです。
csvファイルの例は次のような感じです。

1,2,3,4,5, ...

扱うcsvファイルは全て1行のデータですが、データの数が変わっても対応できるようにしたいです。

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

配列に格納した結果が表示されません。

該当のソースコード

csvファイルの中身は小数を想定しているので、double型配列を用いています。

#include <stdio.h>
#include <stdlib.h>
#define N 256
 
int main(void) {
	FILE *fp; // FILE型構造体
	char fname[] = "brain_waves.csv";
    double f[N] = {};
	int n;
 
	fp = fopen(fname, "r"); // ファイルを開く。失敗するとNULLを返す。
	if(fp == NULL) {
		printf("%s file not open!\n", fname);
		return -1;
	}
 
	while ( ! feof(fp) && n < N) {
        fscanf(fp, "%lf", &(f[n]));
        n++;
		 printf("%lf",f[n]);
	}
 
	fclose(fp); // ファイルを閉じる
 
	return 0;
}

Comments

[hoshi-takanori]

, を読み飛ばしてなかったり、表示する前に n++ してるからでは…。

[setoppu]

ついでにnの初期値はいくつなんでしょう。

[shsh_]

本当に「任意のcsvファイル」を読み込みたいのでしょうか？
それともカンマ区切りされた複数の数値が取得できさえすればよいのでしょうか？
前者と後者とでは難易度が全く異なります。
「実現したいこと」の内容と「該当のソースコード」の内容が乖離していて、どちらを実現したいのかが分かりません。

Answer 1

nを 0 で初期化すること、n はprintf()のあとに増加させること、あとscanf() の使い方を以下のようにすれば動くかと思います。

    // , 付きで試してみて、
    if (fscanf(fp, "%lf,", &f[n]) < 1) {
      // ダメなら , なしでもう一度
      if (fscanf(fp, "%lf", &f[n]) < 1)
        break; // まだダメならループを終了
    }

Answer 2

とりあえず，原始的に(?) １文字ずつ読み込むような処理を考えてみてはどうでしょうか．

ファイルから１文字取ってきて，
カンマじゃなかったらバッファにため込む．
カンマだったら，その直前までにバッファにたまった内容が１個の数値を表すハズだから…
みたいな．

仮に何かしらの意味でより｛かっこいい(?)，効率的(?)，etc｝な実装形態を目指したいのだとしても，まずはある程度動く単純なのを作ってから→それを足掛かりにして改造なりしていけば良いのではないでしょうか．

#define N (256)
#define MAX_WORD_LENGTH (64)

//文字列を double に解釈する作業．
double CvtWordToDouble( const char *Word )
{
	//※
	//この実装では「解釈できない」文字列が来た場合のことを何も考えずに，
	//単に atof を使う．
	//
	double d = atof( Word );
	printf( "[%s] -> %f\n", Word, d );
	return d;
}

//main
int main( int argc, const char *argv[] )
{
	double f[N] = {};
	int nReadValue = 0;
	{
		const char fname[] = "data.txt";
		FILE *fp = fopen(fname, "r");
		if (fp == NULL)
		{
			printf("%s file not open!\n", fname);
			return -1;
		}

		int WordLength = 0;
		char Word[MAX_WORD_LENGTH] = { 0 };  //バッファ
		int c;
		while( (c = fgetc(fp)) != EOF )  //1文字ずつ読む
		{
			if( c == ',' )
			{//カンマが来たなら現在のバッファの中身が１つの数値を表すハズ
				Word[WordLength] = '\0';
				f[nReadValue++] = CvtWordToDouble(Word);
				WordLength = 0;
			}
			else
			{//バッファにためる
				Word[WordLength++] = c;
				if( WordLength >= MAX_WORD_LENGTH )
				{
					printf( "Unexpected Input\n" );
					return -1;
				}
			}
		}

		fclose(fp);

		if( WordLength > 0 )  //行末の要素に対する処理
		{
			Word[WordLength] = '\0';
			f[nReadValue++] = CvtWordToDouble(Word);
		}
	}

	printf( "%d values read.\n", nReadValue );
	return 0;
}

Comments

[fana]

ありゃ，
MAX_WORD_LENGTH　ってのは変な名称だな…　（まぁどうでもいいが）

Answer 3

fscanf(fp, "%lf", &(f[n]));

を
fscanf(fp, "%lf,", &(f[n]));
とすれば、ある程度は進むだろうけど、行末でコケますね
そもそもCSVの取り込みにscanf使うのはだめです。
そこらへんの考え方から変えないとダメかと。