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

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不应该是整数.