Javaにて、2枚のカードを比較する処理を行いたい

Question

現在、numが0の場合gameDate.setCard_one(num);に格納、numが0以外の場合gameDate.setCard_two(num);に格納という流れでソースコードを記載しているのですが、Eclipseのデバッグモードにて、参照画像の赤枠部分を押下した際、0の判定となりgameDate.setCard_one(num);に格納されることを確認、次に青枠部分のどれかを押下した場合も0となっており、gameDate.setCard_one(num);に格納されてしまいます。

以下配列で確認した際、場合だと、numが一致しないため延々とゲームが終わらない状態になってしまいます。

int[][] number = {
        {0,1,2,3,4,5,6,7,8,9,10,11,12},
        {13,14,15,16,17,18,19,20,21,22,23,24,25},
        {26,27,28,29,30,31,32,33,34,35,36,37,38},
        {39,40,41,42,43,44,45,46,47,48,49,50,51}
};

参考画像

現状、いい案が思い浮かばず詰まってしまっている状態のため、
お手数ですが、良き案がございましたらご教示いただけますと幸いです。

Cardclick_Listener.java

package pair_matching;

import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;

public class Cardclick_Listener implements MouseListener{
    private CardLabel card_label;
    private GameDate gameDate;
    
    public Cardclick_Listener(CardLabel card_label, GameDate gameDate) {
        this.card_label = card_label;
        this.gameDate = gameDate;
    }
    
    @Override
    public void mouseClicked(MouseEvent e) {
        boolean open = card_label.isOpen();
        int num = card_label.getNumber();
        int posi = card_label.getPosition();
        int CardOne = gameDate.getCard_one();
        int CardTwo = gameDate.getCard_two();
        
        // カードの表裏チェック
        if(open == true) {
            ; // 裏面にしない
        }else if(open == false){
            card_label.setOpen(true);
            if(posi > 0) {
                // numが0か0以外の判定
                if(num == 0) {
                    // 0の場合：GameDateのCard_oneに格納
                    gameDate.setCard_one(num);
                }else{
                    // 0以外:GameDateのCard_twoに格納
                    gameDate.setCard_two(num);
                }
            }
            
        } 
        
        if(CardOne == CardTwo) {
            ; // 一旦処理終了
        }else if(CardOne != CardTwo){
            // カードを裏面に戻す
            // gameDate.setCard_one , gameDate.setCard_twoをリセット
        }
    }

    @Override
    public void mousePressed(MouseEvent e) {
    }
    @Override
    public void mouseReleased(MouseEvent e) {
    }
    @Override
    public void mouseEntered(MouseEvent e) {
    }
    @Override
    public void mouseExited(MouseEvent e) {
    }

}

GameDate.java

package pair_matching;

public class GameDate {
    private int card_one;
    private int card_two;
    
    public int getCard_one() {
        return card_one;
    }
    public void setCard_one(int card_one) {
        this.card_one = card_one;
    }
    public int getCard_two() {
        return card_two;
    }
    public void setCard_two(int card_two) {
        this.card_two = card_two;
    }

}

CardLabel.java

package pair_matching;

import javax.swing.ImageIcon;
import javax.swing.JLabel;

public class CardLabel extends JLabel{
    private int number;
    private int position;
    private boolean open = false;
    private ImageIcon open_icon;
    private ImageIcon reverse_icon;
    
    // コンストラクタの作成
    public CardLabel(ImageIcon open_icon, ImageIcon revers_icon, int number, int position) {
        this.open_icon = open_icon;
        this.reverse_icon = revers_icon;
        this.number = number;
        this.position = position;
        
        // 裏面のセット
        setIcon(revers_icon);
    }

    public int getNumber() {
        return number;
    }

    public boolean isOpen() {
        return open;
    }

    public void setOpen(boolean open) {
        if(open == true) {
            this.setIcon(open_icon);
        }else {
            this.setIcon(reverse_icon);
        }
        this.open = open;
    }

    public int getPosition() {
        return position;
    }

}

CardList.java

package pair_matching;

~ 中略 ~

public class CardList extends JPanel{
    ArrayList<CardLabel> cards = new ArrayList<>();
    private GameDate gameDate = new GameDate();
    // ラベルをリストにする -> リストの各ラベルにImageIconをセット -> リストをコレクションシャッフル
    boolean flg = false;
    
    final int club = 0;
    final int diamond = 1;
    final int heart = 2;
    final int spade = 3;
    
    /* カード画像 */
    // 裏面
    ImageIcon card_back = new ImageIcon("image/card_back.png");
    // 表面
    ImageIcon[][] card_array = new ImageIcon[4][13];
    String filename;
    
    // カード位置
    int[][] Array2 = {
            {1,2,3,4,5,6,7,8,9,10,11,12,13},
            {14,15,16,17,18,19,20,21,22,23,24,25,26},
            {27,28,29,30,31,32,33,34,35,36,37,38,39},
            {40,41,42,43,44,45,46,47,48,49,50,51,52}
    };
    
