スペードとクローバーまたはハートとダイヤの時に「色は合っている」と表示させる条件式

Question

正解がランダムの数字とマークで返ってきた後、自分で入力した数字とマークが合っているか、合っていないかのプログラミングを作ってます。
その際、マークの色が合っているときだけ「色が合っている」とだけ表示させたいのです。
また、色が合っていないときは「不一致」とだけ表示するように作りたいです。
そのように表示させるための条件式の書き方はどのようにして書けばよいのでしょうか？

package tranpu;

public class GentleCardMaster extends CardMaster{
	GentleCardMaster(CardStack stack){
		super.initialize(stack);//引数を初期化
	}

	public boolean judge(Card card) {
		int answerNumber = getAnswerCard().getNumber();//答えの数字
		Suit answerSuit = getAnswerCard().getSuit();//答えのマーク
		boolean number = true;
		boolean suit = true;

		if(answerNumber > card.getNumber()) {//card.getNumber()は自分で入力した数字
			System.out.println("数：もっと大きい");
			number = false;
		}
		else {
			System.out.println("数：もっと小さい");
			number = false;
		}

		if(answerNumber == card.getNumber()) {
			System.out.println("数：一致");
			number = true;
		}

		if(answerSuit.equals(card.getSuit())) {//card.getSuitは自分で入力したマーク
			System.out.println("マーク：一致");
			suit = true;
		}
		else {
			System.out.println("マーク：色は合っている");
			suit = false;
		}

		if(!answerSuit.equals(card.getSuit())) {
			System.out.println("マーク：不一致");
			suit = false;
		}


		if(number && suit) {
			System.out.println("カードは一致しました");
			return true;
		}
		else {
			System.out.println("カードは一致していません");
			return false;
		}
	}

}

Answer 1

色を取得するメソッドをSuitクラスに置けば簡単に比較できるのでは。

else if(answerSuit.getColor().equals(card.getSuit().getColor())) {

色クラスを作っても良いですし、文字列や数値で表現しても良いでしょう。

Comments

[退会済みユーザー]

色を取得するようなメソッドを作るという発想がありませんでした...
文字列同士を比較するものだとばっかり思っていました。
ありがとうございます。

[LouiS0616]

judgeメソッド内で色を判定する方法もありますけれど、色情報をSuitクラスに持たせた方が実装がすっきりすると思います。
スートの色を変えたい場合（あるのかは別として）もjudgeメソッドがそのまま使えますし。

[退会済みユーザー]

ずっとjudgeメソッド内で色を判定するのにはどう実装すれば良いのだろうと悩んでいました。
確かに他のクラスに色情報をもたせた方があまりコードを書かずにスッキリすることができました。