2つのインスタンスから2つの数字を取得して足し算。その値を比較して大きい方を返すロジックが組めません。

Question

前提・実現したいこと

はじめまして。
最近Javaを勉強し始めましたkai0201を申します。
今Javaのある課題に取り組んでいるのですが、なかなかエラーが解決できず、
今回質問させていただきました。

課題内容は以下の通りです。
1.Testmainメソッド内で変数test3を宣言し、hikakuメソッドの戻り値を受け取る。

2.hikakuメソッド内で受け取った2つのインスタンスのaとbの合計値を比較し、大きい方を戻り値として返す。aとbの合計が一緒の場合は、先に受け取った方を返す。

3.Test2のhikakuメソッドに作成した２つのTest1インスタンスを渡す。

4.Testmainのmainメソッド内でTest1のインスタンス2つとTest2のインスタンスを作成する。

以上です。
現在そこそこ時間を割いて考えているのですが、分からずじまいです。。
（もしかしたら、とんちんかんなロジックを書いているかもしれません。。。。）

また、特に私が頭を悩ませているのは、メソッドの戻り値を他のクラスのへ受け渡す際の、
return の書き方に悩んでおります。
複数のreturnがある場合は、配列に格納して返すのが、一般的なのでしょうか？

お手数をおかけしますが、どなたかご教授お願い致します。

該当のソースコード

class Test1 {

	// フィールド
	private int a;
	private int b;

	// コンストラクタ
	Test1(int a,int b) {
		this.a = a;
		this.b = b;
	}

	// メソッド
	public int getA() {
	    return this.a ;
	}

	public int getB() {
	    return this.b ;
	}
}

class Test2 {

	// フィールド変数
	public int[] result1; // 合計値を入れる変数
	public int result2; // 比較した値を入れる変数
	public int test; // 合計値が同じだった場合
	
	// メソッド
	public Test1 hikaku(Test1 a,Test1 b) {
		
	// インスタンス
	TestMain y = new TestMain();
	
	// getA,getBを取得して足し算
	// 取得した値をresult[]に代入
	int[] result1= {(a.getA() + b.getB())};
	
	if (result1[0] == result1[1]) {
		
		// 合計値が同じだった場合
		test = result1[0];
		
	}else {
		// 合計値を比較
	    result2 = Math.max(result1[0], result1[1]);
	}
	// test3へ値を返す
	return y.test3;
	
	}
}

class TestMain {

	// フィールド変数
	public int test3;

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

		// Test1インスタンス
		Test1 s = new Test1(5,6);
		Test1 s2 = new Test1(5,7);

		// Test2インスタンス
		Test2 f = new Test2();

		// Test2.hikakuの呼び出し
		Test1 test3 = f.hikaku(s,s2);

		System.out.println((test3.getA()),(test3.getB()));
	}

}

補足情報（FW/ツールのバージョンなど）

eqlipce を使用しております。

Comments

[maisumakun]

いま書かれたコードを実行して、どのような状況になるのですか？

[kai0201]

ご連絡ありがとうございます。
現在実行いたしますと以下のエラーが表示されます。

Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: Index 1 out of bounds for length 1
at SampleProject/java_kadai_5.Test2.hikaku(Test1.java:42)
at SampleProject/java_kadai_5.TestMain.main(Test1.java:73)

[coco_bauer]

「受け取った2つのインスタンスのaとbの合計値を比較」という部分ですが、質問のコードでは" result1= {(a.getA() + b.getB())};"となっていて、一方のインスタンスのaと、もう一方のインスタンスのbの合計値が計算されています。　これが「2つのインスタンスのaとbの合計値」という事なのですか？
また、a.getA()と b.getB()はどちらも整数(int)ですから、(a.getA() + b.getB())は一つの整数になります。それをresult1に代入して、result1[0]とresult1[1]を比較するというロジックは無理がありませんか？

[kai0201]

coco_bauer さんご連絡ありがとうございます。
" int[] result1= {(a.getA() + b.getB())};"はおっしゃる通り合計値を計算してそれを配列に格納する目的で記述しております。

１．配列を使用して2つのインスタンスの(a+b)合計値を格納。
// 配列に入れていく

２．格納した２つの合計値を比較。
//配列同士で足し算

３．比較して大きい方を戻り値として返す。
// Math.max（）で配列内の大きい方を返す

４．同じだった場合は先に代入したほうを返す
// 配列の値を(==）で比較

この流れでロジックを組んでみたいと考えていて、
ぱっと思いついたのが、この方法でした。
やっぱり無理がありますよね。。

[dodox86]

「クラス型メソッド内のメソッド呼び出しがわかりません。」https://teratail.com/questions/297440
こちらの課題の発展形なのだと思いますが、コードに混乱があるでしょうか。

> 2.hikakuメソッド内で受け取った2つのインスタンスのaとbの合計値を比較し、大きい方を戻り値として返す。aとbの合計が一緒の場合は、先に受け取った方を返す。

文として、大きい方のTest1インスタンスを返すのか、大きい方のint型の値を返すのか、ちょっと判然としませんね。どちらとも取れます。コードもその混乱が見え隠れしているように思います。
なお、「課題内容」の1.～4. を逆から読んでいくと流れがスムーズに把握できますね。

[kai0201]

dodox86さん
ご連絡ありがとうございます。
先日ありがとうございました。
おっしゃる通り発展形になります。

あ、なるほど！
自分自身これは勘違いしていました。

大きい方のTest1インスタンスを返すのか、
または、
大きい方のint型の値を返すのか、

ここがおっしゃる通り混在しておりました。
ちょっと意識してもう一度課題に取り組んでみます！

[dodox86]

@質問者 kai0201さん
課題文で、
> aとbの合計が一緒の場合は、先に受け取った方を返す。
ここだけ読むとTest1インスタンスを返すようにも思います。先のご質問の延長と考えると、Test1インスタンスでしょうかね。ご確認ください。