アレイに余分なデータが格納されているのはなぜですか？

Question

私の割り当てのためにC++でコードをプログラムしようとしています。

メインと関数のデータを受け入れる必要がある部分の1つは、メインで送信した配列の最初の部分を取得して関数の配列に入れなければならないことです。

 for (int i = 0; i <= strlen(main) && exit == 0; i++){ 
     if (main[i] != ';' || main[i] != '\0'){ 

     keyword[i] = data[i]; 
     if(main[i] == ';' || main[i] == '\0') 
     exit = 1; 

     } 

この

は、指定された配列内のコードである main = "Hello World;Yes;No;Okay;Good Bye",

だから私は私の配列のHello Worldがkeywordと呼ばれることを保存したいですが起こると私は私が後に余分なデータを参照して、キーワード文字列をprintfの後、問題が何でありますか単語Hello World

ここでは、私はprintfの

Your keyword-----> 'Hello World;? ' Actual keyword---> 'Hello World'

に持っているものです

私のロジック使用上の問題はありますか？

おかげ

Answer 1

i <= strlen(main)i < strlen(main)へのニーズやif内の余分な終端文字チェックの必要がありません。また、必要なデータをコピーした後にkeywordがヌルで終わることを確認してください（\0）。

目的地がコピーされるデータを保持するのに十分な大きさであると仮定して、ちょっと考えてみましょう。

for (int i = 0; i < strlen(main) ; i++) 
    if (main[i] != ';'){ 
     keyword[i] = data[i]; // Copy the characters until `;` isn't found 
    } else { 
     keyword[i] = '\0' ; // If `;` found, null terminate the copied destination. 
     break; 
    } 
} 

Answer 2

余分なデータはおそらくヌル終了文字です。あなたは

i <= strlen(main) 

を使用している 注これは実際にはすべての文字に加えてヌル終了を出力します。

i < strlen(main) 

一般に、あなたが何をするかをよりよく理解するためには、もう少し多くのコードが役立ちます。

if(main[i] != ';' && main[i] != '\0') 

Answer 3

if (main[i] != ';' || main[i] != '\0') 

は常に真である条件、あなたがどうかをテストしたい：

しかしbegemothとして

は、あなたがやりたいことのためにあなたの場合は（）のテストがあるべきと思われると述べました文字は ';'ではない'\ 0'でないので、（ or）ではない &&演算子（ and）でテストを接続する必要があります。

いくつかの注意：

1. strlen関数は、文字列の長さを決定するか、または*main[i]でテストを置き換えるためにループの前に一度、それを呼び出すために優れているO（N）の複雑さを持っています。

2. exit変数は冗長ですので、breakを使用してください。