Discussion :

[astran]

Bonjour,

J'ai une erreur

Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException

sur l’exécution d'une méthode.
Je ne suis pas très sûr sur la façon dont j'ai écrit cette condition. Mais je vois encore moins pourquoi je dépasserais les cases du tableau.

Code :

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

1
2
3
4
5
6
7
8
9
10
    public static void changeNo(String[] annuaire, String nom, String nvno){
 
        int cpt=0;
 
        for (int i=0; i<annuaire.length || (nom.equals(annuaire[i].split(":"))); i++){
            cpt++;
            if(annuaire[i].split(":").equals(nom)){
                annuaire[i]=(nom+": "+nvno);
            }
        }

et puis je ne sais pas pourquoi la publication efface l'indentation. Ça fait pas très beau comme ça.

Quelqu'un saurait-il m'indiquer où se situe le problème ?

Merci d'avance pour votre aide.

[olivier.pitton]

J'ai du mal à comprendre ton code.

Pourquoi ne pas faire cela :

Code :

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

1
2
3
4
5
6
7
8
 
public static void changeNo(String[] annuaire, String nom, String nvno){
  for (int i=0; i < annuaire.length; i++){
    if(annuaire[i].split(":").equals(nom)){
      annuaire[i]=(nom+": "+nvno);
    }
  }
}

Tu testes deux fois la même condition, le equals.

[thelvin]

Mais je vois encore moins pourquoi je dépasserais les cases du tableau.

D'un côté tu vérifies si i est bien dans la taille du tableau, et d'un autre côté tu dis, "bon, ça, ou alors si annuaire[i] répond à telle condition".
Donc dès que i dépasse la taille du tableau, la première condition est fausse, et il va vérifier la deuxième, qui accède à annuaire[i], qui est en-dehors du tableau puisque la première condition est fausse.

[astran]

Merci, je comprends mon erreur. En fait l'idée était d'arrêter la boucle si la valeur est trouvée, car si elle n'existe pas il faut envoyer une erreur.
Je vais réfléchir à cela d'une autre façon.

[thelvin]

C'est pas OU - SI
C'est ET - SI PAS