ｃ言語 fscanf以降が読み込めない

Question

前提・実現したいこと

fscanfをつかってtxtファイルの数値を読み込みいたいです。

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

以下の数値をよみこみたいのですがfopenまではうまくいき、数値の一番左上の0.000000は読み込みひょうじも可能です。ですがそれ以降の2.780320によみこみが進みません。
実行結果は
0.000000
0.000000
と二つの変数とも0.000000のままです。
aaa.txtです。

0.000000,2.780326

0.010000,2.785436
0.020000,2.790545
0.030000,2.790545
0.040000,2.793100
0.050000,2.790545
0.060000,2.785436
0.070000,2.785436
0.080000,2.785436
0.090000,2.800765
0.100000,2.800765

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

struct lvlv{
	int score;
	int time;
};

int main(void){
	
	FILE *fp;
	struct lvlv *lv;
	int i = 0;
	int score;

	lv = (struct lvlv*)malloc(sizeof(struct lvlv));
	
	 if ((fp=fopen("s:\aaa.txt","r"))==NULL){
	 	printf("Cant open file %s\n", "jx18250_0426_100Hz.txt");  /*オープンに失敗したらメッセージ表示*/ 
	 	}
		
	for(i=0;i>0;i++){
		fscanf(fp,"%f %f",lv[i].time,lv[i].score);
	}
	printf("%f\n",lv[i].time);
	printf("%f",lv[i].score);
	
	fclose(fp); 
	return 0;
}

Comments

[Y.H.]

ところで、タグのC#は関係ないので削除していただけませんか？

Answer 1

逆順でごめんね。

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

struct lvlv{
  double score;
  double time;
  struct lvlv* next;
};

int main(){

    FILE *fp;
    struct lvlv *lv = NULL;
    char line[256];
    
    fp = fopen("aaa.txt","r");
    if ( fp == NULL) {
         printf("Cant open file aaa.txt\n");
         return -1;
    }

    while ( fgets(line, 256, fp) ) {
      double s, t;
      sscanf(line, "%lf, %lf", &s, &t);
      struct lvlv* lvp = (struct lvlv*)malloc(sizeof(struct lvlv));
      lvp->score = s;
      lvp->time = t;
      lvp->next = lv;
      lv = lvp;
    }
    fclose(fp); 

    while ( lv != NULL ) {
      printf( "%f  %f\n", lv->score, lv->time );
      struct lvlv* p = lv->next;
      free(lv);
      lv = p;
    }

    return 0;
}

Answer 2

• このforループ１回も回らないのでは？

iの初期値=0、終了条件継続条件 iが0より大きい

    for(i=0;i>0;i++){
        fscanf(fp,"%f %f", lv[i].time,lv[i].score);
    }

• 確保していない範囲のメモリを参照していますよ メモリ壊してますよ

    struct lvlv *lv;
//略
    lv = (struct lvlv*)malloc(sizeof(struct lvlv));
//略
    for(i=0;i>0;i++){
        fscanf(fp,"%f %f",lv[i].time,lv[i].score);
    }

//この時点でi=1なので、lv[i]はmallocで確保していない領域にアクセスしている
    printf("%f\n",lv[i].time);

---

愚直に書くとこんな感じ？
※ゴリゴリのC言語書くの何年振りかわからないので間違ってる可能性あり。

//略
    struct lvlv **lv;
//略
    i=0;
    lv = (struct lvlv**)malloc(sizeof(struct lvlv *));
    (*lv) = (struct lvlv*)malloc(sizeof(struct lvlv));

    while(fscanf(fp,"%f %f", &((*lv)->time), &((*lv)->score)) > 1){
        lv = (struct lvlv**)realloc(lv, sizeof(struct lvlv *) * i);
        *(lv + i) = (struct lvlv*)malloc(sizeof(struct lvlv));
        i++;
    }

    int j=0;
    for(j=0; j<i; j++) {
        printf("%f ", (*(lv + j))->time);
        printf("%f\n",   (*(lv + j))->score);

        free(*(lv + j));
    }
    free(*(lv + j));
    free(lv);

Comments

[cateye]

for(i=0;i>0;i++){←これって一回も回らないｗ
0で初期化して0以上ならって成り立たない・・・・scanf()ばっかり見てたけど・・・いっぱい有るなぁ

[tmtmmm]

ありがとうございます。よみこむことができました。最後の数字までいったらやめるというコードはどうすればいいでしょうか

[cateye]

scanf()が失敗した時にブレークするのかなぁ・・・一応私の回答に追記しておきました。・・・ただ、読み込む回数がわからなければメモリの確保もその都度する必要がありますよ。

[Y.H.]

ぶっちゃけこのプログラムを自力で完成できるのであればC言語中級と認定してもいいくらい色んな要素が含まれてますね。

[cateye]

lv = (struct lvlv**)realloc(lv, sizeof(struct lvlv *) * i);←これ失敗したら前のポインタ消えちゃうから、開放できなくなるのでは？

[Y.H.]

cateyeさん、突っ込みありがとう。やっぱりポカしてたw
そもそも mallocのエラー判定も抜けてましたwww

[cateye]

＾＾；

Answer 3

ダメな原因は、実際のファイルはコンマ区切りなのに、

fscanf(fp,"%f %f",lv[i].time,lv[i].score);

とスペースで区切るようになってます。これではダメです
"%f,%f" とすればこの場ではいけるようになるかもだけど、
fscanfでは、想定外の入力でフリーズしたりするので使わないほうがいいです

fgetsで１行読み込み、そこからコンマで分割して数値を読み込むようにしましょう

---

struct lvlv{
float score;
float time;
};
にしといて、

fscanf(fp,"%f,%f",&lv[i].time,&lv[i].score);

ですね

Comments

[tmtmmm]

ご指摘ありがとうございます。カンマを入れて実行しましたがけっかはかわりませんでした

[y_waiwai]

なら、fscanfはつかわないようにしましょう

Answer 4

カンマ(,)が抜けてるｄ＾＾
fscanf(fp,"%f %f",lv[i].time,lv[i].score);→fscanf(fp,"%f,%f",lv[i].time,lv[i].score)
//

   for(i=0;i>0;i++){
        fscanf(fp,"%f %f",lv[i].time,lv[i].score);
    }
    ↓
   for(i=0;i< ?;i++){
        if(fscanf(fp,"%f,%f",&lv[i].time,&lv[i].score)!=2){
            break;
        }
    }

かな？

Comments

[tmtmmm]

ご指摘ありがとうございます。カンマを入れたのですが治りませんでした。結果はおなじでした

[cateye]

あと、 fscanf(fp,"%f %f",lv[i].time,lv[i].score);　→　 fscanf(fp,"%f %f",&lv[i].time,&lv[i].score); かな？
ポインタを渡さなきゃダメですね・・・気がつくのが遅かった。・・・てかdoubleじゃないのか？・・・int じゃ読めないよ＾＾；

[cateye]

一回ファイルを空読みして行数を数えるとかしないと、for文の脱出条件がfscanf()の失敗でしか判定できないですね。

[cateye]

私なら・・・ですが、構造体を1個用意して正常に読めたらメモリ確保してコピーかな？　確保したメモリはリストかなんかに繋がないと制御できなくなりますよ。free()での開放もいるし・・・

[asm]

> doubleじゃないのか？
doubleだったら今度は%fの罠にはまりますね

[cateye]

ｗ；