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

[totot]

Merci sa fonctionne!!!

Maintenant je vais me battre avec une autre méthode:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
	public Piece[] pieceStock(int annee){
		Piece pieceStock[] = new Piece[10];
		int j = 0;
 
		for (j = 0 ; j<=piece.length ; j++){
			if(piece[i].getAnneeEmission() == annee){
				pieceStock[j] = piece[i];
				j ++;
			}
		}	
		return pieceStock;
	}

[totot]

J'ai réussi à la faire mais après je bloque dans mon programme principal pour les afficher:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
	public Piece[] pieceStock(int annee){
		Piece pieceStock[] = new Piece[10];
		int j = 0;
 
		for (j = 0 ; j < i ; j++){
			if(piece[j].getAnneeEmission() == annee){
				pieceStock[j] = piece[i];
			}
			j ++;
		}	
		return pieceStock;
	}

Programme principal:

Code :

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

1
2
3
4
5
		Piece piece[] = new Piece[10];
		piece = stock.pieceStock(2012);
		for (int i = 0; i<piece.length; i++){
			piece[i].afficher();
		}

[joel.drigo]

Déjà, for (j = 0 ; j<=piece.length ; j++){ va planter : c'est j<piece.length (les index vont de 0 à taille tableau - 1). En plus, il y a des problèmes dans ton utilisation de i et j (un sacré mélange).

Mais pour faire ce genre de choses, tu devrais utiliser des ArrayList, pour éviter d'avoir à dimensionner tes tableaux de manière arbitraire et avoir des nulls dans certains éléments, et donc à avoir à les traiter. En plus, tu pourrais avoir plus facilement le compte d'éléments.

[joel.drigo]

Code :

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

1
2
3
4
5
		Piece piece[] = new Piece[10];
		piece = stock.pieceStock(2012);
		for (int i = 0; i<piece.length; i++){
			piece[i].afficher();
		}

[

Bah, oui, il y a (probablement) des nulls dans ton tableau, donc ça doit faire des NullPointerException. Utilise des List !

[totot]

Je sais que la première fonctionne pas. c'est pour sa que j'ai une seconde version. Mais après je bloque dans le programme principal pour afficher les pièces

[totot]

D'accord pour le programme principal.
Sinon elle est bonne cette méthode car j'ai rien qui s'affiche:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
	public Piece[] pieceStock(int annee){
		Piece pieceStock[] = new Piece[10];
		int j = 0;
 
		for (j = 0 ; j < i ; j++){
			if(piece[j].getAnneeEmission() == annee){
				pieceStock[j] = piece[i];
			}
			j ++;
		}	
		return pieceStock;
	}

Je n'est pas le droit d'utiliser les listes

[joel.drigo]

De toute manière, il y a des problèmes dans (toujours avec i et j) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
	public Piece[] pieceStock(int annee){
		Piece pieceStock[] = new Piece[10];
		int j = 0;
 
		for (j = 0 ; j < i ; j++){
			if(piece[j].getAnneeEmission() == annee){
				pieceStock[j] = piece[i];
			}
			j ++;
		}	
		return pieceStock;
	}

Tu incrémentes 2 fois j dans chaque itération, donc ça risque de planter (sauf si i<=piece.length/2), mais en tout état de cause, ça fait un peu n'importe quoi cette méthode : ça met la même pièce piece[i] dans une case sur 2 au mieux (si toutes les pièces sont de l'année demandée).

Là, on a un code logique :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
public Piece[] pieceStock(int annee){
     Piece[] pieceStock = new Piece[piece.length];
     int i = 0;
     for (int j = 0 ; j < piece.length; j++){
        if(piece[j].getAnneeEmission() == annee){
	     pieceStock[i] = piece[j];
             i ++;
	}
    }	
    return pieceStock;
}

Et en List :

Code :

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

1
2
3
4
5
6
7
8
9
public List<Piece> pieceStock(int annee){
     List<Piece>  pieceStock = new ArrayList<>;
     for (Piece piece : pieces){
        if(piece.getAnneeEmission() == annee){
	     pieceStock.add(piece);
	}
    }	
    return pieceStock;
}

[joel.drigo]

Si tu n'a pas le droit d'utiliser les listes, soit tu dois prendre en compte qu'il peut y avoir des nulls dans tes tableaux, soit il faut copier le tableau dans un nouveau tableau plus court quand on a déterminé sa taille (qui ne peut être connue d'avance, dans le cas du filtre sur les années).