Discussion :

[Waldoo]

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 :

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

mise_a_jour 1

et j'aimerai traduire en

Code :

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

<mise_a_jour>true</mise_a_jour>

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

Code :

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

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 :

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

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

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

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

[Invité]

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 :

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

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 :

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

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 :

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

1
2
3
4
 
bool ne_pas_ajouter_un;
ajouter_un ? ma_valeur_intégrale
           : ma_valeur_intégrale + 1;