Discussion :

[eelloo]

Bonjour,
je débute juste en java et j'ai un soucis avec :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
public static void main(String[] args) {
		Scanner scan = new Scanner(System.in);
		String mot;
		System.out.println("Quel est le mot à tester ?");
		scan.nextLine();
		mot = scan.nextLine();
		scan.close();
		mot.length();
		int n = mot.length();
		String s1 = mot.substring(1, (n - 1));
}

J'ai l'erreur Exception in thread "main"

java.lang.StringIndexOutOfBoundsException: String index out of range: -2

at java.lang.String.substring(Unknown Source)
at tp3.Exo3.main(Exo3.java:16)

mais je ne comprends pas car je rentre des chaines de caractères toujours assez longues comme " abcdef"...
J'ai fait un test avec une chaîne de caractère en dur et ça fonctionne bien, quelqu'un saurait d'où vient l'erreur ?
Merci d'avance!!

[thelvin]

Hello,

est-ce vraiment voulu d'appeler deux fois scan.nextLine() et de ne rien faire avec la première ?
Ça m'a tout l'air d'une erreur et de la cause de tes problèmes.

[joel.drigo]

Salut,

nextLine(), ça consomme ce qu'il y a dans le tampon jusqu'au prochaine "newline", donc tout ce qui a été saisi jusqu'au premier "retour chariot". S'il n'y a rien de tel à consommer, ça attends qu'il y ait quelque chose à consommer, donc qu'on saisisse quelque chose.

Tu fais un premier nextLine() qui attend/consomme une première saisie, puis un second qui attend/consomme un seconde saisie, et qui la met dans la variable mot. La première saisie est perdue. Tu as d'ailleurs te rendre compte que tu étais obligé de taper deux fois sur entrée.

Si on saisit abcd puis entrée, puis entrée, tu as cette erreur, parce que la chaîne fais 0 (la seconde).
Si on saisit abcd puis entrée, puis efgh puis entrée, tu n'auras plus cette erreur, parce que la var mot contient efgh.
Si on saisit entrée, puis abcd puis entrée, tu n'auras pas non plus cette erreur.

Pourquoi faire le premier nextLine() ?

En revanche, de toute manière, tu devrais tester si mot est suffisant grand pour faire le substring, car rien n'empêche personne de saisir une chaîne de 1 caractère ou moins.

[eelloo]

Bonjour, merci beaucoup, en effet c'est le scanLine seul qui m'empechait de faire fonctionner mon code.
Je l'avais mis car dans un autre code, justement le scanLine ne fonctionnait pas et on m'avait dit de le mettre tout seul avant pour le faire fonctionner..
Pour la longueur de la chaine je vais la verifier merci beaucoup du conseil.
Bonne continuation.
Eelloo