C# 如何优化一个简单的求和？

Question

我正在尝试将用户在文本框中输入的数字相加。操作相当简单，但要求我尽可能优化代码。到目前为止，我已经创建了一个方法来验证文本框是否为空。然后我有一个将字符串从 textBox 转换为“int”的方法。但是，现在我不知道如何获取这些数字并在我的“求和”方法中使用它们。也许我忘记了一些基本的东西。

这是我的代码:

private void btnSum_Click(object sender, RoutedEventArgs e)
{
    String num1 = txtNum1.Text;
    String num2 = txtNum2.Text;    

    if(validate(num1,num2) == false)
    {
        MessageBox.Show("Empty fields");
    }
    else 
    {
       convertNum(num1, num2);    
       MessageBox.Show("The sum is: ");   
    }
}


public static Boolean validate(String n1, String n2) 
{
    if (n1 == null || n1.Equals("") || n2 == null || n2.Equals(""))
    {
        return false;
    }
    else 
    {
        return true;
    }
}

public static void convertNum(String n1,String n2) 
{
    int num1 = 0;
    int num2 = 0;
    try 
    {
        num1 = Int32.Parse(n1);
        num2 = Int32.Parse(n2);       
    }catch(FormatException)
    {
        MessageBox.Show("Input only numbers.");   
    }                
}

public static int sum(int n1, int n2) 
{
    int sum = n1 + n2;
    return sum;    
}

Answer 1

似乎只在按钮单击方法中执行所有逻辑会简单得多。您真正需要做的所有验证是检查文本值是否可以解析为整数。 int.TryParse 将为 null、空字符串和不是有效整数的字符串返回 false，所以这就是您真正需要的。

private void btnSum_Click(object sender, RoutedEventArgs e)
{
    int n1, n2;
    if(int.TryParse(txtNum1.Text, out n1) && int.TryParse(txtNum2.Text, out n2))
    {
       MessageBox.Show("The sum is: " + (n1 + n2));
    }
    else 
    {      
        MessageBox.Show("Enter valid numbers");
    }
}