Ｃ言語におけるＣＳＶファイルの読み込みについて（日付形式の読み込み）

Question

前提

使用しているC言語のバージョンはC17、gccのバージョンは9.2.0です。

質問

C言語のfscanf関数を使って、CSVファイルの内容を読み込む練習をしていましたが、CSVの日付形式のセルの読み込みがうまくいきません。１列目に日付が記載してあり、２列目に数値が記載しております。そして１列目の月を読み取り、月ごとに数値を集計するプログラムを組もうとしています。どのようにすれば読み込めるかご教示お願い致します。

CSVファイル

該当のソースコード

#define _CRT_SECURE_NO_WARNINGS
#include<stdio.h>
#include<time.h>
#include<stdlib.h>
int main(void)
{
    FILE *fp;
    int month;
    int jan = 0;
    int feb = 0;
    int mar = 0;
    int daynum;
    if((fp = fopen("csvdata1.csv","r")) == NULL)
    {
        printf("No File"); exit(0);   
    }
    else
    {   
        do
        {
            fscanf(fp,"%*d/%d/%*d,%d\n",&month,&daynum);
            if(month == 1)
            {
                jan += daynum;
            }
            else if(month == 2)
            {
                feb += daynum;
            }
            else if(month == 3)
            {
                mar += daynum;
            }
        }while(month == 1 || month == 2 || month == 3);
    }
    fclose(fp); 
    printf("January:%d\nFebruary:%d\nMarch:%d\n",jan,feb,mar);
    return 0;
}

プログラムの実行結果

January:0 
February:0
March:0 

試したこと

日付型を読み取れていないと考えて、

"%*d/%d/%*d,%d\n"

の部分を

"%*[^/]/%[^/]/%*[^,],%d\n"

と変更して、文字列として月を読み取ってから atoi 関数で数値に置き換えようとしましたが、うまくいきませんでした。

Comments

[y_waiwai]

うまくいかないとは、どーゆーふーになってうまくいかないんでしょうか
問題が見えません

[maisumakun]

> CSVファイル

テキストとして見たときに、スペースや引用符が入ったりしてはいませんか？

[suugaku_nyumon]

y_waiwaiさん
ありがとうございます。「プログラムの実行結果」の部分の説明が不足しておりました。質問自体は解決しておりますが、質問の補足部分を付け加えると、CSVの最初の列に日付が記載されており、次の列に数値が記載されていますが、日付から月を読み取って、１月であれば２列目の数値をjanにカウントして、２月であれば２列目の数値をfebにカウントして、３月であれば２列目の数値をmarにカウントして計上していきます。計上されると１月の合計、２月の合計、３月の合計が算出されるはずですが、全て０と出てきたため、数値が先ほどの説明通りに読み取れていない問題が起こっているのかなと思いました。

maisumakunさん
ありがとうございます。テキストエディタで開いて確かめてみましたが、特にスペースや引用符はありませんでした。

Answer 1

日付型を読み取れていないと考えて、

それが正しいか、まずは検証してみましょう。fscanfの実行直後に、monthやdaynumに何が入っているかを確認するのが第一歩かと思います。

Comments

[suugaku_nyumon]

CSVの最後の行を読み取った後のmonthとdaynumの値が変わらず、whileの処理から抜け出せていないことが原因の１つでした。fscanfの戻り値がEOFとなった時にbreakとなる処理を入れたところ、プログラムを思った通りに実行することができました。

Answer 2

意外とありそうなのが，csvdata1.csv を「csv UTF-8 (コンマ区切り) (*.csv)」の形式で保存していて，BOMが邪魔をして読み込みに失敗した，というパターンです．

あと，文字列として月を読み込んでatoi()するのであれば，いっそのことfgets()で一行丸ごと読んでからsscanf()でも良いのかなと思いました．

/* 9eoxx4euqch6j9_revised.c */

#define _CRT_SECURE_NO_WARNINGS
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <time.h>

#define N 256

int main(void) {
    FILE *fp;
    char str[N] = "";
    int month = -1;
    int jan = 0;
    int feb = 0;
    int mar = 0;
    int daynum = -1;

    if((fp = fopen("csvdata1.csv","r")) == NULL) {
        printf("No File");
        exit(1);   
    }
    else {   
        while (fgets(str,N,fp) != NULL) {
            printf("%s\n",str);
            sscanf(str,"%*d/%d/%*d,%d\n",&month,&daynum);
            if(month == 1)
            {
                jan += daynum;
            }
            else if(month == 2)
            {
                feb += daynum;
            }
            else if(month == 3)
            {
                mar += daynum;
            }
        }
    }
    fclose(fp); 
    printf("January:%d\nFebruary:%d\nMarch:%d\n",jan,feb,mar);

    return 0;
}

Comments

[suugaku_nyumon]

ご回答ありがとうございます。
仰る通り、「csv UTF-8 (コンマ区切り) (*.csv)」の形式で保存しておりました。
バイト順マーク（BOM）という用語は初めて聞きましたが、それが原因の１つだったのですね。

Answer 3

fscanfの戻り値を判定してください。
fscanf(fp,"%*d/%d/%*d,%d\n",&month,&daynum);
を
if (fscanf(fp,"%*d/%d/%*d,%d\n",&month,&daynum) == EOF) break;
に変えてください。

Comments

[suugaku_nyumon]

ご回答ありがとうございます。
fscanfの戻り値を検証したところ、数値が入っている最後の行を読み込んだ後は常に-1となっておりました。
基本的なミスでした。