StringIndexOutOfBoundsException, trim et equals

Version imprimable

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:

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! :ccool:

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:

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

13/01/2010, 17h10
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!
13/01/2010, 18h29
Uther

Si par exemple word vaut " " alors word.trim() vaut bien "" et ne passera pas dans ton if.
13/01/2010, 22h37
k o D

Citation:

Envoyé par Uther

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 :)
14/01/2010, 08h01
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.
14/01/2010, 08h51
k o D

Citation:

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

Oui et c'est pourquoi la condition "if (!word.equals("")) {" a été placée plus haut depuis le début. :ccool:
14/01/2010, 09h16
ultimax

j'ai testé ce code rapidement et je n'obtiens aucune erreur. (avec vide, espace , mot standard et uniquement un retour chariot).

peux tu nous donner la stack trace? ou alors la liste de mots que tu essaye de traiter.

Citation:

Envoyé par ultimax

j'ai testé ce code rapidement et je n'obtiens aucune erreur. (avec vide, espace , mot standard et uniquement un retour chariot).

peux tu nous donner la stack trace? ou alors la liste de mots que tu essaye de traiter.

Je n'ai pas de liste de mot étant donné que c'est l'utilisateur qui encode les possibilités. C'est pourquoi je pensais à un problème de charset: l'utilisateur a peut-être copié du texte d'un document word ou autre.
En fait, je me dis que c'est une explication qui peut-être cohérente:
La méthode length() d'un String ne compte que les caractères unicode reconnus par Java et donc me renvoie 0. Tandis que la méthode equals() se base sur par exemple l'espace mémoire pris.

Quoiqu'il en soit, j'ai modifié le code et la condition en utilisant les StringUtils mais je posais la question ici parce que ça m'intrigue ^^

Voici la stack:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 15:32:48,080 ERROR [ExceptionMappingInterceptor] String index out of range: -1 java.lang.StringIndexOutOfBoundsException: String index out of range: -1 at java.lang.String.substring(String.java:1932) at java.lang.String.substring(String.java:1905) at be.ict.core.util.AddressUtil.replaceCharacters(AddressUtil.java:142) at be.ict.core.util.AddressUtil.getSignificantsPartsOfStreet(AddressUtil.java:110) at be.ict.core.util.AddressUtil.getSignificantsPartOfStreet(AddressUtil.java:89) at be.ict.core.services.DefaultAddressMatcherImpl.matchAddressFields(DefaultAddressMatcherImpl.java:351) at be.ict.core.services.DefaultAddressMatcherImpl.matchMunicipality(DefaultAddressMatcherImpl.java:125) at be.ict.core.services.DefaultAddressMatcherImpl.match(DefaultAddressMatcherImpl.java:62) at be.ict.web.actions.secure.AddressAction.match(AddressAction.java:188) at be.ict.web.actions.secure.AddressAction.processFormSubmission(AddressAction.java:132) at be.ict.web.actions.AbstractFormAction.execute(AbstractFormAction.java:42) at sun.reflect.GeneratedMethodAccessor9277.invoke(Unknown Source) at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25) at java.lang.reflect.Method.invoke(Method.java:597)

je ne vois pas d'ou viens le problème :calim2:

mais bon tu peux le résoudre comme cela

Code:

1
2
3
4
5
6
7
8
9
10
11
 
for (String word : words) {
			if (word!= null && word.trim().length()>0) { // check size instead of compare to ""
				String lastChar = word.substring(word.length()-1);
				if (lastChar.equals(".") || lastChar.equals("'")) {
					inputStreet += word;
				} else {
					inputStreet += word+ " ";
				}
			}
		}

14/01/2010, 11h23
dinobogan

Citation:

Envoyé par k o D

Quoiqu'il en soit, j'ai modifié le code et la condition en utilisant les StringUtils mais je posais la question ici parce que ça m'intrigue ^^

Tu vas devoir ajouter des logs pour tracer ce que l'utilisateur entre.
14/01/2010, 14h18
thelvin

Citation:

Envoyé par k o D

En fait, je me dis que c'est une explication qui peut-être cohérente:
La méthode length() d'un String ne compte que les caractères unicode reconnus par Java et donc me renvoie 0. Tandis que la méthode equals() se base sur par exemple l'espace mémoire pris.

Ce n'est pas comme ça que ça marche. Grosso-modo un String est fait d'un char[]. La méthode length() renvoie la taille de ce char[], et la méthode equals() vérifie que les char[] ont la même taille et que chaque char, à chaque emplacement, est le même.

En réalité c'est un peu moins simple, mais toujours est-il qu'une distortion entre length() et equals() serait un bug de la JVM, bug que je n'observe pas.
14/01/2010, 15h11
hibour

+1 pour ultimax, j'allais proposer la même chose :)