Java エラーが表示された。

Question

前提

Javaの初心者です。
トランプでHigh&lowを作成しております。
引いたカードの情報を取得ができません。数字とマークがうまく取得ができず、実行しても必ず「0」になってしまいます。
ゲーム画面の作成まではできております。
数字とマークを取得できるようにするにはどのようにすればよいのでしょうか。
よろしくお願いいたします。

実現したいこと

カードを引いて数字とマークを取得できるようになりたい。

発生している問題・エラーメッセージ

Cannot invoke "Kensyu.Card$Deck.GetCard()" because "deck" is null

カードを引いた情報を取得できない。

該当のソースコード

Main.Java　クラス
package Kensyu1;

import Kensyu1.Card.Deck;


public class Main {
	private static Deck Deck() {
		// TODO 自動生成されたメソッド・スタブ
		return null;
	}
	public static void main(String[] args) {
		// 山札を生成
		Deck deck = Deck();

		// 姉がカードを引く
		Sensyu ane = new Sensyu();
		ane.Draw(deck);

		// 弟がカードを引く
		Sensyu otouto = new Sensyu();
		otouto.Draw(deck);

		// ゲーム画面を生成
		new Gamegamen(ane, otouto);

		return;
	}



}

Sensyu.java　クラス
package Kensyu1;

import Kensyu1.Card.Deck;

public class Sensyu {

	// メンバ変数定義
	// 選手が持っているカード情報
	private Card card_info;

	// 初期化処理
	public Sensyu(){
		card_info = new Card(); // オブジェクトを生成

		// カード情報を初期化
		card_info.no = 0;   // 数字
		card_info.suit = 0; // マーク
	}

	// カードを1枚引く
	public void Draw(Deck deck)
	{
		// 引いたカード情報を取得
		card_info = deck.GetCard();
		return;
	}

	//カードの数字を取得
	public int GetNo(){
		return card_info.no;
	}

	//カードのマークを取得
	public int GetSuit(){
		return card_info.suit;
	}

}

Deck.java クラス

package Kensyu;

import java.util.ArrayList;
import java.util.Arrays;
import java.util.Collections;
import java.util.List;
class Card{

	int suit; // マーク（0：スペード／1：ハート／2：ダイヤ／3：クラブ）
	int no;   // 数字（A＝1／J＝11／Q＝12／K＝13）

	public class Deck {
		//カード枚数
		final int GOUKEI_CARD = 52;

		// メンバ変数定義
		// カードリスト（山札） 0～51の要素を入れ、それぞれマークと数字を割り当てる
		// 0 ～12：スペードA～K ／ 13～25：ハートA～K ／ 26～38：ダイヤA～K ／ 39～51：クラブA～K
		private List<Integer> card_list;

		// 次に取り出すカード番号(1枚引くごとに1加算する)
		private int card_index = 0;

		// 初期化処理
		public Deck()
		{
			// 次に取り出すカード番号を初期化
			card_index = 0; // 1枚目

			// 山札を初期化
			card_list = new ArrayList<>(Arrays.asList(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));

			// 山札をシャッフルする
			Collections.shuffle(card_list);
		}

		// 山札からカードを1枚取り出し、そのカードのマークと数字をリターンする
		public Card GetCard()
		{
			Card card_info = new Card(); // カード情報格納用
			int card_no = 0; // カード番号

			// カードを一枚取り出して、カード番号(0～51のいずれか)を取得
			card_no = (int) card_list.get(card_index); // リストの先頭要素を取り出す

			// 1枚取り出したのでカード番号を1加算
			card_index++;

			// 山札をすべて引いた場合、山札を初期化する
			if( GOUKEI_CARD <= card_index )
			{
				card_index = 0; // 先頭に戻す
				Collections.shuffle(card_list); // 山札をシャッフルする
			}

			card_info.suit = card_no / 13;       // マーク＝カード番号を13で割った商
			card_info.no   = (card_no % 13) + 1; // 数字＝カード番号を13で割った余り＋1

			return card_info;
		}
	}

}

試したこと

カード情報で数字とマークをそれぞれ取得するように記載をしてみましたがうまくいきませんでした。

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

実行環境　JavaSE-16(P)(Java16)

Comments

[TN8001]

