ごちゃごちゃしてるのでスッキリさせたい

Question

前提・実現したいこと

教科の点数を入力して、合格の場合は各教科ごとに合格、赤点の場合も各教科ごとに不合格と表示させ、合計点と平均点をだすプログラムを書きました。プログラムは正常に動くんですが、中間らへんの怒涛のif else文がすごく気になるので、もっと短縮して書く方法などがあれば教えていただきたいです。まだ始めて一週間なので文法などはここまでしか学んでいません。

該当のソースコード

package rensyuu5;

public class Rensyuu5 {

	public static void main(String[] args) {
		// TODO 自動生成されたメソッド・スタブ

		System.out.println("今からあなたのテストについて聞きます");
		System.out.println("あなたの国語の点数を教えてください");

		 int write1 =new java.util.Scanner(System.in).nextInt();
		System.out.println("次にあなたの数学の点数を教えてください");

		 int write2 =new java.util.Scanner(System.in).nextInt();
		System.out.println("次にあなたの英語の点数を教えてください");

		 int write3 =new java.util.Scanner(System.in).nextInt();
		System.out.println("次にあなたの理科の点数を教えてください");

		 int write4 =new java.util.Scanner(System.in).nextInt();

		System.out.println("最後にあなたの社会の点数を教えてください");
		 int write5 =new java.util.Scanner(System.in).nextInt();

    int writeborder =35;


		     if(write1>writeborder) {
		    	 System.out.println("国語合格");
	     }else{
	    	 System.out.println("国語不合格");
	     }	 if(write2>writeborder) {
	    	 System.out.println("数学合格");
     }else{
    	 System.out.println("数学不合格");
     }   if(write3>writeborder) {
    	 System.out.println("英語合格");
 }else{
	 System.out.println("英語不合格");
 }   if(write4>writeborder) {
	 System.out.println("理科合格");
}else{
 System.out.println("理科不合格");
}             if(write5>writeborder) {
	 System.out.println("社会合格");
}else{
System.out.println("社会不合格");
}

	int sum =0;
	int [] total = {write1,write2,write3,write4,write5};
     for(int i =0;i<total.length;i++) {
    	 sum +=total[i];
     }
	     int avg =sum/total.length;

	     System.out.println("あなたの合計点は"+sum+"点で、"+"平均点は"+avg+"点です。");

	     int border =180;
	     if(sum>border) {
	    	 System.out.println("あなたは合計が180点以上なので合格です。");
	     }else {
	    	 System.out.println("あなたは180点以下なので不合格です。以後追試を行います。");
	     }
	}

}

Answer 1

最初に…

if文がごちゃごちゃしているのは分かりますが、まずインデントを合わせて欲しい。
Win版しか知りませんが、EclipseならCtrl+Shift+F、IntelliJ IDEAならCtrl+Alt+Lで組み込みのフォーマッタで整形してくれます。
スコープを理解し、インデントを合わせるだけでコードの印象が大分変わります。

さて本題。
1教科分のデータ考えてリスト化、それを回せば単純にif文自体が減りそうですね。
swordoneさんの意見も取り入れて書いてみました。
目についた部分を直していったらもとよりコードが増えてしまうというオチが付いてしまいましたが、まあ必要な部分だけつまみ食いしてみれば良いと思います。

動作サンプル

https://paiza.io/projects/Wa6x3TCQgWNc5sJD1Td5ng

正直今はまだ読むのがキツイと思うんですが、配列とかリスト、クラス辺りを覚えた時にもう一度見ると良いかと思います。

import java.util.*;

public class Main {
    public static void main(String[] args) throws Exception {
		new Main().test();
	}

	private void test() {
		// 教科名、入力点数を持つデータリストを作る
		List<ScoreData> datas = new ArrayList<>() {{
			add(new ScoreData("国語", 0));
			add(new ScoreData("数学", 0));
			add(new ScoreData("理科", 0));
			add(new ScoreData("社会", 0));
			add(new ScoreData("英語", 0));
		}};

		System.out.println("今からあなたのテストについて聞きます");

		// try文を使うと、tryを抜ける時に要closeなstream系オブジェクトをクローズしてくれる
		// とはいえ、今回は標準入力だし要らないか…。ちゃんとデバッグしておこうという事で今回はこのままにする
		// @see https://qiita.com/Air_Hold/items/6535b96a06ec7308fda5
		try (Scanner scan = new Scanner(System.in)) {
			// 点数入力
			int datasSize = datas.size();
			for (int i = 0; i < datasSize; i++) {
				ScoreData data = datas.get(i);
				String prefix;
				if (i == 0) {
				    prefix = "";
				} else if (i < datasSize - 1) {
				    prefix = "次に";
				} else {
				    prefix = "最後に";
				}
				String msg = "あなたの" + data.subject + "の点数を教えてください";
				System.out.println(prefix + msg);

				// 教科の点数をデータリスト内に格納している
				data.score = scan.nextInt();
			}
		};

		// 教科別合否判定（合計点も計算しておく）
		int sum = 0;
		for (ScoreData data : datas) {
			sum += data.score;
			System.out.println(data.subject + (data.score > 35 ? "合格" : "不合格"));
		}

		// 合計合否判定
		int avg = sum / datas.size();
		System.out.println("あなたの合計点は" + sum + "点で、" + "平均点は" + avg + "点です。");

		int border = 180;
		if (sum > border) {
			System.out.println("あなたは合計が" + border + "点を超えているので合格です。");
		} else {
			System.out.println("あなたは" + border + "点以下なので不合格です。以後追試を行います。");
		}
	}

	private class ScoreData {
		String subject;
		int score;
		ScoreData(String subject, int score) {
			this.subject = subject;
			this.score = score;
		}
	}
}

Comments

[bafubafu_]

細かいところまで教えて下さりありがとうございます！動作サンプル見ましたが、addやtryなどまだ分からないことが多かったので、早く勉強して使いこなせるよう頑張りたいと思います。

Answer 2

         if(write1>writeborder) {
             System.out.println("国語合格");
         }else{
             System.out.println("国語不合格");
         }

↓

         System.out.println(write1>writeborder ? "国語合格" : "国語不合格");

他も同様に書き換えられる

Comments

[bafubafu_]

凄く見やすくて分かりやすいです！これから使っていきたいと思います。
ありがとうございます