Discussion :

[Kodut]

Bonjour à tous, je suis débutant en java et je bloque sur ce probleme :
j'ai créé une methode ChangeTab car je veux pouvoir remplacer tous les Caractères "A" par des "1". Eclipse ne me renvoie pas d'erreur de syntaxe mais le programme ne compile pas :



Merci par avance à ceux ou celles qui auront la patience de me répondre

[SkyZoThreaD]

Bonjour, essaye d'utiliser la balise code c'est plus pratique pour toi et nous
Je pense que pendant le

Code :

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

while(i<tab2.length && tab2[i] != "A")

Si i == tab2.length, Il vérifie aussi que tab2[i] != "A" et donc il essaye d'accéder en dehors du tableau.
Il faudrait placer la vérification tab2[i] !="A" dans le while pour éviter le dépassement.
De plus, dans ton exemple, cette vérification ne donnera jamais vrai car galeretab ne contient jamais "A". Ton code remplace les chaînes de caractères "A" par "1" et no pas les caractères 'A' de la chaîne...

edit: Ah non! le dépassement se passe pendant le System.out.print(galeretab[3]) !!
Quand il y'a 3 éléments dans un tableau, le dernier est le 2 car on commence de 0

[Kodut]

Merci de ta réponse j'ai modifié le code (voir ci-dessous), par contre quelque chose m'échappe. Je n'ai pas de dépassement puisque la ligne String galeretab [] = galere.split(""); crée un tableau avec 8 éléments. si j'avais mis un espace entre les "" ca aurait effectivement créé un tableau avec a1 b1 c1. Ici normalement ca crée un tableau avec a, 1, , b, 1, , c, 1.

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
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
public static void main(String[] args) {
 
 
 
		String galere = new String("a1 b1 c1");
		galere = galere.toUpperCase();
		String galeretab [] = galere.split("");
 
		ChangeTab(galeretab);
 
 
 
		System.out.print(galeretab [0]);
 
	}
 
	 static void ChangeTab(String tab2[]) {
 
 
		int i = 0; 
		while(i<tab2.length) {
			if (tab2[i] == "A"){
				tab2[i] = "1";
				i++;}
			else
				i++;
			}
		i = 0;
 
 
		}
	}

Avec ce code je n'ai pas d'erreur de compilation, par contre ca ne fonctionne pas. Les "A" du tableau ne se changent pas en "1"...

[joel.drigo]

Salut,

On n'utilise pas l'opérateur "identique" (==) pour comparer des chaînes, mais la méthode equals() (on peut utiliser l'opérateur == mais lorsqu'on sait exactement ce qu'on fait et comment les String et les objets plus généralement fonctionnent en Java).

Code :

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

1
2
if (tab2[i].equals("A")) {
   tab2[i] = "1";

Mais ton code serait bien plus simple et plus rapide si tu passais par des caractères, et, là, tu pourras utiliser == :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
String s="a1 b1 c1".toUpperCase();
 
char[] tableau = s.toCharArray();
for(int i=0; i<tableau.length; i++) {
   if ( tableau[i]=='A' ) {
      tableau[i]='1';
}
}
s=new String(tableau);
System.out.println(s);

[Kodut]

un tres grand merci joel.drigo tu m'as encore sorti de la panade