c# - 为什么我的 "Divide by Zero"预防不起作用？

Question

<分区>

Answer 1

改变这个:

if (symbol == "/" && number2 == 0) // This part seems to be broken.
                answer.Text = "Invalid input.";

收件人:

if (symbol == "/" && number2 == 0) { answer.Text = "Invalid input."; return; }

解释: 您代码中“if”子句中的条件是正确的。它做了预期的事情 - 它将 answer.Text 属性更改为“无效输入”但是稍后它在这一行再次更改:

answer.Text = ans.ToString("n"); // Change label value to a number.

并且因为您在“if”子句中的条件返回 true - “else” block 未执行。这就是您看到 0.00( double 类型的默认值)的原因。

附言 因此，通过将 return 语句添加到 if 子句，您基本上可以结束您的方法。 这就像你对编译器说*“嘿如果这家伙试图除以零提醒他'Ivalid input'并且什么都不做，从方法返回”。*

解决问题的其他方法是:

if (divide by zero attempt) { your code here } else
{
    and place rest of your method code here
}

但我不推荐这样做，因为它使用了多余的 else 语句和 {} 数字。您可以使用 if() {....; 来避免所有这些情况。返回; } 在你的情况下。

您还可以从使用 switch block 中获益，您的代码可能会被重构为如下内容:

double number1, number2, ans; // Identify variables as double to account for decimals.
number1 = Convert.ToDouble(num1.Text); // Convert the contents of the textBox into a double.
number2 = Convert.ToDouble(num2.Text); // 
ans = 0.0;
string symbol = modifier1.Text;

if (symbol == "/" && number2 == 0) { answer.Text = "Invalid input."; return;}
switch(symbol)
{
   case "+": ans = number1 + number2; break;
   case "-": ans = number1 - number2; break;
   case "*": ans = number1 * number2; break;
   case "/": ans = number1 / number2; break;
   default : answer.Text = "Invalid sign."; return;
}                

answer.Text = ans.ToString("n"); // Change label value to a number.