Supprimer élément d'un tableau

Version imprimable

01/11/2014, 10h53
totot

Supprimer élément d'un tableau
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:

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
01/11/2014, 12h01
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().

Je viens de faire sa mais sa fonctionne toujours pas:
Code:

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)

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:

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:

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 ?

01/11/2014, 12h20
totot

Pourtant que je l'ai est créé elles avaient toutes des noms
01/11/2014, 12h23
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> ?

Merci sa fonctionne!!!

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

Code:

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

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

Code:

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:

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

01/11/2014, 12h45
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.
01/11/2014, 12h47
joel.drigo
Code:

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 !
01/11/2014, 12h48
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

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

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

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

Code:

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:

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:

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

01/11/2014, 13h03
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).