Сокращения кода

@Denis199 · Регистрация: 26.09.2016

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

доброе утро уважаемые программисты вопрос такой можно ли сократить этот код

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
   private void button_continue_Click(object sender, EventArgs e)
        {
            if (label_location_id.Text == "1")
            {
                intro_lection1 form1 = new intro_lection1();
                form1.Show();  
            }
            
            if (label_location_id.Text == "2")
            {
                Lection1 form2 = new Lection1();
                form2.Show();
            }
 
            if (label_location_id.Text == "3")
            {
                Lection2 form3 = new Lection2();
                form3.Show();
            }
            if (label_location_id.Text == "4")
            {
                Lection3 form4 = new Lection3();
                form4.Show();
            }
 
            if (label_location_id.Text == "5")
            {
                Lection4 form5 = new Lection4();
                form5.Show();
            }
            if (label_location_id.Text == "6")
            {
                Lection5 form6 = new Lection5();
                form6.Show();
            }
            if (label_location_id.Text == "7")
            {
                Lection6 form7 = new Lection6();
                form7.Show();
            }
            if (label_location_id.Text == "8")
            {
                Lection7 form8 = new Lection7();
                form8.Show();
            }
            if (label_location_id.Text == "9")
            {
                Lection8 form9 = new Lection8();
                form9.Show();
            }
            if (label_location_id.Text == "10")
            {
                intro_lection2 form10 = new intro_lection2();
                form10.Show();
            }
            if (label_location_id.Text == "11")
            {
                Lection2_1 form11 = new Lection2_1();
                form11.Show();
            }
            if (label_location_id.Text == "12")
            {
                Lection2_2 form12 = new Lection2_2();
                form12.Show();
            }
            if (label_location_id.Text == "13")
            {
                Lection2_3 form13 = new Lection2_3();
                form13.Show();
            }
            if (label_location_id.Text == "14")
            {
                Lection2_4 form14 = new Lection2_4();
                form14.Show();
            }
            if (label_location_id.Text == "15")
            {
                Lection2_5 form15 = new Lection2_5();
                form15.Show();
            }
            if (label_location_id.Text == "16")
            {
                intro_lection3 form16 = new intro_lection3();
                form16.Show();
            }
            if (label_location_id.Text == "17")
            {
                Lection3_1 form17 = new Lection3_1();
                form17.Show();
            }
            if (label_location_id.Text == "18")
            {
                Lection3_2 form18 = new Lection3_2();
                form18.Show();
            }
            if (label_location_id.Text == "19")
            {
                Lection3_3 form19 = new Lection3_3();
                form19.Show();
            }
            if (label_location_id.Text == "20")
            {
                Lection3_4 form20 = new Lection3_4();
                form20.Show();
            }
            if (label_location_id.Text == "21")
            {
                Lection3_5 form21 = new Lection3_5();
                form21.Show();
            }
            if (label_location_id.Text == "22")
            {
                Lection3_6 form22 = new Lection3_6();
                form22.Show();
            }
        }

а то работает то работает просто визуально плохо смотрется

@Aferuga · 19.12.2017, 09:31

Да можно, если все формы сократить до одной(Если конечно есть возможность, а судя по названиям она скорее всего есть).
И тогда вызов примет такой вид:

Кликните здесь для просмотра всего текста

C#
1
2
3
4
5
6
   private void button_continue_Click(object sender, EventArgs e)
        {
                Lection form1 = new Lection();
                form1.id = int.Parse(label_location_id.Text);// ну или как-то так
                form1.Show();  
}

@Agapov_stas · 19.12.2017, 13:07

Создайте словарь с перечнем форм и их индексов.

Кликните здесь для просмотра всего текста

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
private Dictionary<int, Form> listForms = new Dictionary<int, Form>()
        {
            { 1, new intro_lection1() },
            { 2, new Lection1() },
            { 3, new Lection2() },
            { 4, new Lection3() },
            { 5, new Lection4() },
            { 6, new Lection5() },
            { 7, new Lection6() },
            { 8, new Lection7() },
            { 9, new Lection8() },
            { 10, new intro_lection2() },
            { 11, new Lection2_1() },
            { 12, new Lection2_2() },
            { 13, new Lection2_3() },
            { 14, new Lection2_4() },
            { 15, new Lection2_5() },
            { 16, new intro_lection3() },
            { 17, new Lection3_1() },
            { 18, new Lection3_2() },
            { 19, new Lection3_3() },
            { 20, new Lection3_4() },
            { 21, new Lection3_5() },
            { 22, new Lection3_6() }
        };

Затем вызывайте:

C#
1
2
3
private void button_continue_Click(object sender, EventArgs e) {
            listForms[Convert.ToInt32(label_location_id.Text)].Show();
}

