コンパイルエラーの理由

Question

#include "画像の名前"
int main ( void ){
BMPnew ();
int x, y;
int cr = BMP_width3/10;
int cg = BMP_width5/10;
int cb = BMP_width*7/10;
int cy = BMP_height/2;
for ( x = 0; x < BMP_width; x++ ){
for ( y = 0; y < BMP_height; y++ ){
if ( ( cr-50) <= x ) && ( x <= (cr+50) ) && ( (cr-50) <= y ) && ( y <= (cy+50) ){
BMPpoint ( x, y, 255, 0, 0 );
} else if ( ( cg-50) <= x ) && ( x <= (cg+50) ) && ( (cy-50) <= y ) && ( y <= (cy+50) ){
BMPpoint ( x, y, 0, 255, 0 );
} else if ( ( cb-50) <= x ) && ( x <= (cb+50) ) && ( (cy-50) <= y ) && ( y <= (cy+50) ){
BMPpoint ( x, y, 0, 0, 255 );
}
}
}
BMPsave ();
return 0;
}
上記をコンパイルしたところ、12行目の28の'('が、expected identifierとなってしまいます。
解決方法お願いします。

Comments

[hope_mucci]

ソースコードは<code>ボタンを使ってシンタックスハイライトにしてもらうと回答者はコードが読みやすく、テストしやすくなります。以下ヘルプを読んでソースコードの部分を修正してください。プレビューもちゃんと確認してね。
https://teratail.com/help/question-tips#questionTips3-5

Answer 1

カッコが多すぎて目が滑るので、演算子の優先順位から不要なカッコは外しましょう。
必要なif文のカッコだけにすると、こんな感じです。

if (cr - 50 <= x && x <= cr + 50 && cr - 50 <= y && y <= cy + 50) {
    BMPpoint(x, y, 255, 0, 0);
} else if (cg - 50 <= x && x <= cg + 50 && cy - 50 <= y && y <= cy + 50) {
    BMPpoint(x, y, 0, 255, 0);
} else if (cb - 50 <= x && x <= cb + 50 && cy - 50 <= y && y <= cy + 50) {
    BMPpoint(x, y, 0, 0, 255);
}

Comments

[IYI]

間違いだけでなく、追加情報まで詳しくありがとうございます。
おかげで、見やすいプログラミングができました　。

Answer 2

そのif文のカッコの対応が間違っています。if ( 条件式 ) { ... の形式になっていません。

if ( ( cr-50) <= x ) && ( x <= (cr+50) ) && ( (cr-50) <= y ) && ( y <= (cy+50) ){
   2 1------1      2 <-ここでカッコが切れてる
   |_______________|

もう一度落ちついてカッコをつけなおしてみてください。

なお、後続のelse if文も、2つとも同じようにカッコの対応が間違っています。

Answer 3

C/C++ の if 文は基本的にはこのような形式を持ちます。

if ( 条件式 ) 文

ですから質問者の書いたコードにおいては if に続く ( ( cr-50) <= x ) の範囲のみが条件式と解釈されます。 && 以降を文として解釈を試みてももちろん成立しませんからエラーです。

その下の else if の箇所についても同様です。