我怎样才能让这个方程在 C# 的嵌套 if/else 语句中工作？

Question

当我在用户端获得 pass 20 后，我的 if/else 不会获取方程的下一个变体。我知道它是因为设置了总计 = (numberofchecks *二十检查) + (fee);但我不知道用什么来让它移动到方程的下一个变体。负数、超过 1000 和格式例外有效。

总计 = (支票数量 * 二十张支票) + (费用)​​;

            TotalLabel.Text = total.ToString("C");

            // if else (5 or 6)

            if ( numberofchecks < 20 && numberofchecks == 0)
            {
                total = (numberofchecks * twentychecks) + (fee);
               
            }
            else if ( numberofchecks < -1)
            {
                MessageBox.Show("Number of checks must be between 0 and 1000");
            }
            else if (numberofchecks == 20 && numberofchecks <= 39)
            {
                total = (numberofchecks * thirtyninechecks) + (fee);
                
            }
            
            else if (numberofchecks == 40 && numberofchecks <= 59)
            {
                total = (numberofchecks * fiftyninechecks) + (fee);
                
            }
            else if (numberofchecks == 60 && numberofchecks <=1000)
            {
                total = (numberofchecks * thousandchecks) + (fee);
                
            }
            else if (numberofchecks > 1000)
            {
               MessageBox.Show("Number of checks must be between 0 and 1000");
            }
            
        }
        catch(FormatException )
        {
            MessageBox.Show(" Please enter a number");
        }

Answer 1

首先，我们只查看您的IF语句之一

else if (numberofchecks == 20 && numberofchecks <= 39)

，如果numberFchecks等于20，那么我们会看到以下良好的状态，然后始终是＆lt; = 39，因此第二部分是多余的。所有陈述都是一样的。

我认为您实际上的意思是

else if (numberofchecks >= 20 && numberofchecks <= 39)

，这将使它更加合乎逻辑。

这应该解决您的直接问题。但是我们可以进一步改善它。

通常最好先检查数据输入是有效的。因此，这应该首先完成。您甚至可以将其移至另一种方法。 （如果您有很多验证可以执行）。然后，您可以执行计算，

if ( numberofchecks < -1 || numberofchecks > 1000)
{
        MessageBox.Show("Number of checks must be between 0 and 1000");
}
else
{    
    //Rest here
}

但是如果使用C＃9或更高版本，则实际上可以将IF语句移至支持范围的Switch语句。在这种情况下，我们也可以将默认情况用于无效数据。

switch (numberofchecks)
{
    case >= 0 and < 20:
        total = (numberofchecks * twentychecks) + (fee);
        break;
    case >= 20 and <= 39:
        total = (numberofchecks * thirtyninechecks) + (fee);
        break;
    case >= 40 and < 59:
        total = (numberofchecks * fiftyninechecks) + (fee);
        break;
    case >= 60 and < 1000:
        total = (numberofchecks * thousandchecks) + (fee);
        break;
    default:
        MessageBox.Show("Number of checks must be between 0 and 1000");
        break;
}

Answer 2

查看代码，似乎条件无法正确设置。

一个更好的有条件设置是始终首先检查不适当的案例。

因此，可以将以下检查范围降低到有效的检查范围。

// Check inappropriate cases first to reduce the scope of other checks
if (numberofchecks < 0 || numberofchecks >= 1000) {
    MessageBox.Show("Number of checks must be between 0 and 1000");
} else {
    // Guess you don't want 20 to satisfy the twentychecks as well? 
    if (numberofchecks < 20) {
        // cases: 0 to 19
        total = (numberofchecks * twentychecks) + (fee);
    } else if (numberofchecks < 40) {
        // cases: 20 to 39
        total = (numberofchecks * thirtyninechecks) + (fee);
    } else if (numberofchecks < 60) {
        // cases: 40 to 59
        total = (numberofchecks * fiftyninechecks) + (fee);
    } else {
        // cases: 60 to 999
        total = (numberofchecks * thousandchecks) + (fee);
    }
    TotalLabel.Text = total.ToString("C");
}