Большой код - проблема?

@odnaglaziw · Регистрация: 11.10.2023

Студворк — интернет-сервис помощи студентам

Здравствуйте! Всегда так получается что у моих одногруппников получаются коды меньше моего, но мне хочется верить что это не от какой-то проблемы и, надеюсь, в будущем это не помешает работать. Но вдруг в этом есть проблема, вот и спрашиваю

Нам задали написать программу для решения квадратных неравенств. Сначала пойдём мой(длинный) код, а потом код друга(короткий).

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
using static System.Console;
using static System.Convert;
using static System.Math;
//ny i ny
WriteLine("Сколько неравенств хотите просчитать?");
int n  = ToInt32(ReadLine());
double[] answer = new double[n];
double a, b, c;
double x=0, x1=0, x2=0;
bool zero = false, positive = false;
double D(double a, double b, double c)
{
    double D = Pow(b, 2) - (4 * a * c);
    return D;
}
for (double i = 0; i < n; i++)
{
    WriteLine($"Введите данные для {i+1} случая:");
    Write("a="); a = ToDouble(ReadLine());
    Write("b="); b = ToDouble(ReadLine());
    Write("c="); c = ToDouble(ReadLine());
    if (D(a, b, c) < 0)
    {
        WriteLine("Корней нет!");
        continue;
    }
    else if (D(a,b,c) == 0)
    {
        x = -1*b/(2 * a);
        zero = true;
        WriteLine("0");
    }
    else
    {
         x1 = (b * (-1)+ Sqrt(Abs(D(a,b,c)))) / (2 * a);
         x2 = (b * (-1) - Sqrt(Abs(D(a, b, c)))) / (2 * a);
         if (x1 > x2)
        {
            (x1, x2) = (x2, x1);
        }
         if (x1== -0) x1 = 0;
         if (x2== -0) x2 = 0;
        positive = true;
        WriteLine(">0");
    }
    WriteLine("Выберите знак:");
    WriteLine("1.>\n2.>=\n3.<\n4.<=");
    string? ans = ReadLine();
    switch (ans)
    {
        case ">" or "1":
            {
                if (zero)
                {
                    if (a > 0)
                    {
                        WriteLine($"Ответом являются все числа, кроме {x}");
                    }
                    else
                    {
                        WriteLine("Решения нет.");
                    }
                } 
                else if (positive)
                {
                    if (a > 0)
                    {
                        WriteLine($"(-inf;{x1})({x2};+inf)");
                    }
                    else
                    {
                        WriteLine($"({x1};{x2})");
                    }
                }
            }break;
        case ">=" or "2":
            {
                if (zero)
                {
                    if (a > 0)
                    {
                        WriteLine($"(-inf;+inf)");
                    }
                    else
                    {
                        WriteLine($"Решения нет.");
                    }
                }
                else if (positive)
                {
                    if (a > 0)
                    {
                        WriteLine($"(-inf;{x1}][{x2};+inf)");
                    }
                    else
                    {
                        WriteLine($"[{x1};{x2}]");
                    }
                }
            }
            break;
        case "<" or "3":
            {
                if (zero)
                {
                    if (a > 0)
                    {
                        WriteLine($"Решения нет.");
                    }
                    else
                    {
                        WriteLine($"Все числа, кроме {x}");
                    }
                }
                else if (positive)
                {
                    if (a > 0)
                    {
                        WriteLine($"({x1};{x2})");
                    }
                    else
                    {
                        WriteLine($"(-inf;{x1})({x2};+inf)");
                    }
                }
            }
            break;
        case "<=" or "4":
            {
                if (zero)
                {
                    if (a > 0)
                    {
                        WriteLine($"Ответом является {x}");
                    }
                    else
                    {
                        WriteLine($"(-inf;{x1}][{x2};+inf)");
                    }
                }
                else if (positive)
                {
                    if (a > 0)
                    {
                        WriteLine($"[ {x1} ; {x2} ]");
                    }
                    else
                    {
                        WriteLine($"(-inf;{x1}][{x2};+inf)");
                    }
                }
            }
            break;
    }
}

