構造体の配列を扱う際のセグメンテーションエラー

Question

構造体を作成し、そのID番号、その値、およびステータスを持っています。 私は、データ（1 199 0 2 199 1 ...）1から成るファイルを持っています。数字は199、値は0、ステータスは0です。このように進んでいます... 私は1つの関数を呼び出しました一度に3つの数値を読み込むためのfilldata（）は、たとえば1 199 0であり、構造体配列の渡された要素に入れます。 そして、私はこの関数をstruct配列を埋めるために呼び出す別の関数を使用しました。 fillAll関数は、ファイルから構造体配列にコピーされたデータセットを返します。 しかし、セグメント化エラーが発生しました。どんな考え？あなた繰り返し開いたファイル名filelocation

int filldata(struct Data_point *a, const char *filelocation) 
    { 

     FILE *f; 
     if((f=fopen(filelocation,"r"))==NULL) 
      printf("You cannot open"); 

     if(fscanf(f, "%ld%lf%d", &(a->sampleNumber), &(a->value), &(a->status)) == 3) 
      return 1; 
     else 
      return 0; 
    } 

    int fillAll(struct Data_point *a, const char *filelocation)// I will pass the struct array and the location of my file string 
    { 
     int index=0; 
     while(filldata(&a[index], filelocation)) 
      index++; 

     return index; 
    } 

Answer 1

が、ファイルハンドルfを閉じることはありません： コードは、より良い説明します。最初の行を何度も何度も読み続けると、最終的にファイルハンドルがなくなります。

あなたは私以下のスニペットを確認するファイルポインタを取るためにfilldataを変更することができますが いくつかの追加のチェックを追加した、あなたもsize of Data_point *aをチェックする必要がありますが

int filldata(struct Data_point *a, File *f) 


    if(fscanf(f, "%ld%lf%d", &(a->sampleNumber), &(a->value), &(a->status)) == 3) 
     return 1; 
    else 
     return 0; 
} 

int fillAll(struct Data_point *a, const int data_point_size,const char *filelocation)// I will pass the struct array and the location of my file string 
{ 

    FILE *f; 
    if((f=fopen(filelocation,"r"))==NULL) { 
     printf("You cannot open"); 
     return 0; 
    } 


    int index=0; 
    while(index < data_point_size && filldata(&a[index])) { 
     index++; 
    } 
    fclose(f); 
    return (index != data_point_size); 
} 

Answer 2

をそれを埋めるよう 割り当てられた範囲内にありますwhileループのためにセグメンテーション違反が発生しています。 filldataが0を返すまでは、決して停止しません。その前に、あなたのプログラムは、& [index]を渡すときに、すでに配列の境界を越えているでしょう。また、プログラムが最初にfscanf（）内のバインドされていないメモリにアクセスしようとすると、実行時エラーが発生したり、ガーベジ・バリューを取って、それをaとみなしたりして、filldataが0を返すという保証はないと思います。成功。

私が間違っている場合は私を修正してください。