色を判断するメソッドをboolean型で作りましたがコンパイルエラーされてしまいます。

車のボディーのカラーが正式なカラーバリエーション（白、黒、赤）に属さない色を渡された場合は、〇〇（色名）は正式なカラーバリエーションに属しません。BLACK/WHITE/REDから選択してください。と表示させたいです。
上記を実現させるために以下のようにコーディングしました。

Java
1//paintメソッド（引数あり、戻り値なし（ここが呼び出し元なので）、色をぬる、正式なカラーバリエーションの範囲内かをチェックするメソッドを呼び出す）
2	public void paint( String cl ){ 
3		if( colorCheck(cl) = true ) {
4			this.carColor = cl ;
5		}else if( colorCheck(cl) = false ){
6			System.out.println( cl + "は正式なカラーバリエーションに属しません。BLACK/WHITE/REDから選択してください。" ) ;
7		}
8	}
9	
10	//colorCheckメソッド（外部からいじれない、引数あり、戻り値boolean、カラーはWHITE/BLACK/RED)
11	private boolean colorCheck( String cl ) {
12		boolean colorResult = false ;
13		
14		if(cl.equals("WHITE")||cl.equals("BLACK")||cl.equals("RED")){
15			colorResult = true ;
16		}
17		
18		return colorResult ;
19	}

これをコンパイルすると以下のようなエラーが出てきました。

【エラー内容】－－－－－－－－－－－－－－－－－－－－－－－－－－－
.\Car_function2.java:59: error: unexpected type
if( colorCheck(cl) = true ) {
^
required: variable
found: value
.\Car_function2.java:61: error: unexpected type
}else if( colorCheck(cl) = false ){
^
required: variable
found: value
2 errors
ーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーー

おそらく、if(colorCheck(cl) = true){...と書くのではなく、if(colorCheck(cl) ){...と書けということだと思うのですが、この部分が分かりません。

colorCheckメソッドでtrueだと判断されればtrueが戻り値としてpaintメソッドのcolorCheck(cl)に代入されるはずと考えました。

すなわち、もしtrueだった場合、if(colorCheck(cl) ){...と書いていれば戻り値のtrueを受け取って　if( true ){...　と変換されるのではないかなと思いました。

しかしそうすると、何がtrueなのかわからないし、falseだった場合の場合分けもできなくなると考えました。

どのようにコードを書けば場合分けがちゃんとできるでしょうか。きちんと理解もしたいので、コードの理由も一緒に教えていただきたいです。よろしくおねがいします。

行動規範の内容に同意します

回答3件

=が代入なのはすでに出てるので。

すなわち、もしtrueだった場合、if(colorCheck(cl) ){...と書いていれば戻り値のtrueを受け取って　if( true ){...　と変換されるのではないかなと思いました。

しかしそうすると、何がtrueなのかわからないし、falseだった場合の場合分けもできなくなると考えました。

前半については質問者様の思っている通りです。
後半（しかしそうすると…）については、メソッド名で「何がtrueなのか」分かるようにするのが一般的かなと。

java
1    if(isHoge("hoge")) {
2      System.out.println("hogeです");
3    } else {
4      System.out.println("hogeじゃないです");
5    }
6
7
8  private boolean isHoge(String str) {
9    return str.equals("hoge");
10  }

（falseだった場合の場合分けは、上記の例のように　if{}else{}で可能です。）
今回の質問者様のコードのように、引数が妥当な値かを判別するメソッドであれば
is〇〇等で良いのではないかなと。
isValidColor()とか。

投稿2020/03/10 05:14

tetutetu

総合スコア419

chimo

2020/03/10 05:41

なるほど！そういう方法があったのですね！勉強になりました、ありがとうございます！！

行動規範の内容に同意します

ベストアンサー

=は代入なので、colorCheck(cl) = true では「関数の結果に代入する」となってしまって意味が通りません。

そして、ifのカッコ内にはbooleanの式を置くので、わざわざtrueと比較しなくてもif( colorCheck(cl) )だけで大丈夫です。

投稿2020/03/10 05:04

maisumakun

総合スコア145184

maisumakun

2020/03/10 05:05

> しかしそうすると、何がtrueなのかわからないし、falseだった場合の場合分けもできなくなると考えました。 elseとだけ書けばfalseの場合の振り分けができます。else ifは冗長なだけです。

chimo

2020/03/10 05:39

回答ありがとうございます。そうでした、=は代入でした。では、==と書けばいけますか？ifの（）内にはtrueかfalseかしか返らないと思うのでやっぱり if colorCheck( cl )だけしかifの()内に書かないのは理解が難しいです。

maisumakun

2020/03/10 05:44

> ifの（）内にはtrueかfalseかしか返らないと思うのでやっぱり if colorCheck( cl )だけしかifの()内に書かないのは理解が難しいです。すみません、何がわからないのか理解ができません。「colorCheck( cl )の返り値がbooleanであること」も「ifのカッコ内にはbooleanの値を書く」こともわかっているのに、この2つを組み合わせられない、という判断の根拠がわかりません。

maisumakun

2020/03/10 05:45

「colorCheck( cl )の返り値」は「trueかfalseのどちらか」です。

maisumakun

2020/03/10 05:50

書きたければ「if(colorCheck( cl ) == true)」と書いてもいいのですが、全く無意味です。

chimo

2020/03/10 07:08

わたしにもなぜ自分が理解できないのかわかりませんがひとまずそういうもんなんだと受け入れて慣れることにします。

行動規範の内容に同意します

=だと代入です。

投稿2020/03/10 05:03