Problème suppression élément d'une collection

Bonjour à tous,

J'ai réalisé une méthode pour supprimer un objet d'une collection dont on passera le numéro en paramètre.
Lorsque je souhaite supprimer le dvd n°2 pas de soucis par contre lorsque je souhaite supprimer le dvd n°1, ma méthode affiche n° dvd non trouvé.

D'où cela pourrait-il venir ?

Voici le code de ma procédure :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

public void suppresionDVD (int noDVD){ int arret = 0; int nbDVD = colDVD.size(); int i=0;   for(i=1;i<nbDVD;i++){ if (colDVD.elementAt(i).getNoMedia() == noDVD) { colDVD.removeElementAt(i); arret = 1; } }   if(arret==1){ System.out.println("Le DVD n° "+noDVD+" a bien été supprimé !"); } else { System.out.println("N°DVD non trouvé !"); } }

---

Merci d'avance pour votre aide

Comment est valorisé ta collection ? Es tu certain qu'elle contient bien l'élément que tu cherche ?
Tu peut rajouter des trace dans ta boucle pour afficher chaque élément et savoir quels sont ceux qui sont traité.

Code:

---

for(i=0;i<nbDVD;i++)

---

Le premier élément d'un tableau ou d'une collection se trouve a l'index 0

Citation:

---

Envoyé par Heimdal

Code:

---

for(i=0;i<nbDVD;i++)

---

Le premier élément d'un tableau ou d'une collection se trouve a l'index 0

---

Bien vu :ccool:

Salut,

Citation:

---

Envoyé par Heimdal

Le premier élément d'un tableau ou d'une collection se trouve a l'index 0

---

+1

Et au passage on privilégiera également l'utilisation d'ArrayList et de son interface List ainsi que de ses méthodes...

a++

Citation:

---

Et au passage on privilégiera également l'utilisation d'ArrayList et de son interface List ainsi que de ses méthodes...

---

C'est mieux mais je suis convaincu que pour ce genre d'indexage une Map serait tout à propos

Merci pour toutes vos réponses, en fait j'ai déjà essayé de mettre l'index à 0 mais j'ai eu cette erreur :

Citation:

---

Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 1 >= 1
at java.util.Vector.elementAt(Unknown Source)
at biblio.Bibliotheque.suppresionDVD(Bibliotheque.java:57)
at biblio.testBiblio.main(testBiblio.java:44)

---

D'autre suggestions ?

Mouais, j'ai des doutes...

Fais voir ta classe entière

La collection se situe dans ma classe Bibliotheque que voici :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

import java.util.Vector;   package biblio;   import java.util.Iterator; import java.util.Vector;   public class Bibliotheque { public Vector <Album> colAlbums; public Vector <DVD> colDVD;   public Bibliotheque() { colAlbums = new Vector <Album> (); colDVD = new Vector <DVD> (); } }

---

Classe entière parce que la je ne vois toujours pas pourquoi ca marcherait pas...

Voilà je t'ai tout mis

J'ai vu... mais en l'état pas comprendre pourquoi cela... Des idées en l'air

Pourquoi tes attributs sont publics? Sont-ils modifiés ailleurs dans ton programme? Plusieurs threads manipulent-ils ton objet?

Mes attributs sont public pour qu'ils soient accessibles dans l'ensemble du package, sinon ils sont utilisés dans une classe de test et pour le thread je ne pense pas

J'ai testé la méthode chez moi et pas de problèmes. Peut-etre est-ce ta classe de test qui est foireuse

Au passage:

Citation:

---

Mes attributs sont public pour qu'ils soient accessibles dans l'ensemble du package

---

Il existe la visibilité package-protected pour cela. public signifie accessible de partout.
Un attribut (ou une classe, ou une méthode) package-protected se déclare ainsi:

Code:

---

1 2 3 4 5 6 7

class Toto {   int toto;   void toto() {}   }

---

Peux-tu remettre tes méthodes add et affiche stp...

Ok ça marche :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

//permet d'ajouter un DVD à la bibliothèque public void ajoutDVD(DVD D){ colDVD.addElement(D); }   //permet d'afficher l'ensemble des informations des DVD présents dans la bibliothèque public void afficheDVD() { int nbDVD = colDVD.size(); int i = 0;   while ((i<nbDVD)){ System.out.println(colDVD.elementAt(i).getNoMedia()); System.out.println(colDVD.elementAt(i).getTitreMedia()); System.out.println(colDVD.elementAt(i).getAnneeMedia()); i++; } }

---

Merci pour ton aide

Bon déso, je dois être fatigué, je n'avais pas vu qu'on continuait à parcourir le vecteur une fois le dvd trouvé car on ne compare pas à la taille actuelle du vecteur.

Code:

---

1 2 3 4 5 6

for (i = 0; i < colDVD.size(); i++) { if (colDVD.elementAt(i).getNoMedia() == noDVD) { colDVD.removeElementAt(i); arret = 1; } }

---

C'est bien ça, un grand merci à toi, je savais que c'était un détail.
Hop résolu :)