Discussion :

[drake56]

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 :

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
	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

[Mobius]

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é.

[Heimdal]

Code :

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

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

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

[Mobius]

Code :

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

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

[adiGuba]

Salut,

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++

[Heimdal]

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

[drake56]

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

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 ?

[Heimdal]

Mouais, j'ai des doutes...

Fais voir ta classe entière

[drake56]

La collection se situe dans ma classe Bibliotheque que voici :

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
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> ();
	}
}

[Heimdal]

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

[drake56]

Voilà je t'ai tout mis

[Heimdal]

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?

[drake56]

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

[Heimdal]

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:

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 :

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

1
2
3
4
5
6
7
class Toto {
 
int toto;
 
void toto() {}
 
}

[Heimdal]

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

[drake56]

Ok ça marche :

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
	//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

[Heimdal]

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 :

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

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

[drake56]

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