Bonjour, 

Je sais que beaucoup de gens ont déjà rencontré ce problème, j'ai regardé leurs solutions mais je ne comprend pas pourquoi ca ne fonctionne toujours pas ...

la première valeur que je rentre fonctionne mais à la deuxième j'ai une erreur : 

Scanner scanner = new Scanner( System.in );
            do{
                System.out.println("Nouvelle position ("+positionInf+"-"+positionSup+") : ");
                nouvellePosition = scanner.nextInt();//La ligne où on m'indique l'erreur
                scanner.nextLine();
            }while(nouvellePosition < positionInf || nouvellePosition > positionSup);
            
scanner.close();

Salut.

Java est un langage sensible à la casse donc scanner est différent de Scanner
Du coup réessayes avec :

nouvellePosition = Scanner.nextInt();

Abel Rudy a écrit:

Salut.

Java est un langage sensible à la casse donc scanner est différent de Scanner
Du coup réessayes avec :

nouvellePosition = Scanner.nextInt();

Pourquoi mettre une majuscule à "Scanner" car j'ai déclaré "scanner" sans majuscule ? 

Scanner scPosition = new Scanner( System.in );
        while((nouvellePosition<positionInf) || (nouvellePosition>positionSup))
        {
            System.out.println("scanner  : "+scPosition);
            System.out.println("nouvelle Position  : ");
            nouvellePosition=scPosition.nextInt();
            scPosition.nextLine();
        }
        System.out.println("Tu as choisis : " + nouvellePosition);
        scPosition.close();

