fscanfでファイルを正しく読み込みたい

Question

前提・実現したいこと

fscafを用いて読み込んだデータをprintfで表示したいのですが、
思ったようにデータが取得できません。

該当のソースコード

fassion.h

#ifndef FASSION_H
#define FASSION_H
//------------------------------------------------

//------------------------------------------------
//  マクロ定義(Macro definition)
//------------------------------------------------
#define D_FASSION_IDSIZE (6)      // IDバイト数
#define D_FASSION_TYPESIZE (5)     // 種類バイト数
#define D_FASSION_COLORSIZE (4)      // 色バイト数

//------------------------------------------------
//  型定義(Type definition)
//------------------------------------------------
// 服
typedef struct{
    char fassion_id[D_FASSION_IDSIZE];  // ID
    char type[D_FASSION_TYPESIZE];      // 種類
    char color[D_FASSION_COLORSIZE];     // 色
    int pattern;                        // 柄(0:なし 1:あり)
}S_FASSION_DATA;

//------------------------------------------------
//  プロトタイプ宣言(Prototype declaration)
//------------------------------------------------
int memorycatch(void);

//------------------------------------------------
#endif

main.c

#include <stdio.h>
#include "fassion.h"

int main(void){
    memorycatch();
}

fassion.c

#include <stdio.h>
#include "fassion.h"
#include <stdlib.h>
#include <string.h>
#define _CRT_SECURE_NO_WARNINGS

//-------------------------------------
//メモリ確保
//-------------------------------------
int memorycatch(){
    
    FILE * fp = NULL; 
    S_FASSION_DATA * pFdata = NULL;

    // メモリ領域をヒープメモリに確保
    pFdata = (S_FASSION_DATA *)malloc(sizeof(S_FASSION_DATA));
    if(pFdata == NULL){
        return -1;
    }

    // fp = fopen("./fasson/fassion.txt", "r");
    fp = fopen("fassion.txt", "r");


    if(fp == NULL){
        return -1;
    }
    while(fscanf(fp, "%s, %s, %s, %d", 
            pFdata->fassion_id, 
            pFdata->type,
            pFdata->color, 
            pFdata->pattern) != EOF){
        printf("ID：%s タイプ：%s 色：%s 柄：%d\n", 
            pFdata->fassion_id, 
            pFdata->type,
            pFdata->color, 
            pFdata->pattern);
    }

    return 0;
}

fassion.txt

P00001, skirt, pink, 0

printfで表示した結果

理想(正)

ID：P00001 タイプ：skirt 色：pink 柄：0

現在(誤)

ID：P00001, タイプ：, 色：kｶ 柄：9836336
ID：skirt, タイプ： 色：kｶ 柄：9836336
ID：pink, タイプ： 色：kｶ 柄：9836336
ID：0 タイプ： 色：kｶ 柄：9836336

試したこと

• テキストの変更
• 構造体の文字列(type,color)のサイズ変更

---

初歩的な質問だとは思いますが、よろしくお願いいたします。

Answer 1

fscanf関数の書式設定が間違っているのが原因です。
"%s, %s..."は、「(カンマではなく)空白文字を発見するまでストリームを読み込みバッファに格納せよ、そして直後のカンマ、0個以上の空白文字を読み飛ばし、その後、空白文字を発見するまでストリームを読み込みバッファに格納さよ、…」という意味に解釈されます。そのため、1つ目の%sで、"P00001,"がバッファに格納されます。その後、カンマを読み飛ばそうとしますが、実際にストリームに残っているのは空白文字ですから、書式文字列と一致しません。そのため、fscanf関数はそこで終了し、読み取ったフィールド数の1を返します。

直し方ですが、"%6[^,]"のように指定すれば、「カンマ以外の文字を最大6文字読み取れ」という意味になり、2つ目以降のフィールドが読めるようになります。
(ただ、バッファの大きさが足りないのでこのままでは動きませんが…)

Comments

[omochiiii.exe]

回答ありがとうございます。返信遅れて申し訳ないです。

丁寧な説明をありがとうございます。
なぜうまく読み取れないのか原因を理解することができました。

ベストアンサーはどちらにもつけたいのですが、
早さおよび指摘数の面からotnさんにさせていただきました。
お二人とも回答ありがとうございました。

Answer 2

fscanfの引数にはアドレスを渡します。

誤：pFdata->pattern) != EOF){
正：&pFdata->pattern) != EOF){

Comments

[omochiiii.exe]

回答ありがとうございます。
以下のように修正しましたが、やはり間違ったまま表示されます。
配列はそもそもアドレスのため、&はつけないでいいと思ったのですが、
あっていますでしょうか。
----------------------------------------------------------------
while(fscanf(fp, "%s, %s, %s, %d",
pFdata->fassion_id,
pFdata->type,
pFdata->color,
&pFdata->pattern) != EOF){
----------------------------------------------------------------

[otn]

> 配列はそもそもアドレスのため、
patternは配列じゃないです。

> やはり間違ったまま表示されます。
全く同じ出力結果ということであれば、修正前のプログラムを動かしたのでは？

[otn]

ああ、書式がそもそも間違ってました。
"%[^,], %[^,], %[^,], %d"
ですね。
%s は空白文字以外の文字列なので、P00001, にマッチします。「カンマ以外の文字列」にすべき。

fscanfの返り値が4であることを確認しましょう。

[omochiiii.exe]

回答ありがとうございます。返信遅れて申し訳ないです。

>patternは配列じゃないです。
pattern以外(id,typeなどの配列)は＆をつけないとだめなのか気になって確認しました。

otnさんの回答+最後に\n↓
fscanf(fp, "%[^,], %[^,], %[^,], %d\n", ～～～)
で正常に読み込むことができました。
ありがとうございました。