Discussion :

[k o D]

Bonjour à tous.

Alors oui je sais, le titre de ce sujet n'est pas très explicite mais c'est assez complexe de résumer en 1 ligne.

Voici mon problème, j'ai une StringIndexOutOfBoundsException avec comme valeur passée à un substring -1, jusque là c'est possible.

Mais voici le code concerné:

Code :

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

1
2
3
4
5
6
7
8
9
10
		for (String word : words) {
			if (!word.trim().equals("")) {
				String lastChar = word.substring(word.length()-1);
				if (lastChar.equals(".") || lastChar.equals("'")) {
					inputStreet += word;
				} else {
					inputStreet += word+ " ";
				}
			}
		}

La ligne concernée semble être if (!word.trim().equals("")) mais j'imagine que c'est plutôt celle du dessous.
Seulement... comment pourrais-je avoir un length = 0 si je suis passé dans la condition?

Je soupçonne un problème de charset (words provient d'une saisie d'utilisateur) mais j'aimerai en être vraiment certain.

Merci d'avance!

[hibour]

La classe String est immuable, quand tu appel trim() le string word ne change pas mais tu as une autre référence avec laquelle tu compare avec ""
fallait faire plutôt

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
for (String word : words) {
                        if (word == null) continue; //faire attention au NullPointerException
                        word = word.trim(); // <-- C'EST CE QUI MANQUAIT
			if (!word.equals("")) {
				String lastChar = word.substring(word.length()-1);
				if (lastChar.equals(".") || lastChar.equals("'")) {
					inputStreet += word;
				} else {
					inputStreet += word+ " ";
				}
			}
		}

A+-

[k o D]

Je suis bien d'accord que j'ai une autre référence... mais en quoi cela change-t'il le contenu?

Oui effectivement, je peux avoir une différence de longueur entre word et word.trim()... mais si word.trim() n'est pas égal à une chaine vide, word non plus!

[Uther]

Si par exemple word vaut " " alors word.trim() vaut bien "" et ne passera pas dans ton if.

[k o D]

Si par exemple word vaut " " alors word.trim() vaut bien "" et ne passera pas dans ton if.

En effet oui... c'est bien logique
Mais ça n'explique pas le soucis ici

[Uther]

Si tu as une chaine vide qui passe dans ton if, alors word.length()-1 vaudra -1 et donc logiquement, il y a un StringIndexOutOfBoundsException.