Cの 'for'ループの2つの変数

Question

私はforのループで2つの変数を使う必要があるコードを書いています。以下のコードは問題ありませんか？

それは私に予想される結果を与えます。

for (loop_1 = offset,loop_2 = (offset + 2); loop_1 >= (offset - 190),loop_2 <= (190 + offset + 2); loop_1--,loop_2++) 
{ 
    if ( (*(uint8_t*)(in_payload + loop_1) == get_a1_byte(bitslip)) && 
     ((*(uint8_t*)(in_payload + loop_2) == get_a2_byte(bitslip))) 
     ) 
    { 
      a1_count++; 
    } 
} 

は、しかし、私は言うコンパイラの警告を取得しています：

FILE.C：499：73：警告は：コンマ式の左側のオペランドは効果がありません

何これは意味ですか？

Answer 1

問題は、試験条件である：

loop_1 >= (offset - 190),loop_2 <= (190 + offset + 2) 

これは、両方の部分をチェックしません。 （まあ、それはありませんが、2番目の部分の結果だけが使用されます。）

(loop_1 >= (offset - 190)) && (loop_2 <= (190 + offset + 2)) 

へ

変更、それをあなたは両方の条件がチェックされるようにする場合。

Answer 2

マットが正しいですが、あなたはおそらくにあなたのコードを簡素化を検討する必要があります。（あなたは明らかにあまりにも蚊帳の外in_payload + offsetの計算を巻き上げることができますが、オプティマイザはほぼ確実にあなたのためにそれを行うだろう

for (i = 0; i <= 190; i++) 
{ 
    uint8_t *pl1 = (uint8_t *)(in_payload + offset - i); 
    uint8_t *pl2 = (uint8_t *)(in_payload + offset + i + 2); 

    if (*pl1 == get_a1_byte(bitslip) && *pl2 == get_a2_byte(bitslip)) 
    { 
     a1_count++; 
    } 
} 

）。

Answer 3

意味的な問題については、カフェの答えを参照してください。最初に入力を開始する前にあなたの考えをまっすぐにしてみてください。

誤解の1つは、C、初期化と割り当ての2つの異なる概念を混在させていることです。明らかにあなたのコードでは、カンマ付きのものが完全に動作する初期化の行を考えています。したがって、次回同様の問題が発生した場合は、ローカル変数を使用してください。これらはC99の有効な構成要素であり、何とか使用するのに良いことです。

あなたは私たちに、変数の種類を与えるがすごい、私はこのコードを凝視しないとにかかわらず、私は私がやっと何を理解することができますしてみてくださいいかに難しいのですsize_tあなたfor文が

for (size_t loop_1 = offset, loop_2 = (offset + 2); 
    loop_1 >= (offset - 190) && loop_2 <= (190 + offset + 2); 
    loop_1--, loop_2++)