Problème expression ternaire

Bonjour ,

Voici mon code :

Code:

---

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:

---

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.

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:

---

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:

---

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:

---

condition ? "valeur A" : "valeur B"

---

:arrow: 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:

---

(FlagMail != null) ? 0 : 1

---

:arrow: De type int

Code:

---

(FlagMail != null) ? 0 : null

---

:arrow: De type Integer (via autoboxing pour 0).


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

Code:

---

(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++

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

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

Code:

---

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