Discussion :

[menuge]

salut à tous, voici mon code:

Code :

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

1
2
3
if(fromFile){tempOffSet = this.text.indexOf(" " + (String)words.get(i) + " ",tempOffSet+1);}else{
tempOffSet = this.text.indexOf((String)words.get(i),tempOffSet+1);
}

Quand je suis dans le else, tout se passe bien, mais dans l'autre cas (j'ajoute 2 espaces), c'est super long!

Quelqu'un voit une (grosse) connerie de ma part?

MErci d'avance

[bulbo]

Pas une grosse connerie mais ...

1- aere ton code c'est plus maintenable et lisible

2- evite les concatenation de chaines avec "+" mais utilise plutot un StringBuffer

Essaye voir comme ca pour voir si tu gagnes en perfs:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
if (fromFile)
{
  StringBuffer buf = new StringBuffer(" ").append((String) words.get(i)).append(" ");
  tempOffSet = this.text.indexOf(buf.toString(),tempOffSet+1);
}
else
{
  tempOffSet = this.text.indexOf((String)words.get(i),tempOffSet+1);
}

Sinon c'est peut-etre tout simplement parce qu'il ne trouve pas la sous-chaine avec les 2 espaces en plus et que du coup il est oblige de chercher sur toute la chaine au lieu de s'arreter des qu'il trouve.

Bulbo

[adiGuba]

Salut,

1- aere ton code c'est plus maintenable et lisible

+1 : un code lisible c'est nettement mieux

2- evite les concatenation de chaines avec "+" mais utilise plutot un StringBuffer

Dans ce cas là le StringBuffer est "inutile" puisque c'est exactement ce que fait l'opérateur + :

Code :

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

tempOffSet = this.text.indexOf(" " + (String)words.get(i) + " ",tempOffSet+1);

Est l'équivalent de :

Code :

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

tempOffSet = this.text.indexOf(new StringBuffer(" ").append((String)words.get(i)).append(" ").toString(),tempOffSet+1);

Donc l'opérateur + ne gène pas (au contraire il facilite la lisibilité).

C'est dans une boucle qu'il faut éviter de l'utiliser afin d'éviter de créer un StringBuffer temporaire à chaque itération...



Sinon pour ton problème, il nous faudrait un jeu de donnée pour tester, ainsi que la valeur de words.get(i)...

a++

[thibaut]

C'est dans une boucle qu'il faut éviter de l'utiliser afin d'éviter de créer un StringBuffer temporaire à chaque itération...

+1

Tout à fait d'accord, il ne faut pas être plus royaliste que le roi..

[picomz]

Salut,

Indente ton code, ce sera un peu plus clair...
Sinon je ne pense pas que le fait de rajouter 2 espaces augmente considérablement le temps de traitement (Même si je pense qu'il vaut mieux utiliser un StringBuffer).
Par contre regarde si tes variables on les mêmes valeurs quand tu es dans le if et dans le else.

Grilled

[menuge]

Désolé pour l'aération du code, mais j'ai fait un copier-coller de mon emacs et je comprends pas pourquoi il n'a pas gardé l'indentation.

Sinon, je vais essayer le StringBuffer...Merci

[menuge]

Ca ne marche pas...

Je ne comprends pas du tout ce qui peut se passer...

[®om]

Combien de caractères comporte ta chaîne?

[menuge]

pas plus de 10 en gros.

En fait, je récupère un texte et je dois retourner toutes les occurences (index) d'un mot. Et afin de m'assurer que c'est bien le mot, je fais:

Code :

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

text = text.replaceAll("[^a-zA-Z]", " ");

Et ensuite, je cherche le mot entouré de 2 espaces. Comme cela, ca m'évite d'avoir l'index de images alors que je cherche image