Друга:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
double a, b, c, d, x1 = 0, x2 = 0; Console.WriteLine("Введите кол-во неравенств");
int x = Convert.ToInt32(Console.ReadLine()); if (x > 0)
{
    string[] n = new string[x];
    for (int i = 0; i < n.Length; i++)
    {
        Console.WriteLine($"Введите (a b c) неизвестное уравнения {i + 1} | ax^2+bx+c=0"); string[] k = Console.ReadLine().Split(' ');
        a = Convert.ToDouble(k[0]); b = Convert.ToDouble(k[1]); c = Convert.ToDouble(k[2]); Console.WriteLine("Введите знак неравенства (>|<|>=|<=)");
        n[i] = Console.ReadLine(); d = Math.Pow(b, 2) - (4 * a * c);
        if (b == 0)
        {
            x1 = Math.Sqrt(Math.Abs(c) / Math.Abs(a)); x2 = -x1;
            Console.WriteLine($"X1 = {x1}, X2 = {x2}");
        }
        else if (c == 0)
        {
            x1 = 0; x2 = ((-b) / a);
            Console.WriteLine($"X1 = {x1}, X2 = {x2}");
        }
        else if (d > 0)
        {
            Console.WriteLine($"Дискриминант - {d}"); x1 = (-b + Math.Sqrt(d)) / (2 * a);
            x2 = (-b - Math.Sqrt(d)) / (2 * a); Console.WriteLine($"X1 = {x1}, X2 = {x2}");
        }
        else if (d == 0)
        {
            x1 = -b / (2 * a); x2 = x1;
            Console.WriteLine($"X1 = {x1}");
        }
        else
        {
            Console.WriteLine($"Дискриминант - {d}"); Console.WriteLine("Уравнение не имеет корней");
            n[i] = "";
        }
        switch (n[i])
        {
            case ">":
                if (a > 0)
                {
                    Console.WriteLine($"Числовой промежуток - (-inf, {x1})with({x2}, +inf)");
                }
                else
                {
                    Console.WriteLine($"Числовой промежуток - ({x1}, {x2})");
                }
                break;
            case "<":
                if (a > 0)
                {
                    Console.WriteLine($"Числовой промежуток - ({x1}, {x2})");
                }
                else
                {
                    Console.WriteLine($"Числовой промежуток - (-inf, {x1})with({x2}, +inf)");
                }
                break;
            case ">=":
                if (a > 0)
                {
                    Console.WriteLine($"Числовой промежуток - (-inf, {x1}]with[{x2}, +inf)");
                }
                else
                {
                    Console.WriteLine($"Числовой промежуток - [{x1}, {x2}]");
                }
                break;
            case "<=":
                if (a > 0)
                {
                    Console.WriteLine($"Числовой промежуток - [{x1}, {x2}]");
                }
                else
                {
                    Console.WriteLine($"Числовой промежуток - (-inf, {x1}]with[{x2}, +inf)");
                }
                break;
        }
    }
}
else Console.WriteLine("Введено неверно");

Uswer · 25.10.2023, 08:46

Кроме размера кода есть ещё его читабельность, второй вариант в этом плане чуть хуже. Ваш друг мог бы и ещё один способ "упаковки" кода применить вот такой:
Вместо такой записи

C#
1
2
3
4
5
6
7
8
if (a > 0)
{
    WriteLine($"Ответом является {x}");
}
    else
{
    WriteLine($"(-inf;{x1}][{x2};+inf)");
}

Вполне мог записать так

C#
1
2
if (a > 0) { WriteLine($"Ответом является {x}"); }
else { WriteLine($"(-inf;{x1}][{x2};+inf)"); }

@kotelok · 25.10.2023, 09:23

Это всегда баланс - читаемость (+отлаживаемость) или краткость записи.

Ну т.е. вот это, например:

C#
1
2
3
4
5
6
7
8
if (a > 0)
{
    Console.WriteLine($"Числовой промежуток - (-inf, {x1}]with[{x2}, +inf)");
}
else
{
    Console.WriteLine($"Числовой промежуток - [{x1}, {x2}]");
}

Можно и в таком виде оформить:

C#
1
Console.WriteLine($"Числовой промежуток - {(a > 0 ? $"(-inf, {x1}]with[{x2}, +inf)" : $"[{x1}, {x2}]")}");

И в целом, конкретно в этом случае, это даже почти не ухудшит читаемость кода, но вот пройтись в отладке по каждой отдельной ветке уже не получится. Хотя, мне исходный вариант всё равно больше нравится.

@iNNOKENTIY21 · 25.10.2023, 09:31

Сообщение от odnaglaziw

Большой код - проблема?

Судя по всему, да. Вы уже комплексуете и экономите на строках, пустых строках, именах… а это не место экономии.
При таком форматировании кода, трудно замечать логику.
например, у Вас 2 раз с теми же параметрами, вызов метода D (при этом написаны по разному, с пробелами и без, стоит придерживаться одного стиля):

C#
1
2
3
4
5
6
if (D(a, b, c) < 0)
    {
        WriteLine("Корней нет!");
        continue;
    }
    else if (D(a,b,c) == 0)

Почему не вызывать один раз присваивая переменной, а далее с переменной работать. Строки экономим == Проблема.

@Wolfdp · 25.10.2023, 17:46

