有什么方法可以简化这一点吗?

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是否应该可点击？