检查文本框中的否定输入

Question

我正在尝试验证 Windows 窗体应用程序中的文本框。文本框接受任何数字（它应该是小数，但也接受整数）。我不希望该数字为负数，但即使在添加 if 语句之后，即使输入的数字是否定的，应用程序仍然接受它。我不确定我做错了什么。

try
{
    //Different operations being done here that use the input from txtEnterTotal.Text 
}
catch
{
    decimal entertotal = Convert.ToDecimal(txtEnterTotal.Text);
    if (entertotal <= 0)
    {
        MessageBox.Show("Please enter a valid number for the total field.", "Entry Error");
    }
}

我也试过了

catch
            {

                if (decimal.TryParse(txtEnterSubtotal.Text, out decimal value))
                {
                    if (value < 0)
                    {

                        MessageBox.Show("Please enter a valid number for the Subtotal field.", "Entry Error");
                    }
                    else
                    {
                        MessageBox.Show("thank you");
                    }
                }

            }

我发现发布了一个类似的问题，但它使用的是我尚未学过的另一种语言，但由于它仍然询问相同的问题，我尝试了一些发布的答案，但我仍然遇到问题。 链接：How do you check if an input is a negative number in VB

注意：我只想使用 try catch 语句来执行此操作。

Answer 1

示例代码的一个问题是您在catch 块中执行Convert.ToDecimal(txtEnterTotal.Text)，但如果txtEnterTotal.Text 不是有效数字，这将引发异常，因此现在将未处理异常。

既然你说你真的想使用try/catch 来验证文本框，那么基本模式是尝试转换try 块中的数字，如果失败，采取行动（那不会' t 抛出另一个异常）在 catch 块中。

例如：

private void btnValidate_Click(object sender, EventArgs e)
{
    try
    {
        // Here we perform the operation that might throw an exception
        decimal value = Convert.ToDecimal(txtEnterSubtotal.Text);

        // If we get here, no exception was thrown
        MessageBox.Show("Thank you");
    }
    catch
    {
        // Since there was an exception, show a message and clear the textbox
        MessageBox.Show("Please enter a valid, positive number");
        txtEnterSubtotal.Clear();
        txtEnterSubtotal.Focus();
    }
}

---

但是，使用 try/catch 进行简单的错误处理是“昂贵的”（捕获调用堆栈是有代价的），而且这也不是它们的预期目的（它们应该用于异常事件，而不是正常程序流程的控制）。

这是了解数字类型所具有的TryParse 方法的好时机。该方法接受string 进行解析，如果成功，则将数字类型的out 参数设置为转换后的值。最好的部分是它返回一个表示成功的bool，因此我们可以在if 条件下使用它，并在字符串解析失败时采取一些措施。

例如，您可以在不再需要try/catch 的验证方法中包含此代码，因为我们改为使用TryParse 进行验证：

private void btnValidate_Click(object sender, EventArgs e)
{
    // Here we check if `TryParse` does NOT return true (note the exclamation mark), OR
    // if the converted number less than zero, where in either case we take some action
    if (!decimal.TryParse(txtEnterSubtotal.Text, out decimal value) ||
        value < 0)
    {
        // Show a message, then clear the textbox
        MessageBox.Show("Please enter a valid, positive number");
        txtEnterSubtotal.Clear();
        txtEnterSubtotal.Focus();
    }
    else
    {
        MessageBox.Show("Thank you");
    }
}

Answer 2

我认为验证您的输入是否为数字和小数的最佳方法是使用如下代码。作为评论者，@MickyD 建议，Decimal.TryParse 像这样：

try
{
    //Different operations being done here that use the input from txtEnterTotal.Text 
}
catch(Exception ex)
{
    // catch the exception and DO something with it.
    System.Diagnostics.Trace.TraceError("Error before try/parse: {0}", ex);
    //decimal entertotal = Convert.ToDecimal(txtEnterTotal.Text);
    // old code ^^^^^^
    // new code 
    if (decimal.TryParse(txtEnterTotal.Text, out decimal entertotal))
    {
        if (entertotal <= decimal.Zero)
        {
            MessageBox.Show("Please enter a valid number for the total field.", "Entry Error");
        }
    } 
    else 
    {
        MessageBox.Show(string.Format("Failed to parse value: {0}", txtEnterTotal.Text));
    }
}