C# Message Box Display

Question

Der folgende Code ist ein Formular, das nach einem beliebigen Text im Feld suchen sollte. Wenn kein Text gefunden wird, sollte ein Meldungsfeld angezeigt werden, in dem er es erneut versuchen kann. Code ist unter:

if (string.IsNullOrWhiteSpace(OrderField.Text)) 
{ 
    MessageBox.Show("Please input Order Number"); 
} 
else 
{ 
    MessageBox.Show("Derp"); 
} 
if (string.IsNullOrWhiteSpace(BoxField.Text)) 
{ 
    MessageBox.Show("Please input Number of Boxes"); 

} 
else 
{ 
    MessageBox.Show("Derp"); 
} 

Answer 1

Ich denke, Sie möchten das Formular validieren, wenn eine Eingabe leer ist? Schleife alle Textinput-Steuerelement in Form, sollten Sie den angezeigten Namen Sie für jede Textbox von Tag Attribut wollen gesetzt:

foreach(Control c in this.Controls) { 
    if(c is TextBox) { 
     var textbox = c as TextBox; 
     if(string.IsNullOrEmpty(textbox.Text)) { 
     MessageBox.Show(textbox.Tag.ToString() + " is empty"); 
     } 
    } 
} 

Answer 2

Ich schlage vor, eine Methode Extrahieren:

using System.Linq; 

... 

private static Boolean IsControlValid(Control control) { 
    if (!String.IsNullOrWhiteSpace(control.Text)) 
    return true; 

    // Let's be nice: put key board focus to the control 
    if (control.CanFocus) 
    control.Focus(); 

    MessageBox.Show(String.Format("Please input number at {0}.", control.Name)); 

    return false; 
} 

... 

private void SaveAndClose() { 
    // Put here all controls to be tested 
    Control[] controls = new Control[] { 
    OrderField, 
    BoxField, 
    }; 

    // do we have any controls that should be filled in? 
    if (controls.Any(control => !IsControlValid(control))) 
    return; 

    ... 
    // all controls are valid; so save/send the data 
    ... 

    Close(); // close the form 
}