valeur booléenne suist de ? et (cha)"true":(char)"false"

Bonjour à tous,
J'utilise libxml2 en C++ pour transformer un fichier texte en fichier xml.

certains parametres du fichier texte sont donnés avec la valeur 1 ou 0

Code:

---

mise_a_jour 1

---

et j'aimerai traduire en

Code:

---

<mise_a_jour>true</mise_a_jour>

---

J'ai quelqu'un qui m'a dit de marquer dans l'exportXML cette commande.

Code:

---

xmlNewTextChild(pThisNode,NULL,(xmlChar*)"mise_a_jour",m_bmise_a_jour?(xmlChar*)"true":(xmlChar*)"false");

---

que va exactement faire le point d'interrogation et le deux point de

Code:

---

"m_bmise_a_jour?(xmlChar*)"true":(xmlChar*)"false""

---

?

j'hésite à faire un code de ce type, qui fonctionne, ais au moins que je comprends :

Code:

---

1 2	if (m_bmise_a_jour == true) xmlNewTextChild(pThisNode,NULL,(xmlChar*)"mise_a_jour",(xmlChar*)"true"); else if (m_bmise_a_jour == false) xmlNewTextChild(pThisNode,NULL,(xmlChar*)"mise_a_jour",(xmlChar*)"false");

---

mais j'ai peur que le code devienne moins lisible car il y a beaucoup de parametres booléens à traduire

L'operateur ": ?" est l'opérateur (non surchargeable) ternaire, le seul opérateur à trois opérande, comme son nom l'indique, du C++.Effectivment, on peut souvent le traduire par un if-else, et est équivalent à une fonction du genre :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

mon_type operateur_ternaire(bool condition, mon_type expr1, mon_type expr2) // expr1 et expr2 doivent impérativement être du même type { if(condition /* == true */ ) { return expr1; } else /* if (condition == false ) */ { return expr2; } }

---

Pour ce qui est de la lisibilité, ça dépend de ton code : je trouve personnellement que les ternaires ont une tendance rendre le code impossible à relire, vu qu' elles sont moins explicite que les conditions.
Donc, selon moi à n'utiliser qu'avec suffisamment d'espaces quitte à prendre plusieurs lignes :

Code:

---

1 2 3 4 5 6 7 8

xmlNewTextChild ( pThisNode,NULL, (xmlChar*)"mise_a_jour", m_bmise_a_jour ? (xmlChar*)"true" : (xmlChar*)"false" );

---

Enfin ne pas oublier les priorités des opérateurs, par exemple ce code ne fait pas ce qu'il parait, à première vue, réaliser :

Code:

---

1 2 3 4

bool ne_pas_ajouter_un; ajouter_un ? ma_valeur_intégrale : ma_valeur_intégrale + 1;

---