java scannerの警告について

Question

前提・実現したいこと

プログラミングを初めて約一ヶ月です。現在Javaでブラックジャックを作成中です。
BlackjackクラスやMainクラスにあるscannerについてなのですが現在「リーク: 'scan' が閉じられることはありません」という警告が出ています。
繰り返し使うので.closeすることはできません。
この警告は私自身で調べた限りでは警告は出てしまうものなので無視する、気になるようであれば@suppressWarning(resourse)で消す
と学んだのですが
できれば他のきちんとした方法で消してほしいと言われている状態で困っております。
他にどのような解決策があるでしょうか。

scannerが使われているクラスだけ載せています。
ご助力宜しくお願い致します。

該当のソースコード

import java.util.ArrayList;
import java.util.InputMismatchException;
import java.util.List;
import java.util.Scanner;

/* カード枚数は52枚。ジョーカーは含めない。カードの重複が無いように山札を構築する。
プレイヤー、ディーラーの一対一で対戦するものとし、以下の挙動を取る
初期設定として、プレイヤー・ディーラーが交互に1枚ずつ山札からカードを取り手札とする。
プレイヤーからは自分の手札すべてと、ディーラーの1枚めの手札が確認できる。（ディーラーの2枚目移行の手札はわからない）

手札はAが1ポイント、2-10がそれぞれ2-10ポイント、J/Q/Kが10ポイントとして計算される。

プレイヤーは手札を1枚追加するか、しないかを選択できる。
手札を追加した場合、21ポイントを超えるとバーストとなり、ゲームに敗北する。
プレイヤーはバーストするか、好きなタイミングで止めるまで手札にカードを追加できる。
ディーラーは手札の合計ポイントが17以上になるまで山札を引き続ける。
ディーラーの手札が21ポイントを超えた場合、バーストしてプレイヤーの勝利。
ディーラーの手札が18以上21以下になったとき次の段階に移行する。

プレイヤー・ディーラーの手札のポイントを比較して、大きいほうが勝利。

ダブルダウン・スプリット・サレンダーなどの特殊ルールは無し。 */


public class Blackjack {

	/** ゲームの勝敗を記録する */
	private int win = 0;
	private int lose = 0;
	private int draw = 0;

	/** 賭けに使う所持チップ数とベット数 */
	public int tip = 10;
	private int bet = 0;

	/** 山札の進行状況を記録する変数を定義 */
	private int deckCount = 4;

	/** プレイヤーの手札枚数を記録する変数を定義 */
	private int playerHands = 2;
	private int dealerHands = 2;

	/** プレイヤーとディーラーの手札のポイントを定義 */
	private int playerPoint = 0;
	private int dealerPoint = 0;

	/** プレイヤー・ディーラーの手札リストを生成 */
	List<Integer> player = new ArrayList<>();
	List<Integer> dealer = new ArrayList<>();

	/** DeckクラスとCardクラスのインスタンスを作成 */
	Deck deck = new Deck();
	Card card = new Card();

	/**
	 * ゲームを繰り返すにあたって初期化する必要がある変数を初期化するメソッド
	 */
	public void reset() {

		// 手札リストをリセット
		this.player.clear();
		this.dealer.clear();

		// 山札の進行状況を記録する変数を再定義
		this.deckCount = 4;

		// プレイヤーの手札枚数を記録する変数を再定義
		this.playerHands = 2;
		this.dealerHands = 2;

		// プレイヤーとディーラーの手札のポイントを再定義
		this.playerPoint = 0;
		this.dealerPoint = 0;
	}

	/**
	 * ゲームを開始するメソッド
	 */
	public void start() {

		Scanner scan = new Scanner(System.in);

		// 掛け金を設定する為のwhile文
		while (true) {
			try {
				System.out.println("掛け金を設定して下さい。現在所持チップ" + this.tip + "枚");
				this.bet = scan.nextInt();
				if (this.bet > 0 && this.bet <= this.tip) {
					// 入力が正しい場合ブレイク
					break;
				}
				System.out.println("入力を間違えています。1～所持チップ枚数内で入力してください");
			} catch (InputMismatchException e) {
				System.out.println("入力を間違えています。整数で入力してください");
				// 入力バッファ初期化の為、読み捨てる
				scan.next();
			}
		}
		// デッキクラスのメソッドを呼び出し
		this.deck.createDeck();
	}

	/**
	 * プレイヤーとディーラーが山札からカードを引くメソッド
	 */
	public void drawCard() {

		// プレイヤー・ディーラーがカードを2枚引く
		this.player.add(this.deck.getDeck().get(0));
		this.player.add(this.deck.getDeck().get(1));
		this.dealer.add(this.deck.getDeck().get(2));
		this.dealer.add(this.deck.getDeck().get(3));
	}

	/**
	 * 現在ポイントについて記述するメソッド
	 */
	public void saidPoint() {
		// プレイヤー・ディーラーの手札のポイントを表示
		System.out.println("あなたの1枚目のカードは" + this.card.toDescription(this.player.get(0)));

		System.out.println("ディーラーの1枚目のカードは" + this.card.toDescription(this.dealer.get(0)));

		System.out.println("あなたの2枚目のカードは" + this.card.toDescription(this.player.get(1)));

		System.out.println("ディーラーの2枚目のカードは秘密です");

		// プレイヤー・ディーラーのポイントを集計
		this.playerPoint = CardUtil.sumPoint(this.player);
		this.dealerPoint = CardUtil.sumPoint(this.dealer);
		System.out.println("あなたの現在のポイントは" + this.playerPoint + "です");
	}

