Кодревью c# консольного аппа?

Question

[iddmitriykozyrev]

Оцените небольшое консольное приложение на C# по нахождению дискриминанта и корней.

Помогите,пожалуйста, с обработкой исключения для переменных a,b,c через try-catch.

критика принимается,но не грубая. Все-таки программирую второй день после школьного pascal

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace uravnenie1
{
    class Program
    {
        static void Main(string[] args)
        {
            while (true)
            {
                Console.WriteLine("Введите квадратное уравнение:");
                Console.WriteLine("Введите a,b,c:");
                
                double a = Convert.ToDouble(Console.ReadLine());
                double b = Convert.ToDouble(Console.ReadLine());
                double c = Convert.ToDouble(Console.ReadLine());

                double D = (double)(Math.Pow(b, 2)) - 4 * a * c;
                Console.WriteLine("квадратное выражение : " + a + "x^2+" + b + "x+" + c);
                Console.WriteLine("D=" + D);

                double x1;
                if (D > 0)
                {
                    double x2;
                    x1 = ((-b + (Math.Sqrt(D))) / (2 * a));
                    x2 = ((-b - (Math.Sqrt(D))) / (2 * a));
                    Console.WriteLine("X1=" + x1);
                    Console.WriteLine("X2=" + x2);
                }
                else if (D == 0)
                {
                    x1 = -b / (2 * a);
                    Console.WriteLine("X=" + x1);
                }
                else if (D < 0)
                {
                    Console.WriteLine("Действительных корней нет");
                }
                string exit = Console.ReadLine();
                if (exit == "exit")
                    break;
            }
        }
    }
}

Comments

[netrox]

Форматируйте код

Answer 1

[OwDafuq]

Ошибки, которые бросаются в глаза (некоторые из них притянуты за уши(!)):
1. Сразу конвертируем данные из строки в числа:
double a = Convert.ToDouble(Console.ReadLine());
Сломает вашу программу полностью, если написать в консоль символ, либо вообще ничего не писать.
Используйте double.TryParse
2. Используйте интерполяцию строк:

Console.WriteLine($"квадратное выражение: {a} + x^2 + {b} + x + {c}");

3. Лишние скобки:
x1 = ((-b + (Math.Sqrt(D))) / (2 * a));
Для чего?
4. Лишние переменные для чего? Посчитали и выводим на консоль результат, ЕСЛИ дальше результат не нужен для вычислений:

Console.WriteLine($"X1 = {(-b + (Math.Sqrt(D))) / (2 * a)}");
Console.WriteLine($"X2 = {(-b - (Math.Sqrt(D))) / (2 * a)}");

5. Для выхода не обязательно объявлять переменную:

if (Console.ReadLine() == "exit")
	break;

Comments

[stratosmi]

4 - норм для читабельности. Ты предложил вариант хуже.

[OwDafuq]

stratosmi, читаем первую строку, текст в скобках
И имхо, мне лучше когда написано моим вариантом

[stratosmi]

LiptonOlolo,

читаем первую строку, текст в скобках
И имхо, мне лучше когда написано моим вариантом

Скобки - да, у него перебор.
Но пихать все внутрь WriteLine....
Зачем?

[OwDafuq]

stratosmi, Для чего же тогда, по вашему мнению, делать так, как сделал он?

[Melz]

LiptonOlolo, лучше делать как он, а вообще лучше:
Console.WriteLine("X1 = {0}", dblSmth);
Когда вы пишете все в одном WriteLine вы нарушаете SOLID: форматирование и вычисление идет в одной строке.
В качестве примера могу сказать что там тип double и он скорее всего будет форматировать количество точек после запятой чтобы не видеть 4.99999999999... Плюс скорее всего он захочет увидеть результат вырадения как double перед форматированиемс строку.

[stratosmi]

Melz,

Когда вы пишете все в одном WriteLine вы нарушаете SOLID

Да пофиг на SOLID.
Это просто хуже читается, когда все в одной строке.

Answer 2

[Koshelev]

Помогите,пожалуйста, с обработкой исключения для переменных a,b,c через try-catch.

Обработка исключений - дорогое удовольствие. В данном случае их не нужно использовать.
Лучше, как написал LiptonOlolo, используй double.TryParse.

Comments

[stratosmi]

Обработка исключений - дорогое удовольствие. В данном случае их не нужно использовать.

Ты где-то что то краем уха слышал, неправильно понял и теперь живешь без исключений вообще?
Вот как раз в данном случае цена исключения значения не имеет.

[Koshelev]

stratosmi, тебе кто-то сказал, что я не использую исключения?
В данном случае имелось в виду не только время выполнения, но и объем и читаемость кода.