Si déclaration/valeur de contrôle?

Question

J'ai googlé mes doigts nus sur celui-ci et je sais que le code ici n'est pas très acceptable mais j'apprécierais vraiment de l'aide:

Cette case à cocher doit aller vérifier une valeur (booléenne) d'un bouton d'option.

Afficher un message différent en fonction de cette valeur et si True inclure une ligne supplémentaire dans un courrier, mais c'est un autre problème.

Toute aide serait grandement appréciée. J'ai vraiment besoin de prendre le coup de ce C#.

public void field42_Changed(object sender, XmlEventArgs e) 
{ 
    // checking Gsm checkbox(IOgsm@parzs.be) 
    if (e.NewValue.Equals("true")) 
    { 
     XPathNavigator xnMyForm = this.CreateNavigator(); 
     XmlNamespaceManager ns = this.NamespaceManager; 
     xnMyForm.SelectSingleNode("/my:myFields/my:txtGSM", ns).SetValue("P.Bab@gmail.Com"); 
     //string MobielInternet = xnMyForm.SelectSingleNode("/my:myFields/my:GsmMobileInternet", ns).Value; 
     if MobielInternet e.NewValue.Equals("true") 
      MessageBox.Show("An e-mail for GSM with Mobile internet will be sent"); 
     if (e.NewValue.Equals("false")) 
      MessageBox.Show("An email for the GSM will be sent"); 
     if (e.NewValue.Equals("")) 
      MessageBox.Show (" The Mobile Internet option has to been filled out "); 
    } 
    else if (e.NewValue.Equals("false")) 
    { 
     XPathNavigator xnMyForm = this.CreateNavigator(); 
     XmlNamespaceManager ns = this.NamespaceManager; 
     xnMyForm.SelectSingleNode("/my:myFields/my:txtGSM", ns).SetValue(""); 
    } 
} 

Answer 1

Ce serait ma suggestion pour le code plus propre. J'espère que je ne manque pas le point ici, mais vous ne comparez pas réellement la valeur de mobielinternet, vous compariez la valeur d'EventArgs encore.

Essayez ceci.

XPathNavigator xnMyForm = this.CreateNavigator(); 
XmlNamespaceManager ns = this.NamespaceManager; 
var emailAddress = ""; 
var message = ""; 

if (e.NewValue.Equals("true")) 
{ 
    emailAddress = "P.Bab@gmail.Com"; 
    string MobielInternet = xnMyForm.SelectSingleNode("/my:myFields/my:GsmMobileInternet", ns).Value; 

    if (string.IsNullorEmpty(MobielInternet)) 
     message = "The Mobile Internet option has to been filled out"; 

    if (bool.Parse(MobielInternet)) 
     message = "An e-mail for GSM with Mobile internet will be sent"; 
    else 
     message = "An email for the GSM will be sent"; 

    MessageBox.Show(message); 
} 
else if (e.NewValue.Equals("false")) 
{ 
    XPathNavigator xnMyForm = this.CreateNavigator(); 
    XmlNamespaceManager ns = this.NamespaceManager; 
    xnMyForm.SelectSingleNode("/my:myFields/my:txtGSM", ns).SetValue(""); 
} 


xnMyForm.SelectSingleNode("/my:myFields/my:txtGSM", ns).SetValue(emailAddress);