	/**
	 * お互いに追加でカードを引くか決め、引いた場合バーストチェックを行うメソッド
	 */
	public void battlePhase() {

		/** バーストした場合の結果を表示するための変数を定義 */
		boolean yourLose = false;

		// プレイヤーがカードを引くフェーズ
		while (true) {

			System.out.println("カードを引きますか？ Yes:y  or  No:n");

			// キーボードの入力を受け付けて、変数strに代入する
			Scanner scan = new Scanner(System.in);
			String str = scan.next();

			if ("n".equals(str)) {
				// カードを引かない場合、ブレイク
				break;
			} else if ("y".equals(str)) {
				// 手札の追加とバーストチェック

				// 山札から一枚追加
				this.player.add(this.deck.getDeck().get(this.deckCount));

				// 山札とプレイヤーの手札を一カウント進める
				this.deckCount++;
				this.playerHands++;

				System.out.println("あなたの" + this.playerHands + "枚目のカードは"
						+ this.card.toDescription(this.player.get(this.playerHands - 1)));
				this.playerPoint = CardUtil.sumPoint(this.player);
				System.out.println("");
				System.out.println("現在のポイントは" + this.playerPoint + "です");
				// バースト判定を行う
				if (CardUtil.isBusted(this.playerPoint)) {
					// バーストしていた場合true
					yourLose = true;
				}
			} else {
				System.out.println("入力を間違えています。yかnを入力してください");
			}

			// プレイヤーがバーストした場合ディーラー勝利の結果表示
			if (yourLose) {
				System.out.println("残念、バーストしてしまいました。あなたの負けです。");
				this.lose++;
				this.tip -= this.bet;
				System.out.println("--------------------------------------------------");
				System.out.println("現在" + this.win + "勝" + this.lose + "負" + this.draw + "分け");
				System.out.println("所持チップ枚数 " + this.tip);
				System.out.println("--------------------------------------------------");
				return;
			}
		}

		System.out.println("ディーラーの2枚目のカードは" + this.card.toDescription(this.dealer.get(1)) + "でした");

		// ディーラーが手札を17以上にするまでカードを引くフェーズ
		while (true) {
			if (this.dealerPoint >= 17) {
				// ディーラーの手札が17以上になった場合ブレイク
				break;
			}

			// 山札から手札に１枚加える
			this.dealer.add(this.deck.getDeck().get(this.deckCount));

			// 山札とディーラーの手札を１カウント進める
			this.deckCount++;
			this.dealerHands++;

			// ディーラーのポイント合計を算出
			this.dealerPoint = CardUtil.sumPoint(this.dealer);

			System.out.println("ディーラーの" + this.dealerHands + "枚目のカードは"
					+ this.card.toDescription(this.dealer.get(this.dealerHands - 1)));

			// ディーラーのバーストしていた場合プレイヤー勝利の結果表示
			if (CardUtil.isBusted(this.dealerPoint)) {
				System.out.println("");
				System.out.println("ディーラーのポイントは" + this.dealerPoint);
				System.out.println("ディーラーがバーストしました。あなたの勝ちです。");
				this.win++;
				this.tip += this.bet;
				System.out.println("--------------------------------------------------");
				System.out.println("現在" + this.win + "勝" + this.lose + "負" + this.draw + "分け");
				System.out.println("所持チップ枚数 " + this.tip);
				System.out.println("--------------------------------------------------");
				return;
			}
		}
		this.result();
	}

	/**
	 * お互いにバーストしなかった場合、ポイントを比較し結果を表示するメソッド
	 */
	public void result() {
		// ポイントを比較する
		System.out.println("");
		System.out.println("あなたのポイントは" + this.playerPoint);
		System.out.println("ディーラーのポイントは" + this.dealerPoint);
		// 引き分けの場合
		if (this.playerPoint == this.dealerPoint) {
			this.draw++;
			System.out.println("引き分けです");
			// プレイヤー勝利の場合
		} else if (this.playerPoint > this.dealerPoint) {
			this.win++;
			this.tip += this.bet;
			System.out.println("勝ちました！");
			// プレイヤー敗北の場合
		} else if (this.playerPoint < this.dealerPoint) {
			this.lose++;
			this.tip -= this.bet;
			System.out.println("負けました・・・");
		}
		System.out.println("--------------------------------------------------");
		System.out.println("現在" + this.win + "勝" + this.lose + "負" + this.draw + "分け");
		System.out.println("所持チップ枚数 " + this.tip);
		System.out.println("--------------------------------------------------");

	}

}

Answer 1

繰り返し使うので.closeすることはできません。

使わなくなった時点(ループを抜けた直後とか、return直前)で、closeすればよいだけでは？

Comments

[ganogano]

scannerを利用する場面が二回あるのですが一度目をcloseすると二度目の部分でエラーが起きてしまいます。
どうしても一度目のcloseの警告が消せない状態です。

[退会済みユーザー]

newもcloseも一度だけでできるように設計すればいいだけかと

Answer 2

一度new Scanner(System.in)を作成すると、このScannerのcloseに伴ってSystem.inまでもcloseされてしまいます。

ということで、new Scanner(System.in)を使う場所で行うこと自体が適当ではないと思われます。Scanner自体をstaticに持たせるなど、別な構造が必要と考えます。

Comments

[ganogano]

scannerを別クラスに置いたところうまく警告を消すことができました！
ありがとうございました。