fgets（）とstrtok（）を使ってカンマ区切り文字でファイルを読む

Question

3つのフィールドがカンマで区切られたテキストファイルがあります。 私のテキストファイルの内容の例：12345、真のプログラミングnewbie、BS ME ファイルをプログラムに読み込むために、私は以下のコードを使用しました....私の問題は、コードが動作することがあり、 （エラーメッセージは表示されず、プログラムは終了し、続行されません）。私はまた、テキストファイルが空白（何も書かれていない）それが自動的に閉じて、継続しないことを観察した。あなたの助けが高く評価されるでしょう。ありがとう！

int read(){ 
    FILE *stream = NULL; 
    int ctr; 
    char linebuffer[45]; 
    char delims[]=", "; 
    char *number[3]; 
    char *token = NULL; 
    stream = fopen("student.txt", "rt"); 
    if (stream == NULL) stream = fopen("student.txt", "wt"); 
    else { 
      printf("\nReading the student list directory. Wait a moment please..."); 
      while(!feof(stream)){    
       ctr=0; 
       fgets(linebuffer, 46, stream); 
       token = strtok(linebuffer, delims); 
       while(token != NULL){ 
        number[ctr] = linebuffer; 
        token = strtok(NULL, delims); 
        ctr++; 
       } 
      recordCtr++;      
      }      
    recordCtr--; 
    } 
    fclose(stream); 
}  

Answer 1

見つけたらtokenをコピーしないでください。 linebufferをコピーすることはできません。そのデータは、次の行がロードされるときに上書きされるためです。

このライン：

number[ctr] = linebuffer; 

は、最近検出されたトークンを保存するためにtokenを参照する必要がありますが、それはしていません。それはおそらくような何かお読みください：

strcpy(number[ctr], token); 

をしかし、あなたはスペースがありますことを確認する宣言を変更する必要があると思います：明らかに

char number[3][32]; 

が、これがあれば、バッファオーバーランのリスクを紹介します非常に長いトークンがあり、適合しません。それを最良に処理する方法は、運動として残されています。 :)

2つの数字と1つの文字列（名前）を格納するために使用されると、なぜ一時的なベクトルが「番号」と呼ばれるのはなぜですか？

Answer 2

あなたのfgets()呼び出しは、サイズとして45を指定する必要があります。あるいは、fgetsがNULLターミネータを書き込むときにバッファをオーバーフローさせる必要があります。これは、 "delims"文字列を空の文字列に設定します。

また、関数宣言によってintが返されたとしても、値は返されません。

"struct student"の定義が何であるか分かりませんが、strcpy()を使用しているときにバッファがあふれている可能性があります。また、 "recordCtr"を減らします。どうして？あなたが書き込み用にそれを開くことができない場合、なぜあなたは書き込みのためにファイルを開きますか？どうして？それでも失敗した場合は、NULLポインタでfcloseを呼び出します。私はそれが多くを助けることを疑う。

「番号」を初期化していないことに気付きました。最初の行に3つの数字がない場合は、初期化されていないポインタからstrcpy()を取得します。おそらくNULL値を持つので、プログラムはsegfaultになります。

また、サイズが3の配列がありますが、読み込んだ行に3つ以上のカンマ区切りフィールドがある場合は、配列がオーバーフローします。

おそらく他にも多くのエラーがあります。

多くのプログラマーは、戻り値のチェック、変数の初期化などのような優れたコーディングの慣行をするのに気を取ることはできません。彼らはしばしばこのようなコードで終わる。あなたが本当に良いプログラマになりたいなら、これらすべてのことをやってみたり、少なくともあなたが必要とするかどうかについて常に考えてみてください。

このコードには非常に多くの潜在的なバグがあります。行の長さが45文字を超えるとどうなりますか？改行は取り除かないでください。あなたは文字列を数値に変換しません（number [1]は文字列データのように見えますが、なぜ "numbers"という配列に格納するのでしょうか？）、または実際にデータを返すか、取得する。

Answer 3

買い手に注意を促してください。

strtokには、いくつかの悩みがあるかもしれません。

「1,2,3」は3つのトークンを生成します。

"one ,, three"は2つのトークンを生成します。