Question

我刚刚写了我的第一个C＃程序。

这是一段解决二次方程的简单代码。

它完美地适用于某些功能（例如-6x2-6x + 12），而对于其他功能（4x2-20x + 25），它表现出我怀疑是舍入错误。

我对C＃完全陌生，我看不出问题;有人能帮我调试这段代码吗？

namespace ConsoleApplication {
    class Program {
        static int ObtainInput(string prompt, bool canBeZero) {
            double a = ObtainInput("A? ", false);
            double b = ObtainInput("B? ", true);
            double c = ObtainInput("C? ", true);
            double d, x1, x2;

            while (true) {
                Console.Write(prompt);
                string input = Console.ReadLine();
                int result;
                bool success = int.TryParse(input, out result);
                if (success && (canBeZero || result != 0))
                    return result;
                Console.WriteLine("Invalid input!");
            }

            // Calculating a discriminant
            d = b * b - 4 * a * c;

            if (d == 0) {
                x1 = -b / (2 * a);
                Console.WriteLine("The only solution is x={0}.", x1);
                Console.ReadLine();
            }

            // If d < 0, no real solutions exist
            else if (d < 0) { 
                Console.WriteLine("There are no real solutions");
                Console.ReadLine();
            }

            // If d > 0, there are two real solutions 
            else {
                x1 = (-b - Math.Sqrt(d)) / (2 * a);
                x2 = (-b + Math.Sqrt(d)) / (2 * a);
                Console.WriteLine("x1={0} and x2={1}.", x1, x2);
                Console.ReadLine();
            }
        }
    }
}

Answer 1

我刚刚写了我的第一个C＃程序。

真棒。现在是不养成坏习惯的好时机：

entA: Console.Write("a?");   
try { a = Convert.ToInt32(Console.ReadLine()); }
catch 
{ /*If a=0, the equation isn't quadratic*/
  Console.WriteLine("Invalid input"); 
  goto entA;             
}

问题比比皆是。首先，使用int.TryParse，而不是试图捕捉可能失败的东西。

其次，评论与代码的动作不匹配。代码确定结果是否为整数;评论说它检查零。

第三，当你试图表示的是一个循环时，不要使用goto。

第四，看看所有重复的代码！你有相同的代码重复三次，只有很小的变化。

让自己成为帮手方法：

 static int ObtainInput(string prompt, bool canBeZero)
 {
     while(true) // loop forever!
     {
         Console.Write(prompt);
         string input = Console.ReadLine();
         int result;
         bool success = int.TryParse(input, out result);
         if (success && (canBeZero || result != 0))
             return result;
         Console.WriteLine("Invalid input!");
     }
 }

现在你的主线是：

int a = ObtainInput("A? ", false);
int b = ObtainInput("B? ", true);
int c = ObtainInput("C? ", true);

你的错误就在这里：

x1 = x2 = -b / (2 * a);

您在整数中执行算术，然后然后转换为双精度。也就是说，你进行除法，舍入到最接近的整数，然后转换为double。从一开始就做双打（或者，不太可能，小数）。它应该是：

double a = ObtainInput("A? ", false);
double b = ObtainInput("B? ", true);
double c = ObtainInput("C? ", true);

也就是说，a，b和c不应该是整数。

Answer 2

分配给x1和x2时，你正在进行整数除法; （您可以将2更改为2.0以将其更改为双重除法并获得双重结果）

将a，b，c和d值更改为双倍也可能有意义，这也会超出问题，并允许人们为系数输入非int值。

Answer 3

int a，b，c; int d;

首先，尝试使用double而不是int，因为1/3 = 0使用整数。

遇到C＃二次方程求解器问题

3 个答案: