Discussion :

[l4r3nZu]

Bonjour ,

Voici mon code :

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
18
19
20
21
22
23
 
 
public class Test {
 
	/**
         * @param args
         */
	public static void main(String[] args) {
 
             Integer FlagErreur = null;
	Integer FlagMail = null;
	Integer rep = (FlagErreur != null && FlagErreur == 1 )
	      ?(FlagMail != null)
                       ?0
                       :1
	     :(FlagMail != null)
                        ?0
                        :null ; //Si je change par 1 je n'ai pas de NullPointerException
	 System.out.println(rep);
 
	}
 
}

Résultat :

Code :

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

1
2
3
 
Exception in thread "main" java.lang.NullPointerException
	at Test.main(Test.java:15)

J'attend bien sure qu'il maffiche null
Quelqu'un aurai une explication ?

Merci d'avance,
Cordialement.

[adiGuba]

Salut,


Avant toute chose : ce code est affreux !!!

Les expressions ternaires c'est déjà pas très lisible, mais en enchainé trois comme cela c'est affreux ! Mais en plus tu y mélanges de l'autoboxing :/

Un simple if fonctionnerait bien mieux :

Code :

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

1
2
3
4
5
6
7
	final Integer rep;
	if (FlagErreur != null && FlagErreur == 1) {
		rep = (FlagMail != null) ? 0 : 1;
	} else {
		rep = (FlagMail != null) ? 0 : null;
	}
	System.out.println(rep);

Voir même mieux :

Code :

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

1
2
3
4
5
6
7
8
9
	final Integer rep;
	if (FlagMail!=null) {
		rep = 0;
	} else if (FlagErreur != null && FlagErreur == 1) {
		rep = 1;
	} else {
		rep = null;
	}
	System.out.println(rep);

Maintenant pour revenir à ton problème, il semblerait que cette accumulation d'expression ternaire fasse perdre la boule au compilateur, surtout que tu y mélange de l'autoboxing.

Explication : une expression ternaire a un type, qui correspond au type de base des deux opérandes.

Exemple :

Code :

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

 condition ? "valeur A" : "valeur B"

L'expression est de type String, puisque les deux valeurs sont des String :



Dans ton exemple tu as 3 expressions ternaires.
Afin de déterminer le type de la première expression, il faut d'abord déterminer des deux sous-expressions :

Code :

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

(FlagMail != null) ? 0 : 1

De type int

Code :

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

(FlagMail != null) ? 0 : null

De type Integer (via autoboxing pour 0).


Du coup ton expression globale se présente sous cette forme :

Code :

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

(FlagMail != null) ? <int> : <Integer>

Apparemment l'expression ternaire semble privilégié le type natif, et prendra donc le type int. L'Integer de la seconde partie sera donc convertie en int...


Et bien sûr quand on essaye de convertir un null en int, on obtient une joli NullPointerException !



Bref deux conseils :
- Evites les expressions ternaires illisibles.
- Evites de trop jouer avec l'autoboxing, sauf pour des affectations simples.



a++

[l4r3nZu]

Pour les expression ternaire complexes je suis daccord mais j'utilise un ETL Et c'est soit creer une routine (fonction) ou une expression ternaire . Mon bout de code est pas assez gros pour creer une fonction et à peine un peu trop pour une expression ternaire.

Merci beaucoup il fallait trouver quand meme ^^

[adiGuba]

Dans ce cas il faut tout caster en Integer pour éviter le passage en int :

Code :

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

1
2
3
4
5
6
7
	Integer rep = (FlagErreur != null && FlagErreur == 1 )
	      ?(FlagMail != null)
                       ? (Integer)0
                       : (Integer)1
	     :(FlagMail != null)
                        ? (Integer)0
                        :null ;

a++