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.

[k o D]

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.

[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.

[k o D]

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 :

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

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)

[ultimax]

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

mais bon tu peux le résoudre comme cela

Code :

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

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+ " ";
				}
			}
		}

[dinobogan]

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.

[thelvin]

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.

[hibour]

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