Discussion :

[totot]

Bonjour,

J'ai un tableau composé de pièce et je souhaiterai supprimer l'élément que je passe en paramètre.
J'ai donc fait sauf que cela ne peut supprime par le bon élément

Code :

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

1
2
3
4
5
6
7
	public void retirerPieceAuStock(Piece unePiece){
		for (int j =  0 ; j < i ; j++){
			if(piece[j].getNom() == unePiece.getNom()){
				piece[j] = null;
		}
		}
	}

Merci de votre aide

[joel.drigo]

Salut,

Pour tester l'égalité de 2 instances de String, on utilise equals(), pas ==. Le problème est que == compare les références des instances, et tu ne peux pas être sûr que 2 variables de type String de même valeur sont la même instance. On peut le savoir quand on sait exactement comment sont construites les String, mais en général, on ne peut pas. Donc, pour t'éviter d'avoir à le savoir, utilise toujours equals().

[totot]

Je viens de faire sa mais sa fonctionne toujours pas:

Code :

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

1
2
3
4
5
6
7
	public void retirerPieceAuStock(Piece unePiece){
		for (int j =  0 ; j <= i ; j++){
			if(piece[j].getNom().equals(unePiece.getNom())){
				piece[j] = null;
			}
		}
	}

Exception in thread "main"
java.lang.NullPointerException
at retirerPieceAuStock(ligne du if)

[joel.drigo]

C'est parce que tu as des pièces qui n'ont pas de nom (getNom() est null), si ne tu n'avais déjà l'erreur avant. Sinon, cela peut être aussi parce que des cases du tableau sont vides : d'ailleurs, tu devrais prendre en compte ce cas, puisque c'est justement ce que tu fais pour retirer ta pièce du tableau. En tout cas, dès l'appel de ta méthode après l'avoir appelée et qu'elle ait retiré une pièce, tu auras cette exception.

Donc le test complet correct devrait être :

Code :

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

1
2
3
4
5
6
if( piece[j]!=null ) {
     String nom =  piece[j].getNom();
     if ( (nom==null && unePiece.getNom()==null) || (nom!=null && nom.equals(unePiece.getNom()) ) {
         ...
     }
}

Ou, plus globalement :

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
public void retirerPieceAuStock(Piece unePiece){
   if ( unePiece==null ) throw new IllegalArgumentException("unePiece null");
   if ( unePiece.getNom()==null ) {
      for (int j =  0 ; j <= i ; j++) {
         if( piece[j]!=null && unePiece.getNom()==null ) {
            piece[j] = null;
         }
      }
   }
   else {
      for (int j =  0 ; j <= i ; j++) {
         if( piece[j]!=null && unePiece.getNom().equals(piece[j].getNom()) ) {
            piece[j] = null;        
         }
      }
   }
}

Par ailleurs, je ne comprends pas pourquoi tu bornes ton for avec i et pas piece.length ? C'est quoi ce i ?

[totot]

Pourtant que je l'ai est créé elles avaient toutes des noms

[joel.drigo]

Alors c'est que tu as des éléments null dans ton tableau.

Pourquoi n'utilises-tu pas une List<Piece> (ArrayList<Piece>), voire une Map<String, Piece> ?