Discussion :

[Beber_lechat]

Bonjour à tout le monde;

J'ai encore besoin d'un petit peu d'aide.

Plusieurs fois au cours de mon apprentissage, j'ai été confronté à un problème:
Supprimer plusieurs objets en une boucle for sans lever d'exception
java.lang.ArrayIndexOutOfBoundsException

Je comprends pourquoi je la lève mais je ne trouve pas d'astuce pour contourner le problème.

Je vous met un morceau de code mais à titre purement indicatif:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
class EcouteurSuppression implements ActionListener {// notre ecouteur du bouton Suppression
		@Override
		public void actionPerformed(ActionEvent arg0) {
			for (int i=0; i<=Bouton.getNbreBouton(); i++){
				if (tableauBouton[i].getASupprimer() == true){
					System.out.println("A supprimer");
					panelCentre.remove(tableauBouton[i]);
					panelCentre.validate();
				}
			}			
		}		
	}

En gros ici, si la valeur booléenne d'un bouton est true, on supprime l'objet.
Mais si on supprime un objet et bien la dernière case de notre tableau est vide, et donc erreur.
Si je ne suis pas clair n'hésitez pas à me le dire.

Merci beaucoup de votre aide

[Alkhan]

bonjour,

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
class EcouteurSuppression implements ActionListener {// notre ecouteur du bouton Suppression
		@Override
		public void actionPerformed(ActionEvent arg0) {
			for (int i=0; i<=Bouton.getNbreBouton() - 1; i++){
				if (tableauBouton[i].getASupprimer() == true){
					System.out.println("A supprimer");
					panelCentre.remove(tableauBouton[i]);
					panelCentre.validate();
				}
			}			
		}		
	}

Il faut faire -1 car le tableau qui contient n element va de 0 a n -1

edit : et cela dans l'hypothèse ou Bouton.getNbreBouton() te renvoie le nombre de bouton contenu dans tableauBouton !
je dois dire que ton code est étrange

[Alkhan]

A mon avis le code devrait plutot ressembler à :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
lass EcouteurSuppression implements ActionListener {// notre ecouteur du bouton Suppression
		@Override
		public void actionPerformed(ActionEvent arg0) {
			for (int i=0; i<tableauBouton[i].length; i++){
				if (tableauBouton[i].getASupprimer() == true){
					System.out.println("A supprimer");
					panelCentre.remove(tableauBouton[i]);
					panelCentre.validate();
				}
			}			
		}		
	}

difficile de dire mieux sans plus d'infos !

[rolfone]

Code :

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

1
2
3
4
5
6
7
			for (int i=0; i<=tableauBouton.lenght -1 ; i++){
				if (tableauBouton[i].getASupprimer() == true){
					System.out.println("A supprimer");
					panelCentre.remove(tableauBouton[i]);
					panelCentre.validate();
				}
			}

l'index du tableau commence 0

ou pour eviter le -1

Code :

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

1
2
3
4
5
6
7
			for (int i=0; i<tableauBouton.lenght  ; i++){
				if (tableauBouton[i].getASupprimer() == true){
					System.out.println("A supprimer");
					panelCentre.remove(tableauBouton[i]);
					panelCentre.validate();
				}
			}

après ton problème viens peut être du fait que tu supprime des élément de la même collection que celle que tu parcoure dans ce cas la ta question est mal pausé car ton code ne monte pas ça

[Beber_lechat]

Merci beaucoup de votre aide.
Effectivement c'est bien plus propre avec tableauBouton.lenght et j'ai honte de pas l'avoir mis.

Cependant ça me lève toujours une exception. Je comprends pas, je veux supprimer les boutons qui ont comme booléens aSupprimer à true.

Bizarrement le code marche, enfin ca me supprime bien les boutons voulus, mais en fenêtre console j'ai une liste longue comme mon bras d'écritures rouges qui montrent bien que ça a foiré à un moment ou un autre.
Et ces sales lignes rouges commencent par:
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at Fenetre$EcouteurSuppression.actionPerformed(Fenetre.java:50)

La ligne 50 correspond à la ligne de test du booléen.

Et en plus de ça, comme l'a si bien fait remarqué rolfone, je voulais savoir comment supprimer des élements d'une collection que je parcours.

Un grand merci pour votre aide

[Alkhan]

je ne suis pas vraiment d'accord sur le fait que tu dis supprimer des éléments d'une collection que tu parcours. Sauf erreur de ma part tu n'as pas la main sur le tableau du panel qui lui aussi contient ces éléments.
Par conséquent, tu supprimes les éléments du tableau du panel avec le remove mais dans le tiens tu as des objets null (ce qui est étrange pour moi, sauf si tu affecte null dans ton tableau à un autre moment).

En tout cas, pour résoudre le problème il faut aussi que tu les supprimes de ton tableau.

Concrètement je vois deux possibilités pour la suppression :
- Mettre en place la suppression sur un tableau
- Utiliser une liste à la place du tableau car plus simple pour la suppression (les iterator savent très bien le faire)