vscodeのjavaにおいてscanner.nextInt() が飛ばされてしまう

Question

実現したいこと

vscode内でなぜかscanner.nextIntの入力が飛ばされてしまうことを解決したい

前提

vscodeで遊びながらjavaの勉強をしていて、オブジェクト指向に慣れるために複数のファイルを使って一つのプロジェクトを作成する練習をしていました。
そこで簡単なRPG的なものを作っていたら入力部分が飛ばされてしまい、うまく実行できませんでした。
下記にエラーメッセージと使っていた3つのファイルを載せておきます。
問題となっている入力の部分が分かりやすいようにコメントで補足してあります。

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

Let's enjoy playing adventure!
Input your name in English.
testKun
At first, testKun should get to a higher level.
Let's go to kill weaker enemies than testKun.
Current level is 1.
Current HP is 10.
Skills that you can use now are hit protect runAway sleep
You are attacked by mushroom monster.
Let's fight!
Exception in thread "main" java.util.NoSuchElementException
        at java.base/java.util.Scanner.throwFor(Scanner.java:945)     
        at java.base/java.util.Scanner.next(Scanner.java:1602)        
        at java.base/java.util.Scanner.nextInt(Scanner.java:2267)     
        at java.base/java.util.Scanner.nextInt(Scanner.java:2221)     
        at Enemy.fight(Enemy.java:10)
        at Quest.main(Quest.java:24)

該当のソースコード

import java.util.Scanner; //mainで使ってるファイル
public class Quest {
    public static void main(String[] args){
        System.out.println("Let's enjoy playing adventure!");
        System.out.println("Input your name in English.");
        Scanner scanner = new Scanner(System.in);
        String name = scanner.nextLine();
        System.out.println("At first, " + name + " should get to a higher level.");
        System.out.println("Let's go to kill weaker enemies than " + name + ".");
        String[] wayAttack = new String[4];
        wayAttack[0] = "hit";
        wayAttack[1] = "protect";
        wayAttack[2] = "runAway";
        wayAttack[3] = "sleep";
        int[] damage = new int[4];
        int hp;
        damage[0] = 5;
        damage[1] = 0;
        damage[2] = 0;
        damage[3] = 0;
        Hero hero = new Hero();
        hp = hero.printStatus(wayAttack);
        Enemy enemy = new Enemy();
        enemy.fight(wayAttack,damage,hp);
        scanner.close();
    }
}

import java.util.Scanner;
public class Hero {
    public int printStatus (String[] wayAttack){
        Scanner scanner = new Scanner(System.in);
        int level = 1,hp = 10,i = 0;
        System.out.println("Current level is " + level + ".");
        System.out.println("Current HP is " + hp + ".");
        System.out.print("Skills that you can use now are ");
        for(i = 0; i < 4; i++){
            if(i!=3) System.out.printf("%s ",wayAttack[i]);
            else System.out.printf("%s\n",wayAttack[i]);
        }
        scanner.close();
        return hp;
    }
}

import java.util.Scanner;

public class Enemy {
    public void fight(String[] wayAttack, int[] damage, int hp) {
        Scanner scanner = new Scanner(System.in);
        int mashroom = 10, n;
        System.out.println("You are attacked by mushroom monster.");
        System.out.println("Let's fight!");
        while (hp > 0 && mashroom > 0) {
            n = scanner.nextInt();　//ここの入力が飛ばされてしまう。
            System.out.println("You choose " + wayAttack[n] + ".");
            mashroom -= damage[n];
            System.out.println("You are hit by mushroom.");
            hp -= 2;
            System.out.println("Enemy's remaining HP is " + mashroom + ".");
        }
        if (hp == 0) {
            System.out.println("Game Over...");
            System.out.println("You wake up in the hospital.");
        } else if (mashroom <= 0) {
            System.out.println("You defeat the mushroom.");
        }
        scanner.close();
    }
}

試したこと

chatgpt を使ってみたものの、chatgptにはよくわからなかったみたい

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

openjdk version "21.0.2"
vscode -version "1.87.2"