@OwenGlendower · 19.12.2017, 13:48

Agapov_stas, лучше спрятать создание формы в делегат чтобы а) не создавать все формы сразу (возможно не все они нужны) и б) чтобы получать новую форму, а не пытаться работать с уже закрытой и возможно Dispose()-ой

Кликните здесь для просмотра всего текста

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
private Dictionary<int, Func<Form>> listForms = new Dictionary<int, Func<Form>>()
        {
            {  1, () => new intro_lection1() },
            {  2, () => new Lection1() },
            {  3, () => new Lection2() },
            {  4, () => new Lection3() },
            {  5, () => new Lection4() },
            {  6, () => new Lection5() },
            {  7, () => new Lection6() },
            {  8, () => new Lection7() },
            {  9, () => new Lection8() },
            { 10, () => new intro_lection2() },
            { 11, () => new Lection2_1() },
            { 12, () => new Lection2_2() },
            { 13, () => new Lection2_3() },
            { 14, () => new Lection2_4() },
            { 15, () => new Lection2_5() },
            { 16, () => new intro_lection3() },
            { 17, () => new Lection3_1() },
            { 18, () => new Lection3_2() },
            { 19, () => new Lection3_3() },
            { 20, () => new Lection3_4() },
            { 21, () => new Lection3_5() },
            { 22, () => new Lection3_6() }
        };
 
private void button_continue_Click(object sender, EventArgs e)
{
      var form = listForms[Convert.ToInt32(label_location_id.Text)]();
      form.Show();
}

@nedel · 19.12.2017, 15:17

словари, дегаты... ТС начинающий - ему надо что-то попроще

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
private void button_continue_Click(object sender, EventArgs e)
{
    Form form;
    switch (label_location_id.Text)
    {
    case "1":
        form = new intro_lection1();
        break;
    case "2":
        form = new Lection1();
        break;
    // ........ и так далее   
    }
    if (form != null)
       form.Show();
}

@Agapov_stas · 19.12.2017, 16:54

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

словари, дегаты... ТС начинающий - ему надо что-то попроще

Не по теме:

Хреновый из Вас помощник, Сэррр (с)

И чем Вы проще ему сделали?
Как раз держать один список/словарь намного понятнее и проще в использовании(нет нужды в дополнительном коде, достаточно добавить новую форму в словарь и все), чем то, что Вы предложили(да и какой смысл изменять if на swith, если суть кода осталась та же?).
Ведь ТС прямо сказал: "сокращение кода". Вы же предлагаете ему те же яйца...только вид...(тоже, кстати, не очень).

@Denis199 · 20.12.2017, 07:39 **[ТС]**

Спасибо большое я очень рад )) за хорошую помощь и умную идею

делаю по аналогии

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
 if (label_location_id.Text == "0")
            {
                textBox_location.Text = "У вас еще нету точек сохранения. нажмите кнопку НАЧАТЬ НОВЫЙ КУРС";
                button_continue.ForeColor = Color.Silver; 
            }
            if (label_location_id.Text == "1")
            {
                textBox_location.Text = "Основные понятия о компьютерных сетях (Раздел №1)";
            }
            if (label_location_id.Text == "2")
            {
                textBox_location.Text = "Классификация информационно вычислительных сетей (ИВС) (Тема №1.1)";
            }
            if (label_location_id.Text == "3")
            {
                textBox_location.Text = "Программные и аппаратные средства ИВС(Тема №1.2)";
            }
            if (label_location_id.Text == "4")
            {
                textBox_location.Text = "Сети одноранговые и «клиент/сервер» (Тема №1.3)";
            }
            if (label_location_id.Text == "5")
            {
                textBox_location.Text = "Способы коммутации (Тема №1.4)";
            }
            if (label_location_id.Text == "6")
            {
                textBox_location.Text = "Топология сетей (Тема №1.5)";
            }
            if (label_location_id.Text == "7")
            {
                textBox_location.Text = "Многоуровневые ИВС и эталонная модель взаимосвязи открытых систем (Тема №1.6)";
            }
            if (label_location_id.Text == "8")
            {
                textBox_location.Text = "Стандартные стеки коммуникационных протоколов (Тема №1.7)";
            }
            if (label_location_id.Text == "9")
            {
                textBox_location.Text = "Сетевые компоненты (Тема №1.8)";
            }
            if (label_location_id.Text == "10")
            {
                textBox_location.Text = "Аппаратные компоненты локальных компьютерных сетей (Раздел №2)";
            }
            if (label_location_id.Text == "11")
            {
                textBox_location.Text = "Структурированная кабельная система (Тема №2.1)";
            }
            if (label_location_id.Text == "12")
            {
                textBox_location.Text = "Сетевые адаптеры (Тема №2.2)";
            }
            if (label_location_id.Text == "13")
            {
                textBox_location.Text = "Концентраторы (Тема №2.3)";
            }
            if (label_location_id.Text == "14")
            {
                textBox_location.Text = "Мосты (Тема №2.4)";
            }
            if (label_location_id.Text == "15")
            {
                textBox_location.Text = "Коммутаторы (Тема №2.5)";
            }
            if (label_location_id.Text == "16")
            {
                textBox_location.Text = "Основные технические характеристики и качество компьютерных сетей и телекоммуникационных каналов (Раздел №3)";
            }
            if (label_location_id.Text == "17")
            {
                textBox_location.Text = "Показатели качества информационно-вычислительных сетей (Тема №3.1)";
            }
            if (label_location_id.Text == "18")
            {
                textBox_location.Text = "Типы каналов связи (Тема №3.2)";
            }
            if (label_location_id.Text == "19")
            {
                textBox_location.Text = "Типы цифровых каналов (Тема №3.3)";
            }
            if (label_location_id.Text == "20")
            {
                textBox_location.Text = "Цифровое кодирование дискретной информации (Тема №3.4)";
            }
            if (label_location_id.Text == "21")
            {
                textBox_location.Text = "Беспроводные каналы связи (Тема №3.5)";
            }
            if (label_location_id.Text == "22")
            {
                textBox_location.Text = "Системы мобильной связи (Тема №3.6)";
            }

