Parcours arrayList + remove object

Version imprimable

Boujour,
Je voudrais parcourir une arrayList que j'ai crée pour supprimer un élément de celle-ci.

Code:

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
33
34
35
36
37
38
public class Banque {
	private ArrayList<CompteBancaire> banque;
	protected int choix1 = 0;
	protected int compteAchercher = 0;
 
 
	public Banque(){
		banque = new ArrayList<CompteBancaire>();
	}
 
        //......du code qui fait un menu
 
       switch(choix1){
				case 1:
                                        //..j'entre dans mon menu est je demande à l'utilisateur de rentrere un numéro de comte
					Scanner chaine16 = new Scanner(System.in);
					System.out.print("Entrez un numero de compte");
					String s3 = chaine16.nextLine();
					compteAchercher = Integer.parseInt(s3);
 
					if(banque.isEmpty()== false){  //..Si ma liste n'est pas vide je commance l'iteration de la Liste.
						for(Iterator<CompteBancaire> iterator = banque.iterator();iterator.hasNext();){
							if(compteAchercher == CompteBancaire.numeroDeCompte){ //..Si le compteAchercher == au numeroDeCompte de l'objet CompteBancaire je fait mon traitement.
								System.out.println("compte trouvé...");
								int index = banque.indexOf(iterator); //..Je recupère l'index de mon iterator
								System.out.println(index);
								banque.remove(index); // ..je supprime mon objet compte bancaire
								System.out.println("compte supprimé");
							}
							else{
								System.out.println("compte non trouvé");
							}
						}
					}
					else{
						System.out.println("Aucun compte n'est présent dans la liste");
					}
					break;

je ne comprends pas comment faire car comme retour le programme m'affiche:
compte trouvé
-1
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: -1
at java.util.ArrayList.remove(Unknown Source)
at frunivcezanne.sircoulomb.banque.Banque.menu(Banque.java:141)
at frunivcezanne.sircoulomb.banque.Banque.menu(Banque.java:67)
at frunivcezanne.sircoulomb.banque.Banque.menu(Banque.java:43)
at frunivcezanne.sircoulomb.banque.Banque.menu(Banque.java:43)
at main.Test.main(Test.java:11)
Quelqu'un peut il m'aider?
Merci à tous

Bonjour,

L'erreur ce situe ici:

Code:

int index = banque.indexOf(iterator);

Tu passe l'instance d'iterator en paramètre de indexOf, l'instance d'iterator n'est pas dans ta liste (normal c'est une liste d'instance de CompteBancaire) tu as donc -1 en tant que valeur d'index d'où l'exception qui est lancée.

Il faut faire le code suivant pour récupérer l'objet depuis un iterator:

Code:

iterator.next();

La correction pour ton code (tu n'es pas obligé de passer par les index, tu peux directement appeler remove en passant l'objet en paramètre:
Code:

1 2 3 4 /*int index = banque.indexOf(iterator); //..Je recupère l'index de mon iterator System.out.println(index); banque.remove(index); // ..je supprime mon objet compte bancaire*/ banque.remove(iterator.next());

28/02/2009, 17h12
we.are.the.storm

Encore une chose,

Ton code peut-être largement optimisé. Il n'est pas utile de faire une itération sur l'ensemble de ta liste de CompteBancaire pour trouver l'instance correspondant au numéro de compte. Imagine la perte de temps sur une grosse liste !

Je te conseille d'utiliser l'interface Map au lieu de List en te servant du numéro de compte bancaire en temps que clé. Ca te permettra de récupérer directement le bon compte comme ceci :

Code:

banque.get(numeroDeCompte);
28/02/2009, 17h55
fasyr

thx

Ok je vais essayer avec Map car en effet si j'ai une grosse liste de compte cela risque d'être problématique.
Merci pour le coup de pouce!!

soucis

aye décidément!! j'ai encore un soucis.
j'ajoute correctement mes comptes, cepednant je ne peux supprimer que le dernier compte entrée. Si j'essaye de supprimer un autre compte que le dernier, je rentre dans mon else{compte non trouvé} et je boucle indéfiniment!

Code:

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
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
package frunivcezanne.sircoulomb.banque;
 
import java.util.ArrayList;
import java.util.Iterator;
import java.util.Scanner;
 
import frunivcezanne.sircoulomb.personne.Personne;
 
public class Banque {
	private ArrayList<CompteBancaire> banque;
	private float solde = 0;
	protected int choix = 0;//..variable pour le switch
	protected int compteAchercher = 0;
 
 
	public Banque(){
		banque = new ArrayList<CompteBancaire>();
	}
 
	public void menu(){
 
//..ajout d'un compte       
case 1:
					System.out.println("Creation d'un compte anonyme:\nEntrez un solde:");
					Scanner chaine2 = new Scanner(System.in);
					String montantAnonyme = chaine2.nextLine();
					solde = Float.parseFloat(montantAnonyme);
					CompteAnonyme cba1 = new CompteAnonyme(solde);
					cba1.affichageInfoCompte();
					banque.add(cba1);
					System.out.println("La liste contient " + banque.size() + " compte(s)");
					menu();
 
 
//.. supression d'un compte
case 2:
			Scanner chaine15 = new Scanner(System.in);
			System.out.print("1-Supprimer un compte bancaire grace à son numero de compte\n2-Quitter\nentrez choix:");
			String s2 = chaine15.nextLine();
			choix1 = Integer.parseInt(s2);
				switch(choix1){
				case 1:
					Scanner chaine16 = new Scanner(System.in);
					System.out.print("Entrez un numero de compte");
					String s3 = chaine16.nextLine();
					compteAchercher = Integer.parseInt(s3);
 
					if(banque.isEmpty()== false){
						for(Iterator<CompteBancaire> iterator = banque.iterator();iterator.hasNext();){
							if(compteAchercher == CompteBancaire.numeroDeCompte){
								System.out.println("compte trouvé...");
								banque.remove(iterator.next());
								System.out.println("compte supprimé");
							}
							else{
								System.out.println("compte non trouvé");
							}
						}
					}
					else{
						System.out.println("Aucun compte n'est présent dans la liste");
					}
					System.out.println("La liste contient " + banque.size() + " compte(s)");
					break;

Merci à tous!

Hello,

Là encore ton erreur vient de ton utilisation incorrecte de l'Iterator.
Il faut penser à récupérer l'instance courante de ta liste avec iterator.next() et travailler sur celle-ci :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 if(banque.isEmpty()== false){ CompteBancaire compteIterator = null; //Creation d'une variable qui va contenir une instance de compte bancaire pour chaque passage de la boucle for. for(Iterator<CompteBancaire> iterator = banque.iterator();iterator.hasNext();){ compteIterator = iterator.next(); //recuperation de l'instance CompteBancaire fournit par l'Iterator if(compteAchercher == compteIterator.numeroDeCompte){ //ici on teste le numero de compte de l'instance fournit par iterator et pas CompteBancaire.numeroDeCompte System.out.println("compte trouvé..."); banque.remove(compteIterator); //si le compte fournit par l'iterator correspond au numero de compte recherché, on le supprime System.out.println("compte supprimé"); } else{ System.out.println("compte non trouvé"); } }
Ce qu'il faut bien voir ici, c'est l'utilisation de next(), cette méthode va renvoyer l'élément contenu à sa position actuelle (comme un CompteBancaire[i] si c'était un tableau).
En même temps elle fait avancer l'itérator d'un pas (un peu comme tu ferais avec i++ sur une boucle standard).
Il faut donc bien penser à utiliser .next() à chaque passage de boucle pour pouvoir travailler sur l'élément courant rendu par l'iterator. Dans ton else tu n'avais pas de next(), tu ne faisais donc jamais avancer l'iterator d'où la boucle infinie.
Il faut également prendre garde à ne pas l'appeler plus d'une fois à l'interieur de la boucle for sinon tu va sauter des éléments (équivalent d'appeler plusieurs i++ dans une seule boucle).

02/03/2009, 00h55
fasyr

OK

J'ai testé avec les conseils que tu m'as donné la boucle infinie n'est plus présente mais je n'arrive qu'a supprimer le dernier élement entré dans la liste. Si j'essaye d'entrer un autre élément cela m'affiche :"compte non trouver..." X fois X étant le nombre de compte enregistré au préalable.
02/03/2009, 01h44
herch

salut,

est-ce que l'attribut numeroDeCompte de la classe CompteBancaire est static??

tu pourrais poster le code de CompteBancaire pour voir comment tu gères les numéros de compte?

CompteBancaire

Salut,
oui mon attribut numeroDeCompte est un attribut de classe il est en static.

Code:

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
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
package frunivcezanne.sircoulomb.banque;
 
public class CompteBancaire {
 
//------------------------------------------------Variable d'instance
 
	protected float solde = 0;
	protected static int numeroDeCompte  = 100000000 ;
	protected static int compteur = 1;
	protected int compteurTableau = 0;
 
//------------------------------------------------Constructeur CompteBancaire
	public CompteBancaire()
	{
		solde = 0;
		numeroDeCompte++;
	}
 
//------------------------------------------------------------------------------renvoi le numero de compte
	public int getNumeroDecompte(){
		return numeroDeCompte;
		}
 
//------------------------------------------------------------------------------renvoi le solde
	public float getSolde(){
		return solde;
		}
 
//------------------------------------------------------------------------------modification du solde
	public void setSolde(float s){
		solde = s;
		}
 
//------------------------------------------------------------------------------renvoi le solde
	public int getCreationCompteNumero(){
		return compteur;
		}
 
	public void depot(float montant) throws Exception{
		 solde += montant;
	}
 
	public void retrait(float montant) throws Exception{
		solde -= montant;
	}
 
 
//------------------------------------------------------------------------------Virement: retrait du compte cb1 et depot sur le compte cb2
	public static void virement(CompteBancaire cb1, CompteBancaire cb2, float montant)throws Exception {
 
		cb1.depot(montant);
		cb2.retrait(montant);
 
	}
}

voici le code de CompteBancaire. Il faut savoir aussi que 4 classes (CompteAnonyme, CompteClassique, CompteRamunere, ComptePlacement) heritent de cette classe CompteBancaire.
Veux tu que je poste les autres classe aussi?
Merci de ton aide!

02/03/2009, 22h06
herch

salut,

il ne trouvait que le dernier élément parce que le numero de compte est static, et donc tous les objets CompteBancaire partagent le même numéro.

par exp, à la création du premier compte, numeroCompte = 1
à la création du deuxième compte, numeroCompte = 2, c'est bon, mais c'est le cas pour tous les comptes créés, donc même le premier compte aura comme numero le 2

donc, essaie en enlevant l'attribut static du numeroCompte
04/03/2009, 21h42
fasyr

CompteBancaire

Mais si je supprime le static de ma variable numeroDeCompte toutes mes intances de CompteBancaire auront le même numero de compte non?
04/03/2009, 21h50
tchize_

au contraire, le static fait que toutes tes instances partagent cette valeur!
04/03/2009, 22h42
fasyr

CompteBancaire

Oui tout à fait d'accord, mais justement à chaque instance (dans mon constructeur) j'incrémentais de 1 mon "numeroDeCompte" pour que toutes mes instance de compte est un numeroDeCompte different. Si je supprime le static il auront tous le même numero de compte non ? ou alors j'ai vraiment mal compris la notion de static et si c'est le cas pourriez vous me l'expliquer brièvement?
04/03/2009, 22h56
tchize_

tu l'incrémente, mais comme elle est statique, tout les compte on la nouvelle valeur, donc t'as en rien résolu le problème. Le mieux est de passer le numéro de compte en paramètre du constructeur, et c'est celui qui crée les compte qui s'assure de l'unicité de ceux-ci

une autre proposition de solution

tu enlève le static de numeroDeCompte, puisque cette variable doit avoir une valeur propre à chaque instance, et donc elle ne peut être un attribut de classe, et tu crée un attribut nombreDeCompteCree qui est static, et tu l'incrémentes dans le constructeur, et puis tu ajoutes sa valeur à numeroDeCompte
Code:

1 2 3 4 5 6 7 8 9 10 11 protected int numeroDeCompte = 100000000 ; protected static int nombreDeCompteCree = 0; ,,, public CompteBancaire() { solde = 0; nombreDeCompteCree++; numeroDeCompte += nombreDeCompteCree; } ,,,
exemple:

--Construction du premier CompteBancaire :
nombreDeCompteCree = 1
numeroDeCompte = 100000000 + 1
--Construction du deuxième CompteBancaire :
nombreDeCompteCree = 2 ( c'est la même valeur qu'aura le premier CompteBancaire, puisque nombreDeCompteCree est static )
numeroDeCompte = 100000000 + 2 ( numeroDeCompte n'est pas static, donc ce ne sera pas la même valeur que le premier CompteBancaire )
...

08/03/2009, 16h48
fasyr

ok résolu

Ok that's it merci de votre aide