Discussion :

[jmclej]

Bonjour,
Je ne comprends pas pourquoi cela ne fonctionne pas. Ça me donne l'erreur "Type of conditional expression cannot be determined because there is no implicit conversion between 'bool' and '<null>'". J'utilise bien un booléen nullable pourtant :

Code :

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

1
2
3
bool? b;
string s = "";
b = (s == "yes" ? true : (s == "no" ? false : null));

alors que ceci fonctionne :

Code :

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

1
2
3
4
5
6
7
8
bool? b;
string s = "";
if (s == "yes")
          b = true;
else if (s == "no")
          b = false;
else
          b = null;

Y a-t-il une manière plus concise qu'avec mes if/else if/else pour parvenir à mon résulat?
Merci pour vos retours,
Julien

[katkiller]

Salut,

Y a-t-il une manière plus concise qu'avec mes if/else if/else pour parvenir à mon résulat?

L'opérateur "?:" est vraiment utilise pour faire un "si" - "sinon".
Je pense que c'est une mauvaise idée pour la compréhension de vouloir mettre deux conditions dans cet opérateur.

[chaparo]

Bonjour,

@katkiller La msdn dit bien que c'est possible (tiré de ton lien)

L'opérateur conditionnel est associatif sur sa droite. L'expression a ? b : c ? d : e est évaluée comme a ? b : (c ? d : e), et non pas sous la forme (a ? b : c) ? d : e.

Je penses que le problème vient surtout du faite que tu travails sur un nullable.

tu dois renvoyer un même type.

Soit le type de first_expression et second_expression doivent être identiques, soit une conversion implicite doit exister d'un type à un autre.

[Sehnsucht]

Tu peux limiter tes conditionnelles si tu donnes d'avance la valeur null (valeur "par défaut").

Code :

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

1
2
3
4
5
bool? b = null;
if (s == "yes")
    b = true;
else if (s == "no")
    b = false;

Ainsi si s n'est ni yes ni no ; b conserve donc sa valeur de null.

Quant au problème initial, c'est car il n'existe pas de "type commun" entre false et null (qui n'a en soi pas de type) ; tu peux le résoudre en utilisant un cas ou en recourant à default :

Code :

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

1
2
b = s == "yes" ? true : s == "no" ? false : (bool?)null;
b = s == "yes" ? true : s == "no" ? false : default(bool?);

À voir si c'est "mieux" ; la concision ce n'est pas tout ; la lisibilité (et par extension la maintenabilité) c'est souvent plus important

[jmclej]

Merci,
@katkiller : Par rapport à la clarté, quand on a une petite condition imbriquée dans l'autre, je trouve ça plus claire justement de passer par une ligne avec un opérateur ternaire que par 6 lignes de if. Surtout que je dois répéter ce schéma beaucoup de fois d'affilées dans mon cas.

@chaparo : le truc c'est que le propre d'un booléen nullable c'est que ça peut justement avoir "2 types" : booléen et null.
Y a-t-il moyen dans mon cas de créer une conversion implicite pour que ça passe ?

[jmclej]

Merci @Sehnsucht, ça fonctionne comme ça effectivement.