文字列を解析するとゴミが出る

Question

この問題は文字列を解析するのに役立ちます。

私はCフィールドから来たものではありませんので、私がしているばかげたことに気をつけてください。

私はこれを持っている：

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

char DataIn[65] = "WV,000B,404C494748543B."; 

int main() 
{ 
    char *p = DataIn; 

    puts(DataIn); 

    for (int count = 0;count<2 ; ++count) { 
     p = strstr(p, ","); 
     if (!p) 
      break; 
     p++;    
    } 

    char *endPointer = strchr(p, '.'); 
    *endPointer = '\0'; 

    puts(p); 
} 

私はこの例から必要な出力である：404C494748543B（異なる長さを有することができます）。

私は通常そのOK、時には、は一度だけしばらくでp、を印刷するとき、私はこれを取得：

404C494748543B 
X 
@LIGHT; 
�� 
@LIGHT; 

    ��03 

+ゴミいくつかの以前のデータを含んでいます。

データを抽出する方法に問題がありますか？

Answer 1

ほとんどの場合、ほとんどの場合は動作しますが、ごみを取得する場合は未定義の動作です。これは、DataInが無効なメモリを指していることを意味します。つまり、割り当てが解除されているか、動作を終了した関数のスタックに割り当てられています。

この問題を診断して解決する最善の方法は、valgrindなどのメモリプロファイラでコードを実行することです。

注：あなたのループが終了したときに、2つの状況があります。

• countは、2（通常の終了）に達した、または
• pは

（休憩時間に） NULLに設定されています

endPointerを検索するコードは、pがチェックなしでNULLではないと仮定しているため、プログラムがクラッシュする可能性があります。ループがbreakで終了したとき。