Exception in thread "main" java.util.NoSuchElementException

**mateolostanlen** · 03/01/2016, 17h29

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 ..

**joel.drigo** · 03/01/2016, 18h31

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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().

**mateolostanlen** · 03/01/2016, 21h36

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 ?

**joel.drigo** · 03/01/2016, 21h42

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

**mateolostanlen** · 03/01/2016, 21h49

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 )

**mateolostanlen** · 03/01/2016, 21h52

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

**joel.drigo** · 03/01/2016, 21h56

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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();
   }
 
}

**mateolostanlen** · 03/01/2016, 21h59

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

App :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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;
	}
 
 
}

**joel.drigo** · 03/01/2016, 22h06

Il n'y a pas de thread dans ton application ? La classe Start fait quoi ?
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.

**mateolostanlen** · 03/01/2016, 22h42

Voici la classe Start :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

**joel.drigo** · 03/01/2016, 22h52

Aucun lecture de System.in, via Scanner ou pas, que tu aurais oubliée quelque part ?

**mateolostanlen** · 04/01/2016, 09h23

Non aucun je viens de tout vérifier

**mateolostanlen** · 04/01/2016, 09h49

C'est bon ca marche, merci

**joel.drigo** · 04/01/2016, 10h11

Et qu'est-ce que tu as fait pour que ça fonctionne ?

**mateolostanlen** · 04/01/2016, 10h26

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é ..

**joel.drigo** · 04/01/2016, 10h27

Tu avais dû recompiler uniquement App probablement (pourquoi ne pas utiliser un IDE d'ailleurs ?).

**mateolostanlen** · 04/01/2016, 10h30

Quand je fais "javac fr/enseirb/battleship/App.java" ca compile uniquement App ? comment je dois faire pour compiler tout le projet ?

**joel.drigo** · 04/01/2016, 11h16

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 cmd :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Ou <racine> est le chemin complet du dossier qui contient le package racine.

**mateolostanlen** · 04/01/2016, 11h37

Très bien merci

Exception in thread "main" java.util.NoSuchElementException

Java

Discussions similaires

Partager

Partager