Добавлено через 15 минут
делаю так идут ошибки

C#
1
2
3
4
5
private Dictionary<int, Func<Form>> listForms2 = new Dictionary<int, Func<Form>>()
        {
            { 1, () = "У вас еще нету точек сохранения. нажмите кнопку НАЧАТЬ НОВЫЙ КУРС" },
            { 2, () = "Основные понятия о компьютерных сетях (Раздел №1)" },
        };

@Agapov_stas · 20.12.2017, 07:51

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

делаю так идут ошибки

Не, ну так делегаты необходимы были при создании новой формы. Т.е., имелось ввиду, при вызове значения из словаря, создавался новый экземпляр класса формы.

В случае же с текстом..

C#
1
2
3
4
5
6
private Dictionary<int, string> list = new Dictionary<int, string>()
        {
            { 1, "У вас еще нету точек сохранения. нажмите кнопку НАЧАТЬ НОВЫЙ КУРС" },
            { 2, "Основные понятия о компьютерных сетях (Раздел №1)" }
//...и т.д.
        };

Новые блоги и статьи Все статьи Все блоги /
Установка Android SDK, NDK, JDK, CMake и т.д. 8Observer8 25.01.2026 Содержание блога Перейдите по ссылке: https:/ / developer. android. com/ studio и в самом низу страницы кликните по архиву "commandlinetools-win-xxxxxx_latest. zip" Извлеките архив и вы увидите. . .	Вывод текста со шрифтом TTF на Android с помощью библиотеки SDL3_ttf 8Observer8 25.01.2026 Содержание блога Если у вас не установлены Android SDK, NDK, JDK, и т. д. то сделайте это по следующей инструкции: Установка Android SDK, NDK, JDK, CMake и т. д. Сборка примера Скачайте. . .	Использование SDL3-callbacks вместо функции main() на Android, Desktop и WebAssembly 8Observer8 24.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .	моя боль iceja 24.01.2026 Выложила интерполяцию кубическими сплайнами www. iceja. net REST сервисы временно не работают, только через Web. Написала за 56 рабочих часов этот сайт с нуля. При помощи perplexity. ai PRO , при. . .
Модель сукцессии микоризы anaschu 24.01.2026 Решили писать научную статью с неким РОманом	http://iceja.net/ математические сервисы iceja 20.01.2026 Обновила свой сайт http:/ / iceja. net/ , приделала Fast Fourier Transform экстраполяцию сигналов. Однако предсказывает далеко не каждый сигнал (см ограничения http:/ / iceja. net/ fourier/ docs ). Также. . .	http://iceja.net/ сервер решения полиномов iceja 18.01.2026 Выкатила http:/ / iceja. net/ сервер решения полиномов (находит действительные корни полиномов методом Штурма). На сайте документация по API, но скажу прямо VPS слабенький и 200 000 полиномов. . .	Расчёт переходных процессов в цепи постоянного тока igorrr37 16.01.2026 / * Дана цепь(не выше 3-го порядка) постоянного тока с элементами R, L, C, k(ключ), U, E, J. Программа находит переходные токи и напряжения на элементах схемы классическим методом(1 и 2 з-ны. . .

Сокращения кода

Решение