    // jの数値
    int[][] number = {
            {0,1,2,3,4,5,6,7,8,9,10,11,12},
            {13,14,15,16,17,18,19,20,21,22,23,24,25},
            {26,27,28,29,30,31,32,33,34,35,36,37,38},
            {39,40,41,42,43,44,45,46,47,48,49,50,51}
    };
    
    public CardList() {
        setPreferredSize(new Dimension(1150, 550));
        
        // Card_image
        for(int i = 0; i < card_array.length; i++) {
            for(int j = 0; j < card_array[i].length; j++) {
                filename = "image/card";
                switch(i) {
                case club:
                    filename = filename + "_club_";
                    break;
                case diamond:
                    filename = filename + "_diamond_";
                    break;
                case heart:
                    filename = filename + "_heart_";
                    break;
                case spade:
                    filename = filename + "_spade_";
                    break;
                }
                // ファイル名に数字を足す -> 拡張子を足す
                filename = String.format(filename + "%02d", j+1);
                filename = filename + ".png";
                card_array[i][j] = new ImageIcon(filename);
                CardLabel tmp_label = new CardLabel(card_array[i][j],card_back,number[i][j],Array2[i][j]);
                // tmp_labelを押下したら、ひっくり返るのはtmp_label
                tmp_label.addMouseListener(new Cardclick_Listener(tmp_label, gameDate));
                add(tmp_label);
                cards.add(tmp_label);
            }
        }
        Collections.shuffle(cards);

    }

}

Comments

[jimbe]

どこかで神経衰弱を作るのが流行っているのでしょうか。

[jimbe]

そもそも GameDate が何の為にあるのか、何を処理しようとしているのかが分かりません。「GameDate」という名前は「ゲーム日付」となりますが、2つのラベルを持つだけで何の処理もありませんし、何なのでしょうか。

[jimbe]

以下のように num を表示させてみましたが、それぞれクリックした位置の値が表示されました。

if (posi > 0) {
System.out.println("num=" + num); //デバッグ用表示
// numが0か0以外の判定

[sosoooooo3]

ご返答ありがとうございます。
「GameDate」は、カードを選択した時の値を格納しているものになります。
1枚目を選択したら、GameDateの「setCard_one」に格納。２枚目を選択したらGameDateの「setCard_two」に格納し、それを比較して同じカードか違うカードかの判断を行いたいのです。

[jimbe]

> 1枚目を選択したら、GameDateの「setCard_one」に格納
しかし、CardLabel.number が 0 かどうかで one に入れるか two に入れるかというコードを(意図して)書かれたのですよね。number は CardLabel オブジェクトの生成個所を見ると全 CardLabel オブジェクトで違う値(0～51)が入るようにされています。一枚目の選択はプレイヤーがするのですから、 number = 0 であると決まっている訳では無いと思うのですが…。
この number というフィールドは何を意味するつもりで作成されたものでしょうか。

[jimbe]

もしお時間などありましたら、別の方の神経衰弱のご質問で回答として動作するコードを載せたことが有りますので、参考にして頂けると幸いです。

https://teratail.com/questions/354099

[sosoooooo3]

ありがとうございます。
参考にさせていただきます。

Answer 1

ゲームを進行管理する変数必要です。

暫定的に、GameDateに設置してみてはいかがでしょう？

GameDate.java

package pair_matching;

public class GameDate {

    private int PlayCount; // ゲーム進行管理用
    private int card_one;
    private int card_two;

    public int getPlayCount() {
        return playCount;
    }
    public void setPlayCount(int playCount) {
        this.playCount = playCount;
    }
    public int getCard_one() {
        return card_one;
    }
    public void setCard_one(int card_one) {
        this.card_one = card_one;
    }
    public int getCard_two() {
        return card_two;
    }
    public void setCard_two(int card_two) {
        this.card_two = card_two;
    }

}

その上で、Cardclick_Listener.javaを修正します。

修正箇所

            if(posi > 0) {
                // numが0か0以外の判定
                if(num == 0) {
                    // 0の場合：GameDateのCard_oneに格納
                    gameDate.setCard_one(num);
                }else{
                    // 0以外:GameDateのCard_twoに格納
                    gameDate.setCard_two(num);
                }
            }

修正後

            if(gameDate.getPlayCount() == 0) {
                // 0の場合：GameDateのCard_oneに格納
                gameDate.setCard_one(num);
            }else{
                // 0以外:GameDateのCard_twoに格納
                gameDate.setCard_two(num);
            }
            gameDate.setPlayCount(gameDate.getPlayCount() + 1);

ちなみに後々の判定のために、numberは一気通貫の番号で無く、カードの番号通りにした方が良いかも。