C言語のfscanf()関数を用いてCSVファイルを読み込む方法

Question

前提・実現したいこと

C言語で<stdio.h>のfscanf()関数を使って、CSVファイルの内容を読み込みたいのですが、意図しない結果となってしまいます。

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

CSVファイルの中身は

"comma.csv"

test,10,11,1.1,1.2

test,20,21,2.1,2.2
test,30,31,3.1,3

と記述したのですが、後述のコードを実行したときにこのような結果が得られました。
CSVファイルと同じ記述を標準出力に表示するにはどのようにすればよいのでしょうか？

実行結果

・ｿtest 10 11 1.1 1.2

test 20 21 2.1 2.2
test 30 31 3.1 3.2
30 31 3.1 3.2
続行するには何かキーを押してください . . .

該当のソースコード

#include <stdio.h>

int main(void)
{
	FILE *fp;
	char *fname = "comma.csv";
	char s[16];
	int ret, n1, n2;
	float f1, f2;

	fp = fopen(fname, "r");
	if (fp == NULL)
	{
		printf("%sファイルが開けません\n", fname);
		return -1;
	}

	while ((ret = fscanf(fp, "%[^,],%d,%d,%f,%f", s, &n1, &n2, &f1, &f2)) != EOF)
	{
		printf("%s %d %d %.1f %.1f\n", s, n1, n2, f1, f2);
	}

	fclose(fp);
	return 0;
}

試したこと

ファイル操作については初心者であるため、コードを書く以外に試したことはありません。

開発環境

Visual Stdio 2015

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

今回はネットに掲載されていたコードを写経していたために、fscanf()関数を用いていますが、もし他の方法をご存じでしたら参考までに教えていただけると幸いです。

Answer 1

3行目を読んだときに、改行文字が読み残されるので、4回目の読み込みでsに改行が入ります。
改行文字までを喰うために、フォーマット文字列の最後に、空白を入れると良いでしょう。
また、scanf系の関数は戻り値が、変数の数と一致しているか確認しましょう。文字列を読むときは長さを入れましょう。

    while ((ret = fscanf(fp, "%15[^,],%d,%d,%f,%f ", s, &n1, &n2, &f1, &f2)) != EOF)
    {
        if(ret!=5) {
            printf("ファイル内容がおかしい ret=%d\n",ret);
            printf("<%s>\n",s);
            return 1;
        }
        printf("%s %d %d %.1f %.1f\n", s, n1, n2, f1, f2);
    }

別案としては、while条件を!=EOFじゃなくて==5にする。

scanf系はいつも難しいです。この問題に関してだけだと、fgets+sscanfだとフォーマット文字列の空白が無くても大丈夫です。

Comments

[Uh37]

ひとまずフォーマット文字列の最後に空白を入れた段階で、csvファイル内のデータを読み込むことには成功したようです、ありがとうございます。"fgets" と "sscanf" を使った方法にも挑戦してみたいと思います。

[otn]

fgets + strtok という方法もあります。

Answer 2

一つ前の行の改行コードを読み飛ばすために
"%[^,],%d,%d,%f,%f" の %[ の前にスペースを入れて
" %[^,],%d,%d,%f,%f" にする。

ファイル入力の場合はスペースを最後においても構いませんが、
キーボード入力の場合に困ります。

#include <stdio.h>

int main(void)
{
    char s[16]; int n;
    while (scanf("%[^,],%d ", s, &n) == 2)
		printf("%s %d\n", s, n);
}

スペースが前の場合の実行例

abc,123  (入力)
abc 123
def,-45  (入力)
def -45
,  (^D または ^Z でもよい)

スペースが最後の場合の実行例

abc,123  (入力)
def,-45  (入力)
abc 123
,  (^D または ^Z でもよい)
def -45

次の行を入力するまで、今の出力が出ません。

Comments

[Uh37]

参考になります。ありがとうございます。

Answer 3

fscanfには残念ながら正規表現を処理する機能はありません

CSVを読むには、fscanfを使うのはやめといたほうがいいと思います
fgetsで１行読み、そこから分解していくようにしましょう

Comments

[LouiS0616]

正規表現一般を処理する能力は確かにありませんが、少なくとも %[^,] は合法です。
https://linuxjm.osdn.jp/html/LDP_man-pages/man3/scanf.3.html

[Uh37]

scanf()については妄信して使っていたので、勉強になります。
ありがとうございます。

[y_waiwai]

scanf系の関数は危険な関数なんで、実務では殆ど使われません