Javaのif-else文についてです。

Question

このプログラムは

長方形１の面積が長方形２の面積よりも大きい場合、赤く塗りつぶした二つの長方形を表示
長方形１の面積が長方形２の面積よりも小さかった場合、青く塗りつぶした二つの長方形を表示する

というようなものです。
JavaAppletで実行しています。

下のコードを見てもらった際に、実行した結果は良いのですが

g.fillRect(20, 30, width1, height1);
g.fillRect(120, 30, width2, height2);

という長方形を描く処理が、if文の中に入っており
これをif文の外にだして一つにまとめるべきと言われました。

if文の外に出して、まとめる。というのが全くわかりません。
どうやって一つにまとめたら良いのでしょうか？

import java.applet.Applet;
import java.awt.Color;
import java.awt.Graphics;
public class DrawRect extends Applet{
	  public void paint(Graphics  g){
		
		int width1 = 90;//長方形１の幅
		int height1 = 150;//長方形１の高さ
		int width2 = 50;//長方形２の幅
		int height2 = 85;//長方形２の高さ
		int area1 = (width1 * height1);//長方形１の面積
		int area2 = (width2 *  height2);//長方形２の面積
		
		if(area1 > area2){//長方形１の面積が長方形２の面積より大きい場合
			g.setColor(Color.red);//赤色を設定
			g.fillRect(20, 30, width1, height1);
			g.fillRect(120, 30, width2, height2);
		}
		else if(area1 < area2){//長方形１の面積が長方形２より小さい場合
			g.setColor(Color.blue);//青色を設定
			g.fillRect(20, 30, width1, height1);
			g.fillRect(120, 30, width2, height2);

}
	  }
}

Answer 1

なぜ、という点についても少し触れておきます。
g.fillRect(120, 30, width2, height2);の30を40にするとしたら、このままだと2箇所を修正しなければならなくなります。
処理を分岐させる必要のないところは「if文の外にだして一つにまとめるべき」なのです。

Comments

[mi_0502]

ありがとうございます!納得しました!

Answer 2

if文の外に出すというのは下記のようになります。

        if(area1 > area2){//長方形１の面積が長方形２の面積より大きい場合
            g.setColor(Color.red);//赤色を設定
        } else if(area1 < area2){//長方形１の面積が長方形２より小さい場合
            g.setColor(Color.blue);//青色を設定
        }

        g.fillRect(20, 30, width1, height1);
        g.fillRect(120, 30, width2, height2);