トランプの山札から一枚ランダムに取り出す際のエラー

Question

前提・実現したいこと

Javaを参考書で勉強して、ブラックジャックを初めての成果物として作成しているのですが、トランプの山札から一枚ランダムに引くというものを書いたのですが、山札の要素数が０以下になってしまうというエラーが発生しました。調べても見当がつかず、質問させて頂きます。初めてなため、支離滅裂なコードを書いているかもしれませんが、一からやり直さなくてもいい方法があれば教えていただきたいです。非常に申し訳ないのですが、入門書を読み終わった段階でわかるように説明いただけると幸いです。

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

Exception in thread "main" java.lang.IllegalArgumentException: bound must be positive
    at java.base/java.util.Random.nextInt(Random.java:388)
	at ブラックジャック/ブラックジャック.Decks.hit(Decks.java:19)
	at ブラックジャック/ブラックジャック.Player.firstHand(Player.java:15)
	at ブラックジャック/ブラックジャック.Main.main(Main.java:8)

該当のソースコード

package ブラックジャック;


public class Tranp {
	static int[] nums={1,2,3,4,5,6,7,8,9,10,11,12,13};
	static String[] patterns = {"ハート","スペード","ダイヤ","クローバー"};
	
	
	public int point(int num) { 
		if(num == 11 || num ==12 || num==13) {
			num = 10;
			
		}	
		
		return num;
	}
}
-----------------------------------------------------------------------------------------
package ブラックジャック;

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

public class Decks {
static List<String> decks = new ArrayList<>();

	Decks() {
		for(String pattern : Tranp.patterns) {
			for(int num : Tranp.nums) {
				decks.add(pattern + "の," + num );
			}
		}
	}
	public static String hit() {
		Random r = new Random();
		String a = **String.valueOf(decks.get(r.nextInt(decks.size())));**　
		decks.remove(decks.indexOf(a));
		return  a;
	}
}
-----------------------------------------------------------------------------------------
package ブラックジャック;

import java.util.ArrayList;
import java.util.Scanner;

public class Player {
	static Tranp tranp = new Tranp();
	
	static ArrayList<String> player = new ArrayList<>();
	static ArrayList<String> dealer = new ArrayList<>();
	
	static int i = 2;
	
	public static void firstHand() {
		player.add(Decks.hit());
		player.add(Decks.hit());
		dealer.add(Decks.hit());
		dealer.add(Decks.hit());
		
		
		System.out.println("あなたが引いたのは" + player.get(0) + "と" + player.get(1) +"です。");
		System.out.println("ディーラーが引いたのは" + dealer.get(0) + "でもう一枚は裏です。");
		
		System.out.println("現在の得点は" + sumPoint(player) + "です。");
	}
	
	public static void hitOrStand() {
		Scanner sc=new Scanner(System.in);
		System.out.print("Hit or Stand(Hitの場合はY、Standの場合はNを入力してください)：");
		String str = sc.nextLine();
		if(str == "Y") {
			player.add(Decks.hit());
			System.out.println("あなたが引いたのは" + player.get(i) + "です");
			i++;
			System.out.println("現在の得点は" + sumPoint(player) + "です。");
			hitOrStand();
		}else if(str == "N") {
			System.out.println("ディーラーの2枚目は" + dealer.get(1) + "でした。");
			int j = 2;
			while(Player.sumPoint(Player.dealer) < 17) {
				Player.dealer.add(Decks.hit());
				System.out.println("ディーラーは" + dealer.get(j) + "を引きました。");
				System.out.println("ディーラーの現在の得点は" + sumPoint(dealer) + "です。");
				j++;
			}
		}else {
			hitOrStand();
		}
	}
	
	public static int sumPoint(ArrayList<String> list) {
		int sum = 0;
		
		for(int i = 0 ; i < list.size() ; i++) {
			String[] point = list.get(i).split(",",0);
			switch(tranp.point(Integer.parseInt(point[1]))) {
				case 11 : sum += 10; break;
				case 12 : sum += 10; break;
				case 13 : sum += 10; break;
				default : sum += tranp.point(Integer.parseInt(point[1])); break;
			}
		}
		
		return sum;
	}
	
}
------------------------------------------------------------------------------------------
package ブラックジャック;

public class Main {
	public static void main(String[] args) { 
		System.out.println("☆ようこそブラックジャックへ☆");
		System.out.println("ゲームを開始します");
		
		Player.firstHand();
		
		Player.hitOrStand();
		
		System.out.println("プレイヤーの得点は" + Player.sumPoint(Player.player) + "です。");
		System.out.println("ディーラーの得点は" + Player.sumPoint(Player.dealer) + "です。");
		
		int pPoint = Player.sumPoint(Player.player);
		int dPoint = Player.sumPoint(Player.dealer);
		
		if(pPoint > 21) {
			System.out.println("プレイヤーはバーストしました。");
			pPoint = 0;
		}
		if(dPoint > 21) {
			System.out.println("ディーラーはバーストしました。");
			dPoint = 0;
		}
		
		if(pPoint > dPoint) {
			System.out.println("プレイヤーの勝ちです。");
		}else if(pPoint < dPoint) {
			System.out.println("ディーラーの勝ちです。");
		}else {
			System.out.println("引き分けです。");
		}
		
		System.out.println("ブラックジャック終了!!");
	}
	
}

試したこと

トランプの山札の要素数は52なので、r.nextInt(52)としてみたりしましたが無理でした。

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

Eclipseを使用

Answer 1

PlayerのfirstHandメソッドの最初に
Decks decks = new Decks();
を追加することで、動きました。
また、hitOrStandメソッドのif文で、Stringを比較しているところがありますが
str == "Y"　ではなく
str.equals("Y")　です。

どういう動作が正しいのかわからないですが、とりあえずこの変更で動きました。

エラーの原因ですが、Decksのコンストラクタが動いていないためにdecksの配列が空のままでdecks.size()を行った結果、Random.nextIntが意図しない正の数以外を受け取ったために発生したものと思われます。

Comments

[dodox86]

※解決したのであれば、本質問は閉じましょう。
> エラーの原因ですが、Decksのコンストラクタが動いていないためにdecksの配列が空のままでdecks.size()を行った結果、Random.nextIntが意図しない正の数以外を受け取ったために発生したものと思われます。

ほぼそのとおりなのですが、もう少し正確に書くと、Decksクラスのhitメソッドはstaticなクラスメソッドなので、newでインスタンス化していなくても使えます。インスタンス化しないのでDesksコンストラクタも動きません。したがってdecks.size()が0になり、r.nextInt(0)を実行することになり、当該エラーになっています。

[norapomu]

補足していただきありがとうございます。
ぼんやりしていた説明が、はっきりした説明となって、自分も勉強になります。

[poko20001]

解決いたしました。誠にありがとうございます。

なるほど、インスタンス化されていなければコンストラクタが動かないのは当たり前ですね。お恥ずかしい限りです。

非常にわかりやすい解説をお二方とも誠にありがとうございました。teratailを初めて使ったため、申し訳ないのですが、質問を閉じる意図を汲み取れなかったのですが、問題があるようでしたら消します。

改めて、ご回答誠にありがとうございました。

[dodox86]

> 質問を閉じる意図を汲み取れなかったのですが、

ごめんなさい、これは私のコメントが悪かったですね。コメントした当時なぜか、norapomuさんの回答イコール質問者のpoko20001さんの自己回答の投稿だと思い込んでしまっていました。norapomuさんへのベストアンサーで質問を閉じる、の意味で合っています。混乱させて申し訳ありません。