有什么办法可以简化这个吗？

Question

我有两个要相乘的标签（工资和工时）。当用户没有输入数字时，我希望显示一个错误框，但我只能找出最简单的方法是制作两个 if 语句并这样做。任何帮助都会很棒。

    private void btnCalc_Click(object sender, EventArgs e)
    {
        double hours = Double.Parse(tbxHours.Text);
        double pay = Double.Parse(tbxPay.Text);

        if (Double.TryParse(tbxPay.Text,  out pay))
            {
            double result = hours * pay;
            MessageBox.Show($" total amount is {result} ", "Click Event",
            MessageBoxButtons.OKCancel, MessageBoxIcon.Information);

        }
        else
        {
            MessageBox.Show($"You must enter a number", "Input Error",
        MessageBoxButtons.OK, MessageBoxIcon.Error);

            tbxPay.Clear();

        }


        if (Double.TryParse(tbxHours.Text, out hours))
        {
            double result = hours * pay;
        }
        else
        {
            MessageBox.Show($"You must enter a number", "Input Error",
        MessageBoxButtons.OK, MessageBoxIcon.Error);

            tbxHours.Clear();

        }
    }

Answer 1

每个控件只需要解析一次。这是一个简化，您可以重复使用多个包含数字作为文本的控件。

private void btnCalc_Click(object sender, EventArgs e)
{
    if (TryParseFromTextBox(tbxHours, out double hours) && 
        TryParseFromTextBox(tbxPay, out double pay))
    {
        double result = hours * pay;
        MessageBox.Show($" total amount is {result} ", "Click Event", 
                        MessageBoxButtons.OKCancel, MessageBoxIcon.Information);
    }
}

public bool TryParseFromTextBox(TextBox control, out double value)
{
    if (!double.TryParse(control.Text, out value))
    {
        MessageBox.Show($"You must enter a number in {control.Name}", "Input Error", 
                        MessageBoxButtons.OK, MessageBoxIcon.Error);

        control.Clear();
        return false;
    }
    return true;
}

需要考虑的一些额外事项。

• 是否应该专注于数字解析失败的控件？
• 如果这些文本框中没有有效数字，您的btnCalc 是否应该可以点击？