ループが壊れていないとき

Question

以下のコードが誤っている理由を知ることができません。これは、*が入力されるか、配列のサイズに達するまで文字を受け入れ、キーボードから読み取った文字を出力する単純なコードです。読書部分は上手くいくようです。配列サイズに達する前に*を入力してもOKです。しかし、*入力せずに読み込み部分で配列サイズに達するまで待つと、問題があります。印刷中に読み取られた文字は印刷されますが、その後はいくらかのゴミが印刷されます。デバッガを使用して実行しますが、whileループはインデックスが3以上の場合は中断しません。

int main() 
{ 
    char myStr [3]; 
    unsigned int index=0; 
    printf("Enter Single characters. Enter * to stop\n"); 
    do 
    { 
     scanf(" %c",&myStr[index]); 
     index++; 
    } while ((myStr[index-1]!='*')&&((index)<(sizeof(myStr)/sizeof(myStr[0])))); 

    index=0; 

    while ((myStr[index]!='*')&&(index<(sizeof(myStr)/sizeof(myStr[0])))) 
    { 
     printf("%c",myStr[index]); 
     index++; 
    } 
    printf("\n"); 

    return(0); 
} 

Answer 1

コードはここにprintfループの最後の繰り返しに未定義の動作に動作します

while ((myStr[index]!='*')&&(index<(sizeof(myStr)/sizeof(myStr[0])))) 
    { 
     ... 

実際にはそれがmyStr[3]はへのアクセスに

while ((myStr[3] .... 

をやっているようは範囲外です。

while ((index < (sizeof(myStr)/sizeof(myStr[0]))) && (myStr[index] != '*')) 

Boolean short-circuitingが実行されていないmyStr[3]の世話をします。このDOを修正するには

。

Answer 2

あなたは文字列を操作していますが、魔法使いはnullで終了する必要があります。 fgets（3）を使用して文字列を取得し、次にstrlen（3）を使用して長さを取得する必要があります。 そして、あなたは しばらくして、文字列に移動することができる（STR [I]！= '*' & &私<のstrlen（文字列）） 幸運