Discussion :

[ahmedpa]

Bonsoir,

Je suis entrain de tester un exemple sur le piechart avec ce bout de code :

Code :

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

1
2
3
4
<h:form>
			<p:pieChart id="sample" value="#{adminCtr.pieModel}" 
           title = "Mans and Womans" />
						</h:form>

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
public PieChartModel getPieModel() {
 
			listcandidat = candidatBuss.consulterCandidat();
			int numberMans=0;
			int numberWomans=0;
			for(int i=0;i<listcandidat.size();i++){
				if(listcandidat.get(i).getGenre().equals("Masculin"))
					numberMans++;
				else numberWomans++;
			}
			pieModel = new PieChartModel();
			pieModel.set("Mans", numberMans);
			pieModel.set("Womans", numberWomans);
			return pieModel;
		}

et j'ai eu un javanullpointerException sur if(listcandidat.get(i).getGenre().equals("Masculin"))?

Est-ce-que vous savez pourquoi s'il vous plait?

Merci

[joel.drigo]

Salut,

Une NullPointerException veut dire que tu essayes d'appeler une méthode sur une référence null, qui ne pointe donc sur aucun d'objet.

Donc dans ta ligne, tu as l'un des éléments suivant qui est null (ça ne peut pas être listCandidat sinon tu aurais l'exception sur la condition du for) :

• listcandidat.get(i)
• listcandidat.get(i).getGenre()

A toi de déterminer lequel...

[ahmedpa]

oui j'ai trouvé au fait c'est une ligne dans ma base qui était null pour la colonne genre.

Mais si je veux que dans ma ligne que ma colonne soit parfois null, comment je pourrais écrire ça ?

Merci

[joel.drigo]

Tu veux dire que tu ne veux pas que ton code plante lorsque genre est null ?

il te suffit de considérer ce cas, comme n'importe quel autre : du coup, tu vas avoir trois valeurs : "Masculin", "Féminin" (je suppose) et null. Pour éviter le null pointer exception, tu peux inverser la condtiion de test de la chaîne :

Code :

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

1
2
3
4
5
6
7
8
String genre = listcandidat.get(i).getGenre();
if ( "masculin".equals(genre) ) { // ici "masculin" n'étant forcément pas null, tu n'auras jamais de NullPointerException, et ce sera forcément false si genre est null, puisque null n'est pas égal à "Masculin"
   numberMans++;
} else if ( "Feminin".equals(genre) ) {
   numberWomans++;
} else {
   // a toi de voir comment tu gères les cas de candidat sans genre...
}

Autre approche : éliminer au plus tôt les cas d'exception, comme le null :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
String genre = listcandidat.get(i).getGenre();
if ( genre==null ) {
 
    // traiter ce cas...
 
}
else {
 
   // maintenant tu peux faire le test que tu faisais, tu peux être sûr ne plus avoir l'exception, 
   if(genre.equals("Masculin"))
					numberMans++;
				else numberWomans++;
 
}

Afin, d'éviter les problèmes de syntaxe, de majuscule, minuscule, d'accent, etc dans des valeurs comme Masculin et Féminin, je te conseillerais d'utiliser si possible une énumération (enum) plutôt que des chaînes de caractères.

[ahmedpa]

Merci