Answer 1

以下を参照ください。
参考: JAVA　Scannerのインスタンス化について

Scanner#nextIntなどは最後の改行文字(\n)を取らないため、想定している挙動を示しません。
実装される場合はScanner#nextLineを挟んでおくか、そもそもScanner#nextLineで取得して数値に変換された方が安全かと思います。

Comments

[wawakaza]

入力部分を下記のようにしましたがダメでした。
初心者なので具体的にscanner.nextInt()をどの文法を使えると改善できるか教えていただけると助かります。
’’’
String input = scanner.nextLine(); // 改行文字を読み捨てる
n = Integer.parseInt(input);
’’’

[Refrain]

"ダメでした"とは、具体的に何があったのでしょうか？

[wawakaza]

下記がエラーコードです。
'''
Let's enjoy playing adventure!
Input your name in English.
testkun
At first, testkun should get to a higher level.
Let's go to kill weaker enemies than testkun.
Current level is 1.
Current HP is 10.
Skills that you can use now are hit protect runAway sleep
You are attacked by mushroom monster.
Let's fight!
Exception in thread "main" java.util.NoSuchElementException: No line found
at java.base/java.util.Scanner.nextLine(Scanner.java:1660)
at Enemy.fight(Enemy.java:10)
at Quest.main(Quest.java:24)
'''

[Refrain]

恐らく、`Hero#printStatus`の中で`Scanner#close`を実行したことで標準入力が閉じられたからと思われます。
例えば各処理の引数に`Scanner`を渡してあげて1個の`Scanner`で処理を行うなどされると良いかもしれません。

[Refrain]

または、以下のような事例かもしれません。
確認願います。
参考: [Scanner の使用時にエラー: Exception in thread "main" java.util.NoSuchElementException - スタック・オーバーフロー](https://ja.stackoverflow.com/questions/88603/scanner-%E3%81%AE%E4%BD%BF%E7%94%A8%E6%99%82%E3%81%AB%E3%82%A8%E3%83%A9%E3%83%BC-exception-in-thread-main-java-util-nosuchelementexception)

[wawakaza]

Hero.javaのscanner.close()を消したところ改善したので、おっしゃる通りHero.java内で標準入力が閉じられたから問題が起こったことが分かりました。
初心者すぎて申し訳ないのですが、"例えば各処理の引数に`Scanner`を渡してあげて1個の`Scanner`で処理を行うなどされると良いかもしれません。"の意味が分かりません。
もし良ければ、簡単にその部分のコードなどを書いていただけると助かります。
Hero.javaのscanner.close()を消すことで改善はしたので、もしそのままでも良いのであれば解決とさせていただきます。

[Refrain]

一旦は解決して良かったです。

凄く砕けた書き方はさせて貰います。
以下のようにメインの処理部で`Scanner`のインスタンスを持ち、呼び出し時に引数として`Scanner`のインスタンスを渡すという意味合いです。

```java
void main (String[] args) {
Scanner scanner = new Scanner(System.in);
do_something(scanner);
scanner.close();
}
void do_something (Scanner scanner) {
String text = scanner.nextLine();
System.out.println(text);
}
```

[wawakaza]

解決しました！
長い時間付き合って下さり、ありがとうございました。
コードの説明もすごくわかりやすかったです！

[Refrain]

個人的な見解ですが、クラスの構造をもう少し見直した方が、後々の開発時の管理が楽になるかなと感じました。

Answer 2

複数のクラスを作ればオブジェクト指向な訳ではありません。
現実の物質的なモノを模して個々のクラスを意味のあるデータ群とそれを操作するメソッド群の纏まりとして定義し、そのインスタンス(オブジェクト)を組み合わせて全体の動きを成すようにすることで、プログラムの見通しを良くすることに意味があります。
良く在りそうなのは、車を構成するモノとしてボディとかエンジンとかタイヤとかがありそれらもまたもっと細かいモノで成っている…といったことでしょうか。

質問のコードでは、Hero と Emeny が戦うという状況を再現していますが、それぞれクラスになっていてもデータは何も持っていません。これでは Hero や Enemy が無くても Main クラスに printStatus や fight メソッドを定義すれば事足ります。
Hero には Hero の 名前やレベル等、 Enemy には Enemy の 名前や hp 等を持たせて、戦い自体は Enemy のメソッドでは無く "場" として Main のメソッドで Hero や Enemy を使う形にします。

java は public で無ければ class を一つのファイル内に纏めることが出来ますので、簡単なプログラムはそのほうが管理が簡単です。

import java.util.*;

public class Quest {
  public static void main(String[] args){
    Scanner scanner = new Scanner(System.in);

    System.out.println("Let's enjoy playing adventure!");
    System.out.println("Input your name in English.");
    String name = scanner.nextLine();

    System.out.println("At first, " + name + " should get to a higher level.");
    System.out.println("Let's go to kill weaker enemies than " + name + ".");

    Hero hero = new Hero(name, 1, 10, new Attack[]{Attack.HIT,Attack.PROTECT,Attack.RUNAMAY,Attack.SLEEP});
    hero.printStatus();

    Enemy mashroom = new Enemy("mashroom", 10, 2); //敵: name="mashroom", hp=10, damage=2

    fight(scanner, hero, mashroom);
  }

  static void fight(Scanner scanner, Hero hero, Enemy enemy) {
    System.out.println("You are attacked by " + enemy.name + " monster.");
    System.out.println("Let's fight!");

    while (hero.hp > 0 && enemy.hp > 0) {
      int n = scanner.nextInt();
      Attack attack = hero.attacks[n];
      System.out.println("You choose " + attack.text + ".");
      enemy.hp -= attack.damage;
      System.out.println("You are hit by " + enemy.name + ".");
      System.out.println("Enemy's remaining HP is " + enemy.hp + ".");

      hero.hp -= enemy.damage;
    }

    if (hero.hp <= 0) {
      System.out.println("Game Over...");
      System.out.println("You wake up in the hospital.");
    } else {
      System.out.println("You defeat the " + enemy.name + ".");
    }
  }
}

//攻撃手段、ダメージ量
enum Attack {
  HIT("hit",5), PROTECT("protect",0), RUNAMAY("runAway",0), SLEEP("sleep",0);

  final String text;
  final int damage;
  Attack(String text, int damage) {
    this.text = text;
    this.damage = damage;
  }
}

//ヒーロー
class Hero {
  final String name;
  int level, hp;
  Attack[] attacks;

  Hero(String name, int level, int hp, Attack[] attacks) {
    this.name = name;
    this.level = level;
    this.hp = hp;
    this.attacks = attacks;
  }

  public void printStatus(){
    System.out.println("Current level is " + level + ".");
    System.out.println("Current HP is " + hp + ".");

    StringJoiner sj = new StringJoiner(" ", "Skills that you can use now are ", "");
    for(Attack a : attacks) sj.add(a.text);
    System.out.println(sj.toString());
  }
}

//エネミー
class Enemy {
  final String name;
  int hp, damage;

  Enemy(String name, int hp, int damage) {
    this.name = name;
    this.hp = hp;
    this.damage = damage;
  }
}

Let's enjoy playing adventure!
Input your name in English.
YUKIKAZE
At first, YUKIKAZE should get to a higher level.
Let's go to kill weaker enemies than YUKIKAZE.
Current level is 1.
Current HP is 10.
Skills that you can use now are hit protect runAway sleep
You are attacked by mashroom monster.
Let's fight!
0
You choose hit.
You are hit by mashroom.
Enemy's remaining HP is 5.
1
You choose protect.
You are hit by mashroom.
Enemy's remaining HP is 5.
2
You choose runAway.
You are hit by mashroom.
Enemy's remaining HP is 5.
3
You choose sleep.
You are hit by mashroom.
Enemy's remaining HP is 5.
1
You choose protect.
You are hit by mashroom.
Enemy's remaining HP is 5.
Game Over...
You wake up in the hospital.