scanner  : java.util.Scanner
[delimiters=\p{[match valid=false][need input=false][sourc][group separator=\x{202f}]
[decimal separatnegative prefix=\Q-\E][positive suffix=][neNaN\E][infinity string=\Q?\E]

scanner  : java.util.Scanner
[delimiters=\p{javaWhitespace}+][position=0][match valid=false][need input=false][source closed=false][skipped=false][group separator=\x{202f}]
[decimal separator=\x{2c}][positive prefix=][negative prefix=\Q-\E][positive suffix=][negative suffix=][NaN string=\QNaN\E][infinity string=\Q?\E]

int positionInf = getPositionInf(numeroJoueur);
        int positionSup = getPositionSup(numeroJoueur);
        int nouvellePosition=-1;

        Scanner scPosition = new Scanner( System.in );

        while((nouvellePosition<positionInf) || (nouvellePosition>positionSup))
        {
            System.out.println("nouvelle Position  : ");
            if(scPosition.hasNextInt())
            {
                nouvellePosition=scPosition.nextInt();
            }
                
        }

-
Edité par NATHdb 20 novembre 2020 à 17:10:39

Bonjour, 

Je pense que le problème vient de scPosition.nextLine()

Tu peux essayer ce code sinon :

import java.util.*;
import java.lang.*;

public class HelloWorld{

     public static void main(String []args){
        Integer positionInf = null;
        Integer positionSup = null;
        
        Scanner scPosition = new Scanner(System.in);
        
        System.out.println("Saisir position inferieure:");
        do {
            int value = scPosition.nextInt();
            positionInf = new Integer(value);
        } while (positionInf == null);
        
        System.out.println("Saisir position superieure:");
        do {
            int value = scPosition.nextInt();
            positionSup = new Integer(value);
        } while (positionSup == null);
        
        scPosition.close();
        
        System.out.println("Position inferieure = " + positionInf.intValue() + " ; Position superieure = " + positionSup.intValue());
     }
}

thetui a écrit:

Ton code est bon. Mais tu n'entres visiblement pas un entier.

tu es sûr que tu n'es pas dans le cas où tu tapes autre chose qu'un entier ? (un flottant, une chaine font planter avec ce message d'erreur)

traite ce cas via un try catch

        while((nouvellePosition<positionInf) || (nouvellePosition>positionSup))
        {
            System.out.println("scanner  : "+scPosition);
            System.out.println("nouvelle Position  : ");
            try{
                nouvellePosition=scPosition.nextInt();
            }catch(Exception e)
            {
                System.out.println("erreur de saisie !");
            }
            scPosition.nextLine();
        }
        System.out.println("Tu as choisis : " + nouvellePosition);
        scPosition.close();

Bonjour,

C'est un problème bien connu en Java que tout débutant rencontre, et étonnant de voir que personne ici ne l'a jamais rencontré.

Alors, oui nextInt() ne lit pas un entier, et non, @NATHdb n'est pas un abruti, il rentre bien un entier. C'est juste java.util.Scanner qui est pas du tout intuitif pour le coup.

https://www.geeksforgeeks.org/why-is-scanner-skipping-nextline-after-use-of-other-next-functions/.

La plupart des plateformes pour apprendre l'algo/la prog ou même des cours en Java proposent leur propre implémentation de Scanner pour régler ce problème d'ailleurs.

KoaTao a écrit:

Bonjour,

C'est un problème bien connu en Java que tout débutant rencontre, et étonnant de voir que personne ici ne l'a jamais rencontré.

Alors, oui nextInt() ne lit pas un entier, et non, @NATHdb n'est pas un abruti, il rentre bien un entier. C'est juste java.util.Scanner qui est pas du tout intuitif pour le coup.

https://www.geeksforgeeks.org/why-is-scanner-skipping-nextline-after-use-of-other-next-functions/.

La plupart des plateformes pour apprendre l'algo/la prog ou même des cours en Java proposent leur propre implémentation de Scanner pour régler ce problème d'ailleurs.

Alors si nextInt() renvoie un entier, la doc te le confirmera.

Personne n'a traité le PO d'abruti.

Et je ne vois pas le probleme dont tu parles, si tu pouvais nous exposer ta solution.

Pour revenir a ton lien on te dit juste que faire next() apres nextInt() te renvoie un retour chariot et pas de texte, ce qui explique le nextLine() en plus dans le code du PO.

la solution du lien c'est d'utiliser nexLine() et de convertir le résultat en entier

int var = Integer.parseInt(sc.nextLine());

Dans le post d'origine, il y avait bien le nextLine() pour consommer le retour chariot après le nextInt()

Sinon, encore une question qui peut paraître stupide, mais l'erreur se produit bien à cet endroit et pas un autre dans le fichier ? 

(si je pose des questions qui peuvent paraître stupide, c'est que parfois on oublie de vérifier certaines choses de bases ou qui semblent évidente et on s'entête à chercher ailleurs)

Autant pour moi, j'ai mal lu le code de l'OP, je pensais qu'il lisait une chaine de caractère après avoir lu l'entier (avec assignation à une variable), mais pas du tout en fait. Et j'ai mal lu l'erreur de toute manière qui n'a rien à voir en fait, donc totalement à côté pour ma part

(Quand je disais nextInt() ne lit pas un entier, je voulais dire que pour lui ce n'est pas un entier qui est rentré.)

Je trouvais ça par contre un peu dommage de lui dire plusieurs fois qu'il ne sait pas entrer un entier.

Je pense qu'umfred à raison, peut-être qu'il faut prendre en compte tout le code pour avoir le contexte exactement de comment il est exécuté, et ce qui pourrait empêcher Scanner de fonctionner correctement. NoSuchElement c'est qu'il n'y a rien à lire non ? Je veux dire, pas de Stream?

-
Edité par KoaTao 23 novembre 2020 à 20:01:39

Selon la doc, cette exception est déclenchée si l'input est "épuisée", ce qui je pense veut dire que non seulement il n'y a pas d'entier mais il n'y a RIEN (ou juste un retour chariot "\n").

Donc est-il possible que le PO nous dise exactement ce qu'il entre?

Si ce ne serait pas un entier qui serait lu mais autre chose, il aurait un autre type d'erreur:

https://docs.oracle.com/javase/7/docs/api/java/util/InputMismatchException.html

Par contre, son code fait partie d'un code plus vaste, et si par exemple, il a fait un sc.close(), il ferme le scanner et l'InputStreamReader System.in:

https://docs.oracle.com/javase/7/docs/api/java/util/Scanner.html#close()

Donc ce code par exemple renvoie une exception de type NoSuchElementException:

import java.util.Scanner;

class NoSuchElementExceptionExemple {
    public static void main (String[] args) {
        Scanner sc = new Scanner(System.in);
        System.out.println(sc.nextLine());
        sc.close();
        sc = new Scanner(System.in);
        System.out.println(sc.nextLine());
    }
}

 Il y a une exception parce que System.in est fermé la seconde fois.

Toujours selon la doc, l'exception est différente si le scanner est fermé.

Ici cela signifie que le scanner est ouvert mais qu'il n'y a rien à lire.