1. Ваш друг хитрожопит тем, что пишет в одной строке несколько операций. За такое на проектах бьют по рукам.
2. Не соблюдена единая стилистика. Например теже скобки: если в if есть, то в else тоже пишем. Это позволяет нормально видеть блоки ответственности.
3. хромают наименования (у вас обоих)
4. нах массив answer у вас и n у вашего друга?
5. если убрать явно лишний массив, то вместо {i+1} можно использовать for (int i = 1; i <= count; i++)
6. ни у кого нет проверки корректности ввода. Осуждаю.
7. if (D(a,b,c) == 0) это приводит к тому что на каждую(!) проверку вызывается метод вычисления. Ваш друг посчитал в переменую -- это правильнее.
8. Ваша логика с zero и positive по сути и "раздула" код
9. В целом switch можно упаковать в тернарный оператор (см. пример ниже), но с точки зрения дебага такой код не всегда лучше.

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
double a, b, c, d, x1, x2;
string expressionSing, result;
bool correctInput;
 
Console.WriteLine("Введите кол-во неравенств");
var count = Convert.ToInt32(Console.ReadLine());
 
if (count > 0)
{
    for (int i = 1; i <= count; i++)
    {
        Console.WriteLine($"Введите (a b c) неизвестное уравнения {i} | ax^2+bx+c=0");
        var input = Console.ReadLine()?.Split(' ');
        correctInput = input?.Length == 3;
        correctInput &= double.TryParse(input![0], out a );
        correctInput &= double.TryParse(input![1], out b );
        correctInput &= double.TryParse(input![2], out c );
 
        if (!correctInput)
        {
            Console.WriteLine("введеные значения не коректны");
            continue;
        }
 
        Console.WriteLine("Введите знак неравенства ( > , < , >= , <= )");
        expressionSing = Console.ReadLine()!;
 
        if (b == 0)
        {
            x1 = Math.Sqrt(Math.Abs(c) / Math.Abs(a));
            x2 = -x1;
            Console.WriteLine($"X1 = {x1}, X2 = {x2}");
        }
        else if (c == 0)
        {
            x1 = 0;
            x2 = (-b) / a;
            Console.WriteLine($"X1 = {x1}, X2 = {x2}");
        }
        else 
        {
            d = Math.Pow(b, 2) - (4 * a * c);
 
            if (d > 0)
            {
                Console.WriteLine($"Дискриминант - {d}");
                x1 = (-b + Math.Sqrt(d)) / (2 * a);
                x2 = (-b - Math.Sqrt(d)) / (2 * a);
                Console.WriteLine($"X1 = {x1}, X2 = {x2}");
            }
            else if (d == 0)
            {
                x1 = -b / (2 * a);
                x2 = x1;
                Console.WriteLine($"X1 = {x1}");
            }
            else
            {
                Console.WriteLine($"Дискриминант - {d}");
                Console.WriteLine("Уравнение не имеет корней");
                continue;
            }
        }
 
        result = a > 0
            ? expressionSing switch
            {
                ">" => $"Числовой промежуток - (-inf, {x1})with({x2}, +inf)",
                "<" => $"Числовой промежуток - ({x1}, {x2})",
                ">=" => $"Числовой промежуток - (-inf, {x1}]with[{x2}, +inf)",
                "<=" => $"Числовой промежуток - [{x1}, {x2}]",
                _ => "Неизвестный знак"
            }
            : expressionSing switch
            {
                ">" => $"Числовой промежуток - ({x1}, {x2})",
                "<" => $"Числовой промежуток - (-inf, {x1})with({x2}, +inf)",
                ">=" => $"Числовой промежуток - [{x1}, {x2}]",
                "<=" => $"Числовой промежуток - (-inf, {x1}]with[{x2}, +inf)",
                _ => "Неизвестный знак"
            };
 
        Console.WriteLine(result);
    }
}
else
{
    Console.WriteLine("Введено неверно");
}

Добавлено через 2 минуты
p.s. код не проверял.

Добавлено через 12 минут
А, ну и отвечая на общий вопрос темы -- проблема у обоих в нехватке опыта. Напишете 100500 вдумчивых строчек -- появится "чуйка" как делать правильно. Больше кода не всегда признак "хуже", в программировании никто не меряет количеством строк и символов. Порой специально "раздувают", чтобы в дальнейшем программа была более гибкая для модификаций (но до этого момента вам ещё учить и учить).

Пока стоит обращать внимание на такой момент -- насколько много у меня повторяющегося кода? Как правило на начальных этапах это основная ошибка.

Новые блоги и статьи Все статьи Все блоги /
[Owen Logic] Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ФедосеевПавел 14.03.2026 Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ВВЕДЕНИЕ Выполняя задание на управление насосной группой заполнения резервуара,. . .	делаю науч статью по влиянию грибов на сукцессию anaschu 13.03.2026 прикрепляю статью	SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .
Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .	Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .	Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .