Exception in thread "main" java.util.NoSuchElementException

Version imprimable

Bonjour à tous.

J'ai pour projet de créer une bataille navale en java. J'ai créé une classe Action qui a pour but de demander au joueur ce qu'il souhaite faire : "fire" pour tirer ou "view" pour afficher la grille de jeux. Voici ma classe Action :

Code:

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
package fr.enseirb.battleship;
 
import java.util.Scanner;
 
public class Action {
 
    private int x;
    private int y;
    private String act;
    public Action(){
        this.act="action";
        this.x=0;
        this.y=0;
    }
 
    public void get(){
         Scanner sc = new Scanner(System.in);
            System.out.println("Select an action (fire or view) :");
            this.act = sc.next();
            System.out.println("You choose : " + this.act);
            if(this.act.equals("fire")){
                System.out.println("Select the target position X Y");
                this.x = sc.nextInt();
                 this.y = sc.nextInt();
                System.out.println("You choose : " + this.x +","+this.y);
            }
 
    }
 
    public String getact(){
        return this.act;
    }
    public int getx(){
        return this.x;
    }
    public int gety(){
        return this.y;
    }
 
 
}

Lorsque je la teste dans une classe Test elle fonctionne sans problème :

Code:

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
package fr.enseirb.battleship;
 
import fr.enseirb.battleship.ship.*;
 
import java.util.Scanner;
 
import fr.enseirb.battleship.grid.*;
public class Test {
 
    public static void main(String[] args){
 
 
 
    Action action = new Action();
 
 
 
            action.get();
            System.out.println(action.getact());
 
            if(action.getact().equals("fire")){
 
 
                System.out.println("on est dans fire");
            }
            else if(action.getact().equals("view")){
 
                System.out.println("on est dans view");
            }
 
            else if(action.getact().equals("debug")){
 
                System.out.println("on est dans debug");
            }
 
 
        }
 
 
 
    }

Mais quand je l'utile dans ma classe Game cela ne fonctionne plus :

Code:

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
package fr.enseirb.battleship;
 
import fr.enseirb.battleship.ship.*;
import fr.enseirb.battleship.shot.*;
import fr.enseirb.battleship.draw.*;
 
import java.util.Scanner;
 
import fr.enseirb.battleship.grid.*;
 
public class Game {
 
    private int run;
    private int x;
    private int y;
    private View view = new View();
    private Debug debug = new Debug();
 
public Game(){
 
}
 
    public void rungame(ShipList playershiplist, ShipList aishiplist, Grid grid){
 
 
        PlayerShot playershot = new PlayerShot(playershiplist, grid);
        AiShot aishot=new AiShot(aishiplist, grid);
        Action action=new Action();
        action.get();
 
 
        if(action.getact().equals("fire")){
 
 
            playershot.playershot(action.getx(), action.gety());
            aishot.randomshot();
        }
        else if(action.getact().equals("view")){
 
            view.view(grid, aishiplist, playershiplist, aishot.getshotlist(), playershot.getshotlist());
        }
 
        else if(action.getact().equals("debug")){
 
            debug.debug(grid, aishiplist, playershiplist, aishot.getshotlist(), playershot.getshotlist());
        }
 
 
    }
 
}

J'obtient le message d'erreur suivant :

Code:

1
2
3
4
5
6
Exception in thread "main" java.util.NoSuchElementException
    at java.util.Scanner.throwFor(Scanner.java:862)
    at java.util.Scanner.next(Scanner.java:1371)
    at fr.enseirb.battleship.Action.get(Action.java:19)
    at fr.enseirb.battleship.Game.rungame(Game.java:28)
    at fr.enseirb.battleship.App.main(App.java:36)

Et je n'arrive pas à comprendre pourquoi ..

Salut,

Ce serait intéressant de voir le code de la classe App. Mais je suppose que le problème vient du fait que la méthode get() de la classe Action est appelée plusieurs fois, ce qui provoque plusieurs créations d'instances de Scanner sur System.in. Le problème vient du fait que la classe Scanner lit les données depuis le tampon (buffer) de System.in, et peut commencer à y lire d'avance certains octets. Si tu crées plusieurs instances de Scanner, une des instances peut donc commencer à lire des caractères qu'une autre instance va chercher à récupérer lors de l'appel de next() ou nextInt(), mais ces octets n'étant plus dans le buffer, l'instance de Scanner ne trouve rien à lire et soulève l'exception.

POC d'Illustration :
Code:

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 public class DemoScanner { public static void main(String[] args) { // on simule la saisie de l'utilisateur, des 26 lettres de a à z, avec appui de la touche entrée entre chaque int nb=0; StringBuilder inputString = new StringBuilder(); for(char c='a'; c<='z'; c++, nb++) { inputString.append(c); inputString.append('\n'); } try(ByteArrayInputStream inputStream = new ByteArrayInputStream(inputString.toString().getBytes())) { // on lit la saisie simulée exactement comme si on la lisait dans System.in //Scanner scanner = new Scanner(inputStream); // OK while (nb-->0) { Scanner scanner = new Scanner(inputStream); // KO String s = scanner.next(); System.out.print(s); System.out.print(' '); } System.out.println(); } catch (IOException e) { e.printStackTrace(); } } }
Il ne faut donc créé qu'une seule instance unique de Scanner sur un même InputStream (dans ton cas, une variable static par exemple dans la classe Action). De plus, tu as intérêt à utiliser les méthode du type hasNext(), ou hasNextInt(), pour vérifier que la saisie du joueur est conforme à ce que tu cherches à lire par les appels de next() ou nextInt().

03/01/2016, 21h36
mateolostanlen

Merci de ta réponse rapide :) .
Pour avoir un seul est unique scanner je dois donc le créer dans ma classe App (Classe contenant le main) et le faire passer en entrée de toutes mes autres classes ?
03/01/2016, 21h42
joel.drigo
Oui, c'est une solution, en particulier sur tu as besoin de l'utiliser dans plusieurs instances (de même classe ou de différentes classes). Ou la déclarer en public final static, dans App.
Code:

1 2 3 4 5 6 public class App { public static final Scanner SCANNER = new Scanner(System.in); /* ... */ }
Et y accèder là où tu en a besoin par App.SCANNER
03/01/2016, 21h49
mateolostanlen

y acceder par App.SCANNER ?

Je dois faire un "import App.SCANNER; " , comme pour une classe ?

(j'ai l'impression que cette question est stupide désolé je débute en java )
03/01/2016, 21h52
mateolostanlen

A c'est bon il suffit d'écrire App.SCANNER pour utiliser la variable, en effet question stupide :?

Ce n'est pas du tout une question stupide, au contraire. Ce n'est pas utile : il faudra importer la classe App en revanche. Mais, pour éviter d'avoir à indiquer App. systématiquement (et donc à faire l'import de App), tu peux faire un import static.

Exemple :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 package fr.enseirb.battleship.machinchose; import static fr.enseirb.battleship.App.SCANNER; public class Exemple { public int lireTruc() { System.out.print("Saisir le truc :"); while( !SCANNER.hasNextInt() ) { SCANNER.next(); System.out.print("Le truc doit être un int...\nRecommencez : "); } return SCANNER.nextInt(); } }

Cela ne fonctionne toujours pas je met donc ma classe App et ma classe Action modifié :

App :

Code:

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
package fr.enseirb.battleship;
 
import fr.enseirb.battleship.draw.*;
import fr.enseirb.battleship.grid.Grid;
import fr.enseirb.battleship.shot.*;
import fr.enseirb.battleship.ship.*;
import java.util.Scanner;
 
public class App {
 
	public static final Scanner SCANNER = new Scanner(System.in);
 
 
 
	public static void main(String[] args) {
 
		String action = args[0];
		System.out.println(action);
		String gridfile = args[1];
		String shipfile = args[2];
		System.out.println("Start");
		Start start = new Start(gridfile,shipfile);
		System.out.println("Start done");
		Game game = new Game();
		Debug debug = new Debug();
		AiShot aishot = new AiShot(start.getaishiplist(),start.getgrid());
		PlayerShot playershot = new PlayerShot(start.getplayershiplist(),start.getgrid());
 
		  if(action.equals(new String("debug"))){
 
				debug.debug(start.getgrid(), start.getaishiplist(), start.getplayershiplist(), aishot.getshotlist(), playershot.getshotlist());
			}
 
		  else if(action.equals(new String("play"))){
			  System.out.println("Play");
			  game.rungame(start.getplayershiplist(), start.getaishiplist(), start.getgrid());
			  System.out.println("Play done");
		  }
		  else{
			  System.out.println("Action not found");
		  }
	}
 
}

Action :

Code:

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
package fr.enseirb.battleship;
 
 
 
public class Action {
 
	private int x;
	private int y;
	private String act;
	public Action(){
		this.act="action";
		this.x=0;
		this.y=0;
	}
 
	public void get(){
 
			System.out.println("Select an action (fire or view) :");
			this.act = App.SCANNER.next();
			System.out.println("You choose : " + this.act);
			if(this.act.equals("fire")){
				System.out.println("Select the target position X Y");
				this.x = App.SCANNER.nextInt();
				 this.y = App.SCANNER.nextInt();
				System.out.println("You choose : " + this.x +","+this.y);
			}
 
	}
 
	public String getact(){
		return this.act;
	}
	public int getx(){
		return this.x;
	}
	public int gety(){
		return this.y;
	}
 
 
}

03/01/2016, 22h06
joel.drigo
1. Il n'y a pas de thread dans ton application ? La classe Start fait quoi ?
2. Tu exécutes comment ton programme ? (par une ligne de commande en pipant un fichier de test pour les saisies par exemple, au lieu d'utilise l'entrée console standard ?)
Par ailleurs, rien à voir, mais, au lieu d'écrire if(action.equals(new String("debug"))){, écris if(action.equals("debug")){ ! Ou encore if( "debug".equals(action)){ pour éviter les risques de NullPointerException, ou encore mieux utilise un switch, puisque tu testes plusieurs valeur pour action (code plus clair et concis).

PS : pense à la notation lowerCamelCase pour les noms de méthodes, c'est tout de même plus lisible que seulement avec des minuscules, et puis c'est la convention.

Voici la classe Start :

Code:

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
package fr.enseirb.battleship;
 
 
 
import fr.enseirb.battleship.ship.*;
import fr.enseirb.battleship.grid.*;
 
public class Start {
 
 
	private Grid grid = new Grid();
	private ShipList aishiplist = new ShipList();
	private ShipList playershiplist = new ShipList();
 
 
 
 
	public Start(String gridfile, String shipfile){
	     ReadGridXml readgridxml = new ReadGridXml();
		 SetPlayerShipList setplayershiplist = new SetPlayerShipList();
		 SetAiShipList setaishiplist = new SetAiShipList();
 
 
		this.grid=readgridxml.getgrid(gridfile);
		this.aishiplist=setaishiplist.initialiseaishiplist(grid, shipfile);
		this.playershiplist=setplayershiplist.initialiseplayershiplist(grid, shipfile);
	}
 
 
 
public Grid getgrid(){
	return this.grid;
}
 
public ShipList getaishiplist(){
	return this.aishiplist;
}
public ShipList getplayershiplist(){
	return this.playershiplist;
}
}

Elle permet d'initialiser le jeux.

Non je n'est pas de thread.

Je lance le jeux via la console:

Code:

1
2
3
 
javac fr/enseirb/battleship/App.java
java fr.enseirb.battleship.App play grid.XML ship.XML

03/01/2016, 22h52
joel.drigo

Aucun lecture de System.in, via Scanner ou pas, que tu aurais oubliée quelque part ?
04/01/2016, 09h23
mateolostanlen

Non aucun je viens de tout vérifier :?
04/01/2016, 09h49
mateolostanlen

C'est bon ca marche, merci :lol:
04/01/2016, 10h11
joel.drigo

Et qu'est-ce que tu as fait pour que ça fonctionne ?
04/01/2016, 10h26
mateolostanlen

Justement rien c'est ce que je ne comprends pas je me demande si les modifications précédente avaient bien été prises en compte. J'étais en train d'essayer autre chose en effectuant des scanner un peu partout pour trouver le moment précis du bug puis ca a marché ..
04/01/2016, 10h27
joel.drigo

Tu avais dû recompiler uniquement App probablement (pourquoi ne pas utiliser un IDE d'ailleurs ?).
04/01/2016, 10h30
mateolostanlen

Quand je fais "javac fr/enseirb/battleship/App.java" ca compile uniquement App ? comment je dois faire pour compiler tout le projet ?
04/01/2016, 11h16
joel.drigo

Si on indique qu'un nom de classe, seule cette classe est compilée, oui. On peut indiquer la liste de tous les fichiers .java (ou un fichier qui contient la liste). On peut utiliser un <chemin>/*.java, mais ce n'est pas récursif.

Le plus simple est d'utiliser un IDE : ça n'aide pas seulement pour la compilation.

Sous Windows, on doit pouvoir faire :

Code:

for /r <racine> %i in (*.java) do javac -cp <racine> %i

Ou <racine> est le chemin complet du dossier qui contient le package racine.
04/01/2016, 11h37
mateolostanlen

Très bien merci