これですよね？（率直に言ってクソいです）
[Javaでゲーム開発！サンプルプログラム付きでわかりやすく解説 | 侍エンジニアブログ](https://www.sejuku.net/blog/129948)

変にいじらずにそのままやったらどうですか。

[退会済みユーザー]

一度試しに実施してみました。うまくいかないようでしたので修正して行ってます。

[TN8001]

> 一度試しに実施してみました。うまくいかないようでしたので修正して行ってます。

元々がひどいのも問題ですが、それを勘でいじっても動くようにはなりません。

こちらで試した手順は以下です。

記事通りに4つのファイルに5つのクラスを作りました。

class Deckの

private List card_list;
がエラーでしたので

private List<Integer> card_list;
に修正しました。

画像ファイル4つを作成（32*32位がいい）し、適切な場所に配置しました。

以上で動作は確認できました。

[TN8001]

なんなんだこいつ

[dodox86]

無言で退会。こんなのばっかり。他人のコードを流用してちゃんと修正するのは一定以上のスキルが必要なはずなのですけど、勘でいじって動いて勘違いしたままでOKなんでしょうね。

Answer 1

MainクラスのDeckメソッドでnullを返しているからです。
ここでDeckオブジェクトを返す必要があります。

現状で無理くりやるなら

package Kensyu;

public class Main {
	private static Card.Deck Deck() {
		// TODO 自動生成されたメソッド・スタブ
		return (new Card()).new Deck();
	}
	public static void main(String[] args) {
		// 山札を生成
		Card.Deck deck = Deck();

                //省略

		return;
	}
}

Comments

[退会済みユーザー]

ご回答ありがとうございます。nullの部分を「Deck」に変更して実行してみました。
private static Deck Deck() {
return Deck() ;

実行した結果が
Exception in thread "main" java.lang.StackOverflowErrorが表示されて
at Kensyu.Main.Deck(Main.java:8)がたくさん出てきました。
こちらはどう変えたらよいかわかりますでしょうか。

[RiaFeed]

それはDeckオブジェクトを返しているのではなく、MainクラスのDeckメソッドを延々と呼び出している形(再帰呼び出し)になってスタックオーバーフローしています。
Deckクラスがどういうクラスなのかコードがないのでわかりませんが、newすればいいんじゃないでしょうか。

[退会済みユーザー]

Deckクラスのコードは下記になります。
package Kensyu;

import java.util.ArrayList;
import java.util.Arrays;
import java.util.Collections;
import java.util.List;
class Card{

int suit; // マーク（0：スペード／1：ハート／2：ダイヤ／3：クラブ）
int no; // 数字（A＝1／J＝11／Q＝12／K＝13）

public class Deck {
//カード枚数
final int GOUKEI_CARD = 52;

// メンバ変数定義
// カードリスト（山札） 0～51の要素を入れ、それぞれマークと数字を割り当てる
// 0 ～12：スペードA～K ／ 13～25：ハートA～K ／ 26～38：ダイヤA～K ／ 39～51：クラブA～K
private List<Integer> card_list;

// 次に取り出すカード番号(1枚引くごとに1加算する)
private int card_index = 0;

// 初期化処理
public Deck()
{
// 次に取り出すカード番号を初期化
card_index = 0; // 1枚目

// 山札を初期化
card_list = new ArrayList<>(Arrays.asList(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));

// 山札をシャッフルする
Collections.shuffle(card_list);
}

// 山札からカードを1枚取り出し、そのカードのマークと数字をリターンする
public Card GetCard()
{
Card card_info = new Card(); // カード情報格納用
int card_no = 0; // カード番号

// カードを一枚取り出して、カード番号(0～51のいずれか)を取得
card_no = (int) card_list.get(card_index); // リストの先頭要素を取り出す

// 1枚取り出したのでカード番号を1加算
card_index++;

// 山札をすべて引いた場合、山札を初期化する
if( GOUKEI_CARD <= card_index )
{
card_index = 0; // 先頭に戻す
Collections.shuffle(card_list); // 山札をシャッフルする
}

card_info.suit = card_no / 13; // マーク＝カード番号を13で割った商
card_info.no = (card_no % 13) + 1; // 数字＝カード番号を13で割った余り＋1

return card_info;
}
}

}

[jimbe]

コードは質問を編集して追加してください。

カードの中にデッキがあるのは、現実的に見て変では無いでしょうか。

[退会済みユーザー]

上記のコードを質問の所に記入いたしました。

[退会済みユーザー]

デッキクラスはカードの山札ですのでカードのことを書いておりますが、何か変なところありますでしょうか。
私の認識違いでしたら申し訳ございません。

[RiaFeed]

カードを表すクラスとデッキを表すクラスは別にした上でデッキを表すクラスをnewして下さい。
仮に中に入れる(内部クラスにする)としてもデッキはカードの集まりですから逆にしたほうがいいのですが、
現状の認識では内部クラスを利用すること自体控えたほうがいいと思います。
この使い方ではそもそも内部クラスにする意味がないですし、なんならカードクラスも要らないんじゃないかと。

[退会済みユーザー]

すみません。私の知識不足でこれから新たにクラスを作成するのは難しいかと思っております。
現状のコードで完成まではいけないのでしょうか。
質問の所に現状私の方で表示できるゲーム画面を追記いたしました。
よろしくお願いいたします。

[退会済みユーザー]

数字の取得ができるようになりました。
たまに数字を表示する部分の背景が赤くなったりしますが、こちらは何かの不具合になるのでしょうか。

別の質問になるのですが、数字は表示できたのですが、♠などのアイコンは表示ができておりません。
フォルダにアイコンの写真を格納してそのフォルダのパスをコードに記入をしているのですがうまくぃません。
case 0: // スペード
icon = new ImageIcon("C:\\pleiades\\workspace\\High&Low\\src\\Kensyu1\\img/spade.jpg");
return icon;
上記のように記入をしてるのですが、ダメなのでしょうか。
よろしくお願いいたします。