配列の照合がうまく書けなくて困っています

Question

配列の照合がしていのですがうまくできなくて困っています

List<PlayerC> winners;

List<PlayerC> judgeJanken(PlayerC player1, PlayerC player2, PlayerC player3){
winners = new ArrayList<PlayerC>();
Te te1 = player1.showHand();
Te te2 = player2.showHand();
Te te3 = player3.showHand();

	Te[][] winPattern = {
		 { new Stone(), new Scissors() },
		 { new Scissors(), new Paper() },
		 { new Paper(), new Stone() }
	};
	Te[][] losePattern = {
		 { new Stone(), new Paper() },
		 { new Scissors(), new Stone() },
		 { new Paper(), new Scissors() }
	};
	Te[] hands = { te1, te2, te3 };

	for(int i = 0; i < hands.length; i++){
		winFlg = false;
		loseFlg = false;
		for(int j = 0; j < hands.length; j++){
			Te[][] turn = {{ hands[i], hands[j] }};

ここでturnとwinPatternの照合をして合致なら、winFlg = true
次にtruneとlosePatternの照合をして合致なら、loseFlg = true

		}
		if((winFlg == true) && (loseFlg == false)){
			switch(i){
				case 0:
					winners.add(player1);
					break;
				case 1:
					winners.add(player2);
					break;
				case 2:
					winners.add(player3);
					break;
				default:
					break;
			}
		} 
	}
	return winners;	

}

勝者だけをList winnersに追加する

Comments

[kaputaros]

どこがどううまくいかないのか、具体的に記載してください。また、ソースコードは```でくくってください。

Answer 1

他の方のコメントとのやり取りをみて回答しています。
「配列の照合がうまく書けなくて困っています」
の意味合いはturn[0][0]とwinPattern[a][b]の比較ということですね？

まず、次のコードを理解できるでしょうか

Te[][] winPattern = { 
             { new Stone(), new Scissors() }, 
             { new Scissors(), new Paper() }, 
             { new Paper(), new Stone() } 
};
Te a=winPattern[0][0];

配列の各要素はTeそのものになります。
つまりturn[0][0]もwinPattern[a][b]もそれぞれTeのインスタンスとなります。

turn[0][0].equals(winPattern[a][b])

で比較できます。
[]が多すぎて理解し難ければ次のように理解してください。

Te poi=turn[0][0];
Te pattern=winPattern[a][b];

poi.equals(pattern)

配列の照合というより、そのままオブジェクト同士の比較でしたね。
質問内容からの回答は以上です。

ただし、今回の場合別の問題が出てうまくいかないでしょうね。
（オブジェクト比較特有の問題が出てきます)

ただのおせっかいですが・・・
オブジェクトで作成することは別に問題ではありません。
現在の質問者さまの課題は「オブジェクトを理解したい」ということではないのですか？
列挙型に移行する前に現在の問題を理解していないと列挙型でも壁にぶち当たります。
まずは現状のオブジェクトで作成を完成させてみては？

見たところ後で列挙型に改造するのは難しくなさそうです。

Comments

[gyro16]

ありがとうございます。
そこの部分は解決しました。
Stone Scissors Paperに equals() と hashCode() もオーバーライドしました。
が勝敗判定が正確にできません。

Answer 2

「合致」を判断する際、各要素を"=="で比較していませんか？
おなじStoneクラスからのインスタンスでも、winPattern[0][0]のStoneインスタンスとwinPattern[2][1]のStoneインスタンスは別のインスタンスです。「Stoneインスタンスである」という別のロジックを組み立てる必要があります。
私が前の質問で列挙型にしたほうがいいといったのは、こういった理由もあるのです。

Comments

[gyro16]

はい、列挙型は理解しました。
配列では
for(int a = 0; a < winPattern.length; a++){
for(int b = 0; winPattern[a].length; b++){
if(turn[0][0] instanceof winPattern[a][b]){
winFlg = true;
}
}
}
こんな書き方なのでしょうか？

[swordone]

instanceofの右側にオブジェクトは持ってこれないはず…
getClass()同士で比較するくらいでしょうか？

[gyro16]

難しいのですね。クラスにこだわり過ぎました。列挙型に取り組んでみたいと思います。
getClass()はリフレクション系の方面ですね。まだそちらは完全にものにできていません。
一度さらっと流して読んだくらいです。勉強中です。

[swordone]

リフレクション関係ないですよ？getClass()はそのオブジェクトの実体のクラスを表すClassオブジェクトを取得するだけです。