Java トランプゲーム トランプの山を作って表示させたい

Question

###前提・実現したいこと
私はプログラミングの勉強を始めて1週間に満たないものです。

現在Javaトランプゲームを作成中です。今実現させたいことは以下の通りです
13 x 4 枚のカードを作成しダイヤ1〜13、スペード1〜13、ハート1〜13、クラブ1〜13 の順に並べて山にする。
つまり、山の一番上(先頭)がダイヤのエースになっている。
そして、作成した山をテストで、main()を持つクラスで実行させたいのですが正しい実行のさせ方がわかりません。
そもそも山を作成するコードが間違っているのでしょうか。

今後シャッフルしたり、山を半分に分けたりして、最終的に大小ゲームをする予定です。
未熟者ですがよろしくお願いします

カードを作成するために以下のCardsクラスを作成しました。

public class Cards {
    private int number;
    private String mark;
    public String anotherName;
    private boolean open;

    public Cards(String mark, int number) {
        this.number = number;
        this.mark = mark;
        this.open = true;
        if (this.number == 11) {
            this.anotherName = this.mark + ":" + "Jack";
        } else if (this.number == 12) {
            this.anotherName = this.mark + ":" + "Queen";
        } else if (this.number == 13) {
            this.anotherName = this.mark + ":" + "King";
        } else if (this.number == 1) {
            this.anotherName = this.mark + ":" + "Ace";
        }
    }

    public int getNumber() throws NotCardsOpenException { /*札の数を取得*/
        if (!this.open) {
            throw new NotCardsOpenException("Cards Closed");
        }
        return this.number;
    }

    public String getMark() throws NotCardsOpenException { /*札のマークを取得*/
        if (!this.open) {
            throw new NotCardsOpenException("Cards Closed");
        }
        return this.mark;
    }
以下長いので割愛させていただきます。

以下が山を作るために作成したCardsMountクラスです

import java.util.ArrayList;
import java.util.List;

public class CardsMount {

    private ArrayList<Cards> cards;

    public List<Cards> makeMountCards() {
        cards = new ArrayList<Cards>();
        for (int i = 1; i <= 13; i++) {
            cards.add(new Cards("クラブ", i));
        }
        for (int i = 1; i <= 13; i++) {
            cards.add(new Cards("ハート", i));
        }
        for (int i = 1; i <= 13; i++) {
            cards.add(new Cards("スペード", i));
        }
        for (int i = 1; i <= 13; i++) {
            cards.add(new Cards("ダイヤ", i));
        }
        return cards;
    }
}

###該当のソースコード
訂正 以下のようなコードに書き直しましたが上手く表示されません。

public class TestRun {

    public static void main(String[] args) throws NotCardsOpenException{
       CardsMount cardsmount = new CardsMount();
       for(int i = 0; i < cardsmount.makeMountCards().size(); i++){
           System.out.println(cardsmount.makeMountCards().get(i));
       }
    }
}

Comments

[swordone]

「うまく表示されません」では伝わりません。どう表示される予定が、実際はどう表示されているのですか？それがないと議論できません。

Answer 1

このコードはコンパイルエラーになりますよね？エラーは見ましたか？
mainの中にあるcardsとはいったい何でしょう？

---

###追記回答
おそらく表示されるのがCard@xxxみたいな形だってことですよね？

public class TestRun {

    public static void main(String[] args) throws NotCardsOpenException{
       CardsMount cardsmount = new CardsMount();
       for(int i = 0; i < cardsmount.makeMountCards().size(); i++){
           System.out.println(cardsmount.makeMountCards().get(i));
       }
    }
}

printlnメソッドにオブジェクトが渡された場合、そのオブジェクトのtoString()メソッドを呼び出した結果の文字列を出力する仕組みになっています。
toString()メソッドはデフォルトでは、クラス名@ハッシュコードという形式の文字列となり、オブジェクトの中身とは関係ない内容になります。
なので、取得したインスタンスからカードの情報を取り出して文字列を自分で作るか、CardsクラスでtoString()メソッドをオーバーライドして好きな文字列を出力させることになります。

なお、このコードではカードの山を都合104回も作成することになり、甚だ無駄です。