javaの例外処理のプログラミングで困っています。どのようにしたら改善されますか？

Question

実行結果
1番目のデータ:[3,4,5] [true]
2番目のデータ:０以下の数値は判定不可能
3番目のデータ:[2,2,3] [false]
4番目のデータ:[5,3,4] [true]

このように表示させたいです。

public class Question01 {

	public static void main(String[] args) throws PythagorasException {
		// TODO 自動生成されたメソッド・スタブ
		int [][] nums= {{3,4,5},{2,-2,1},{2,2,3},{5,3,4}};
		int count=1;


		for(int i=0; i<nums.length; i++) {
			System.out.print(count+"番目のデータ：");
			for(int j=0; j<1; j++) {
				Pythagoras pythagoras= new Pythagoras(nums[i][0], nums[i][1], nums[i][2]);
				System.out.print("["+nums[i][0]+","+ nums[i][1]+","+nums[i][2]+"]");
				System.out.println(pythagoras);
				count++;

			}
		}

		try{
			// 試験的に例外を投げて...
			throw new PythagorasException("0以下の数値は判定不可能");

		}catch(Exception e){
			e.printStackTrace();
		}

	}

	public static  Pythagoras getPythagoras(int num1, int num2, int num3) {


		return null ;

	}

}

package question01;

public class Pythagoras {
	//フィールド変数
	int lineA;
	int lineB;
	int lineC;

	public Pythagoras(int lineA, int lineB, int lineC) {
		this.lineA=lineA;
		this.lineB=lineB;
		this.lineC=lineC;


	}


	public boolean isRightTrigangle() {
		lineA*=lineA;
		lineB*=lineB;
		lineC*=lineC;

		if(lineA+lineB==lineC || lineB+lineC ==lineA || lineC+lineA==lineB) {
			return true;
		}else {
			return false;

		}


	}

	public String toString() {

		return "["+isRightTrigangle()+"]";
	}


}

package question01;

public class PythagorasException extends Exception{

	//コンストラクタ　指定された詳細メッセージを持つ、新規例外を構築します。
	PythagorasException(String message){
		super(message);
	}

}

現在の出力結果
1番目のデータ：[3,4,5][true]
2番目のデータ：[2,-2,1][false]
3番目のデータ：[2,2,3][false]
4番目のデータ：[5,3,4][true]
question01.PythagorasException: 0以下の数値は判定不可能
at question01.Question01.main(Question01.java:26)

このように、表示されてしまいます。また、getPythagoras()メソッドはなぜ必要なのでしょうか？

Comments

[gentaro]

https://teratail.com/help/avoid-asking
「何かを作りたいのでコードを書いてほしい、学校の課題を解いてほしい等の質問は、具体的にプログラミングで困っている質問ではないと考え、推奨していません。」

> このように、表示されてしまいます。また、getPythagoras()メソッドはなぜ必要なのでしょうか？

問題を出した人に聞きましょう。

Answer 1

まず、Pythagoras クラスのコンストラクターで引数に0以下の値があったら PythagorasException の new して throw しましょう。

public Pythagoras(int lineA, int lineB, int lineC) throws PythagorasException {
        if (lineA<=0 || lineB<=0 || lineC<=0) {
            throw new PythagorasException("0以下の数値は判定不可能");
        }
        this.lineA=lineA;
        this.lineB=lineB;
        this.lineC=lineC;
    }

今試しに書いてある try ~ catch の処理は削除します。

次に Question01 クラスの for文を次のように変えます。

for(int j=0; j<1; j++
    try {
        Pythagoras pythagoras= new Pythagoras(nums[i][0], nums[i][1], nums[i][2]);
        System.out.print("["+nums[i][0]+","+ nums[i][1]+","+nums[i][2]+"]");
        System.out.println(pythagoras);
        count++;
    } catch (PythagorasException e) {
        System.out.println(e.getMessage());
    }
}

getPythagorasメソッドをあなたがなぜ作ろうとしたのかがよくわかりませんが、もしこれが何らかの課題で与えられたテンプレートか何かであれば、出題者の意図はPythagoras クラスのインスタンス作成のためのメソッドにしたかったのでしょう。ただ、メソッド名がその意図には相応しくないので、そうとも限りません。普通そういうことって課題の説明なりに前提として明記されてありそうな点ですが。

Comments

[退会済みユーザー]

getPythagorasメソッドについては後日聞いてみようと思います。
わかりやすい回答ありがとうございました。安心して眠れます^0^