scanner.nextLine après scanner.nextInt

Version imprimable

Bonjour à vous tous. Je débute en java et du premier jour je tombe sur un os.
voilà mon code:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
import java.util.Scanner;
public class k25 {
 
	/**
         * @param args
         */
	public static void main(String[] args) {
		// TODO Auto-generated method stub
		Scanner clavier= new Scanner(System.in);
 
		System.out.println("entrer nombre");
		int n= clavier.nextInt();
 
		System.out.println("entrer nom");
		String nom= clavier.nextLine();
 
		System.out.println("entrer prenom");
		String prenom= clavier.nextLine();
	}
 
}

résultat:

Code:

1
2
3
4
5
entrer nombre
25
entrer nom
entrer prenom
stracoma

le programme ne me cède pas la main pour écrire mon nom
Merci

31/08/2015, 20h03
joel.drigo

Salut,

L'appel de nextInt() lit le prochain int qui se trouve dans le tampon (jusqu'au prochain délimiteur(une espace par défaut, ou la fin de la ligne), mais celui-ci n'est pas consommé (il sont juste ignorés si on fait un autre nextInt() sur la même ligne), mais quand tu as saisi l'entier, tu as tapé "entrée" après, qui lui est aussi dans le tampon. Il faut donc terminer de lire la ligne pour passer à une autre ligne, donc ajouter un nextLine(), avant de faire le nextLine() qui te sert à saisir le nom.
31/08/2015, 20h39
francky74

Bonjour bonsoir !

Ton instance de Scanner, que tu as appelé clavier n'est jamais clôturée. Il faut faire un clavier.close() lorsque tu as fini de t'en servir.

Le nom d'une classe doit toujours commencer par une lettre majuscule comme tu peux le voir dans le conventions de nommage (voir lien) énumérées ci-dessous par Monsieur Doudoux :

Citation:

91.3.4. Les conventions de nommage des entités

Les conventions de nommage des entités permettent de rendre les programmes plus lisibles et plus faciles à comprendre. Ces conventions permettent notamment de déterminer rapidement quelle entité désigne un identifiant, une classe ou une méthode.

Les classes, les interfaces et les constructeurs :
La première lettre est en majuscule.
Si le nom est composé de plusieurs mots, la première lettre de chaque mot doit être en majuscule, ne pas mettre de caractère underscore '_'
Le nom d'une classe peut finir par impl pour la distinguer d'une interface qu'elle implémente.
Les classes qui définissent des exceptions doivent finir par Exception.

Ainsi ta classe k25 doit être transformée en K25 (avec un 'k' majuscule). Pour ce faire, si tu est sur Eclipse, tu sélectionne la classe (donc le mot k25), puis clic droit et refactor ou renamme ....
;)

Hello !

Pour ma part j'ai une autre façon de faire pour récupérer des entrées clavier, que je trouve un peu moins embêtante. Je te laisse le morceau de code ci-dessous (qui est valide):

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
package fr.songird.buffer;
 
import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
public class ReaderChar {
 
	public static void main(String[] args){
		BufferedReader lecteur = new BufferedReader(new InputStreamReader(System.in)); //Lecture du flux d'entrée en paramètre (l'entrée standard)
		try{
			String retourClavier = lecteur.readLine(); //pour une chaine de caractères;
 
			System.out.println(retourClavier);
 
			int retourClavierEntier = Integer.parseInt(lecteur.readLine()); //pour un entier
 
			System.out.println(retourClavierEntier);
		}catch(NumberFormatException exceptionInt){
			exceptionInt.printStackTrace();
		}catch(IOException exceptionIO){
			exceptionIO.printStackTrace();
		}finally{
			lecteur.close();
		}
 
	}
 
}

Après, il te suffit de remettre le code dans un constructeur pour le réutiliser à volonté ! :)

Cordialement,

Songbird.

01/09/2015, 02h05
thelvin

Citation:

Envoyé par francky74

Ton instance de Scanner, que tu as appelé clavier n'est jamais clôturée. Il faut faire un clavier.close() lorsque tu as fini de t'en servir.

Oui enfin, il faut, il faut... Ça peut pas faire de mal tout au plus.

Il faut toujours fermer ce qu'on ouvre, et du coup c'est un bon réflexe, quand on crée un Scanner, de le fermer à la fin.
Mais bon, System.in on l'a pas ouvert, et qui prend la peine de le fermer à chaque fois ? Dans ce cas, pourquoi faudrait-il fermer un Scanner sur lui ? En réalité ça n'a pas d'importance.

Citation:

Envoyé par Songbird_

Pour ma part j'ai une autre façon de faire pour récupérer des entrées clavier, que je trouve un peu moins embêtante. Je te laisse le morceau de code ci-dessous (qui est valide):

Je me permets de résumer quand même que ça consiste à préférer

Code:

Integer.parseInt(lecteur.readLine())

au lieu de

Code:

scanner.nextInt()

Avec l'avantage néanmoins que la gestion des lignes est très claire, c'est vrai.
01/09/2015, 03h36
joel.drigo

Je trouve ça doublement problématique :

Citation:

Envoyé par Songbird_

Après, il te suffit de remettre le code dans un constructeur pour le réutiliser à volonté ! :)

Déjà, c'est plutôt contraignant que l'instanciation d'une classe nécessite des entrées au clavier : il vaut mieux lire des valeurs au clavier d'une part (dans une méthode dédiée) et instancier la classe en passant des paramètres, ce qui permet d'instancier la classe avec des paramètres de n'importe quelle source (clavier, fichier, base de données, valeurs en dur dans le code, valeurs produites par des méthodes d'autres classes, etc.)

Secondement, le fait de fermer en plus le flux System.in dans le constructeur (ce qui n'est pas utile (voir explication de @Thelvin ci-avant)), fait qu'on ne pourra plus lire quoique ce soit dans ce flux après instanciation de la classe, donc qu'on ne peut instancier qu'une seule instance de cette classe.
01/09/2015, 22h51
stracoma

Bonsoir. Merci de votre aide.
J'ai un peux compris le message de joel.drigo, pour les autres , c'est du chinois pour moi, car je viens à peine de commencer le Java (1 séance de 1 h).
Mai j'ai finalement trouvé une explication détaillée ici
https://www.youtube.com/watch?v=bbCr...?v=bbCr7CE97lU
Merci à vous