Discussion :

[paesportcomputing]

Bjr,
J'ai un problème de 'java.lang.NullPointerException' en essayant de remplir une List. Voici mon code (remarquez que le println est là pour voir où est le problème et le boolean 'ok' est là pour limiter les null dans la liste 'ls').
Et pourtant, on peut voir dans la 1ere ligne d'erreur (avec le println) que 'temp' n'est pas vide !

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
 
private List<Supply> ls;
 
public void setLf() {
		List<Supply> temp = facade.searchSuppliesForOneSeller();
		System.out.println(temp+"");
		boolean ok = false;
		for (Supply s : temp) {
			if(s.isSold()){
				ok=true;
					if(ok){
						ls.add(s);
				}
			}
		}
	}

et j'ai tjrs le même message d'erreur:
[[idSupply=1, refSupply=zsc-000123, Table de jardin], [idSupply=2, refSupply=taja/789552-45, Table de salon], [idSupply=70, refSupply=ghh, tutu], [idSupply=71, refSupply="""", rrrr]]
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at gui.view.jmodel.ToysSoldForOneSellerTableModel.setLf(ToysSoldForOneSellerTableModel.java:91)...

Merci pour votre aide.

Marc

[doons]

Hello,

Ce n'est pas ta liste qui est vide, d'après la trace, la NPE est détectée dans le fichier ToysSoldForOneSellerTableModel.java:91, ligne 91

++

[tchize_]

Et pourtant, on peut voir dans la 1ere ligne d'erreur (avec le println) que 'temp' n'est pas vide !

Oui, mais ls est null, lui, a priori.

[paesportcomputing]

Merci pour votre aide,

En fait, le code que je montre, fait partie de la classe 'ToysSoldForOneSellerTableModel' (je n'ai repris que le code concerné)et la ligne 91 représente la ligne 12 ici c-à-d 'ls.add(s);'

Effectivement, 'ls' est null mais pour moi, ça ne change rien puisque le but ici est de la remplir (par ls.add(...))

[tchize_]

Heu ben si, ça change tout. tu veux remplir quoi, puisqu'il n'y a RIEN à remplir.

[paesportcomputing]

je veux remplir ls (qui est vide mais c'est normal puisque je la déclare !) par des Supply, dont un certain attribut (Sold=boolean) est false (ce qui explique la boucle foreach...)
Je comprends qu'avec une partie du code, c'est pas facile pour vous.

[doons]

ls n'est pas vide, ls est null.

Ok tu as déclaré ls mais tu dois l'instancier avant de la remplir ...

Code :

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

List<Supply> ls = new ArrayList<>();

[tchize_]

Je comprends qu'avec une partie du code, c'est pas facile pour vous.

Si si c'est très facile à comprendre, et je vous ai donné la réponse: ls est null. donc il ne vaut rien, et on ne peux pas appeler une méthode (ici add) sur du rien. Il faut que ls vaille quelque chose. Une liste vide en sommes:

Code :

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

1
2
3
4
ls = new ArrayList<Supply>();
ls = new LinkedList<Supply>();
ls = new Stack<Supply>();
ls = new Vector<Supply>();

au choix

[paesportcomputing]

Effectivement, c'est bien ça.
Et je m'acharnais la-dessus depuis 1 jour.
En fait, dans une autre classe, je remplis une List sans la déclaration 'ls = new ArrayList<Supply>();' et ça marchait. Mais je n'appelais pas de méthode (add par exemple) dessus directement.

Je faisais :

Code :

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

1
2
3
4
 
private List<Supply> ls;
 
ls=facade.searchSuppliesForOneSeller();

Encore merci pour l'aide.

Marc