strcmp関数で同じ文字列を比較したのに、違う文字列と判定された。

Question

プログラムの説明

iris.dataというファイルから1行ずつline変数に入れ、その文字列をstrtok関数で分割し、構造体の変数配列に格納するプログラムです。iris.dataの中身は下記の様な感じです。

5.1,3.5,1.4,0.2,Iris-setosa
4.9,3.0,1.4,0.2,Iris-setosa
4.7,3.2,1.3,0.2,Iris-setosa
4.6,3.1,1.5,0.2,Iris-setosa
5.0,3.6,1.4,0.2,Iris-setosa
5.4,3.9,1.7,0.4,Iris-setosa
4.6,3.4,1.4,0.3,Iris-setosa
5.0,3.4,1.5,0.2,Iris-setosa
4.4,2.9,1.4,0.2,Iris-setosa
4.9,3.1,1.5,0.1,Iris-setosa
5.4,3.7,1.5,0.2,Iris-setosa
4.8,3.4,1.6,0.2,Iris-setosa
4.8,3.0,1.4,0.1,Iris-setosa
4.3,3.0,1.1,0.1,Iris-setosa
5.8,4.0,1.2,0.2,Iris-setosa
5.7,4.4,1.5,0.4,Iris-setosa
5.4,3.9,1.3,0.4,Iris-setosa
5.1,3.5,1.4,0.3,Iris-setosa
5.7,3.8,1.7,0.3,Iris-setosa
5.1,3.8,1.5,0.3,Iris-setosa
5.4,3.4,1.7,0.2,Iris-setosa
5.1,3.7,1.5,0.4,Iris-setosa
4.6,3.6,1.0,0.2,Iris-setosa

質問内容

出力結果が

「data[0].class = Iris-setosa
同じ文字列です。」

となると思っていたのですが、

「data[0].class = Iris-setosa
違う文字列です。」

と出力されていました。
何故なのでしょうか？また、解決策を教えていただきたいです。

ソースコード

#include <stdio.h>
#include <stdlib.h>
#include <string.h>


typedef struct IRIS {
    double sepal_length;
    double sepal_width;
    double petal_length;
    double petal_width;
    char class[100];
} iris;




int main() {
    iris data[150+1];
    FILE *fp;
    char fname[] = "iris.data";
    char line[50];
    
    fp = fopen(fname, "r");
    if(fp == NULL) {
        printf("ファイルを開けませんでした。\n");
        return -1;
    }
    
    for(int i = 0; i < 150; i++) {
        char *ptr;
        fgets(line, 50, fp);
        ptr = strtok(line, ",");
        data[i].sepal_length = atof(ptr);
        ptr = strtok(NULL, ",");
        data[i].sepal_width = atof(ptr);
        ptr = strtok(NULL, ",");
        data[i].petal_length = atof(ptr);
        ptr = strtok(NULL, ",");
        data[i].petal_width = atof(ptr);
        ptr = strtok(NULL, ",");
        strcpy(data[i].class, ptr);
    }
    
    printf("data[%d].class = %s\n", 0, data[0].class);
    if(strcmp(data[0].class, "Iris-setosa") == 0) {
        printf("同じ文字列です。\n");
    }else {
        printf("違う文字列です.\n");
    }
    
    
    
    
    
    return 0;
}

Answer 1

fgetsで読み込んだ文字列の末尾には改行コードがくっついてます。
なので "Iris-setosa" とは一致しません。

ptr = strtok(NULL, ",\n");

で文字列を取り込めばよかったかな。

Comments

[Kino104]

ありがとうございました！
無事解決できました！

Answer 2

別解

    for(int i = 0; i < 150; i++) {
        fgets(line, 50, fp);
        sscanf(line,"%lf ,%lf ,%lf ,%lf ,%s",
          &data[i].sepal_length,
          &data[i].sepal_width,
          &data[i].petal_length,
          &data[i].petal_width,
          data[i].class);
    }

質問中のサンプル文字列の最後みたいに空白が紛れ込んでいてもこれならなんとかなります。逆に文字列がデータとして空白を含んでいたりするとちょっと手を打たないとだめだけど。

クラスメイトかな...まぁどうでもいいですが。