Как мне оптимизировать мой код

@shur1kk · Регистрация: 25.10.2012

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

в общем, есть компонент Combobox, с помощью его я выбираю тот параметр по которому хочу сортировать свои значения, параметров много, и в итоге много одинакового текста, а думаю что все это можно изменить

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
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
//-----------------СОРТИРОВАТЬ--------------------------------------------------------
void __fastcall TForm2::Button3Click(TObject *Sender)
{
if (ComboBox1->ItemIndex == 0)
{
        TZap st, *mas_Z;
        Fz = fopen(File_Zap.c_str(),"rb");
        int D_f = fileno(Fz);               // Находим дескриптор файла
        int len = filelength(D_f);          // Находим размер файла
        int i, j, kol;
        kol = len/size;                 //Количество записей в файле
        mas_Z = new TZap[kol];
// Считываем записи из файла в динамический массив
        for (i=0; i < kol; i++)    
            fread((mas_Z+i), size, 1, Fz);
        fclose(Fz);
 
          Memo1->Lines->Add("----Сортированные сведения по Uобр max----");
          Memo1->Lines->Add("| Тип прибора | Uобр max, B | Iпр max, mA | Iпрг, A | fраб, kHz | Uпр, B | Iпр, mA | Iобр, mkA |");
        for (i=0; i < kol-1; i++)
                for (j=i+1; j < kol; j++)
                        if (mas_Z[i].uobr > mas_Z[j].uobr) {
                                    st = mas_Z[i];
                                    mas_Z[i] = mas_Z[j];
                                    mas_Z[j] = st;
                        }
            for (i=0; i<kol; i++)
            Out(mas_Z[i], Memo1);
    delete []mas_Z;
 
}
if (ComboBox1->ItemIndex == 1)
{
        TZap st, *mas_Z;
        Fz = fopen(File_Zap.c_str(),"rb");
        int D_f = fileno(Fz);               // Находим дескриптор файла
        int len = filelength(D_f);          // Находим размер файла
        int i, j, kol;
        kol = len/size;                 //Количество записей в файле
        mas_Z = new TZap[kol];
// Считываем записи из файла в динамический массив
        for (i=0; i < kol; i++)
            fread((mas_Z+i), size, 1, Fz);
        fclose(Fz);
            Memo1->Lines->Add("----Сортированные сведения по Iпр max----");
            Memo1->Lines->Add("| Тип прибора | Uобр max, B | Iпр max, mA | Iпрг, A | fраб, kHz | Uпр, B | Iпр, mA | Iобр, mkA |");
        for (i=0; i < kol-1; i++)
                for (j=i+1; j < kol; j++)
                        if (mas_Z[i].iprmax > mas_Z[j].iprmax) {
                                    st = mas_Z[i];
                                    mas_Z[i] = mas_Z[j];
                                    mas_Z[j] = st;
                        }
            for (i=0; i<kol; i++)
            Out(mas_Z[i], Memo1);
    delete []mas_Z;
 
}
if (ComboBox1->ItemIndex == 2)
{
        TZap st, *mas_Z;
        Fz = fopen(File_Zap.c_str(),"rb");
        int D_f = fileno(Fz);               // Находим дескриптор файла
        int len = filelength(D_f);          // Находим размер файла
        int i, j, kol;
        kol = len/size;                 //Количество записей в файле
        mas_Z = new TZap[kol];
// Считываем записи из файла в динамический массив
        for (i=0; i < kol; i++)
            fread((mas_Z+i), size, 1, Fz);
        fclose(Fz);
          Memo1->Lines->Add("----Сортированные сведения по Iпрг----");
          Memo1->Lines->Add("| Тип прибора | Uобр max, B | Iпр max, mA | Iпрг, A | fраб, kHz | Uпр, B | Iпр, mA | Iобр, mkA |");
        for (i=0; i < kol-1; i++)
                for (j=i+1; j < kol; j++)
                        if (mas_Z[i].iprg > mas_Z[j].iprg) {
                                    st = mas_Z[i];
                                    mas_Z[i] = mas_Z[j];
                                    mas_Z[j] = st;
                        }
            for (i=0; i<kol; i++)
            Out(mas_Z[i], Memo1);
    delete []mas_Z;
 
}
if (ComboBox1->ItemIndex == 3)
{
        TZap st, *mas_Z;
        Fz = fopen(File_Zap.c_str(),"rb");
        int D_f = fileno(Fz);               // Находим дескриптор файла
        int len = filelength(D_f);          // Находим размер файла
        int i, j, kol;
        kol = len/size;                 //Количество записей в файле
        mas_Z = new TZap[kol];
// Считываем записи из файла в динамический массив
        for (i=0; i < kol; i++)
            fread((mas_Z+i), size, 1, Fz);
        fclose(Fz);
                 Memo1->Lines->Add("----Сортированные сведения по fраб----");
                 Memo1->Lines->Add("| Тип прибора | Uобр max, B | Iпр max, mA | Iпрг, A | fраб, kHz | Uпр, B | Iпр, mA | Iобр, mkA |");
        for (i=0; i < kol-1; i++)
                for (j=i+1; j < kol; j++)
                        if (mas_Z[i].frab > mas_Z[j].frab) {
                                    st = mas_Z[i];
                                    mas_Z[i] = mas_Z[j];
                                    mas_Z[j] = st;
                        }
            for (i=0; i<kol; i++)
            Out(mas_Z[i], Memo1);
    delete []mas_Z;
 
}
if (ComboBox1->ItemIndex == 4)
{
        TZap st, *mas_Z;
        Fz = fopen(File_Zap.c_str(),"rb");
        int D_f = fileno(Fz);               // Находим дескриптор файла
        int len = filelength(D_f);          // Находим размер файла
        int i, j, kol;
        kol = len/size;                 //Количество записей в файле
        mas_Z = new TZap[kol];
// Считываем записи из файла в динамический массив
        for (i=0; i < kol; i++)
            fread((mas_Z+i), size, 1, Fz);
        fclose(Fz);
            Memo1->Lines->Add("----Сортированные сведения по Uпр----");
                Memo1->Lines->Add("| Тип прибора | Uобр max, B | Iпр max, mA | Iпрг, A | fраб, kHz | Uпр, B | Iпр, mA | Iобр, mkA |");
        for (i=0; i < kol-1; i++)
                for (j=i+1; j < kol; j++)
                        if (mas_Z[i].upr > mas_Z[j].upr) {
                                    st = mas_Z[i];
                                    mas_Z[i] = mas_Z[j];
                                    mas_Z[j] = st;
                        }
            for (i=0; i<kol; i++)
            Out(mas_Z[i], Memo1);
    delete []mas_Z;
 
}
if (ComboBox1->ItemIndex == 5)
{
        TZap st, *mas_Z;
        Fz = fopen(File_Zap.c_str(),"rb");
        int D_f = fileno(Fz);               // Находим дескриптор файла
        int len = filelength(D_f);          // Находим размер файла
        int i, j, kol;
        kol = len/size;                 //Количество записей в файле
        mas_Z = new TZap[kol];
// Считываем записи из файла в динамический массив
        for (i=0; i < kol; i++)
            fread((mas_Z+i), size, 1, Fz);
        fclose(Fz);
         Memo1->Lines->Add("----Сортированные сведения по Iпр----");
         Memo1->Lines->Add("| Тип прибора | Uобр max, B | Iпр max, mA | Iпрг, A | fраб, kHz | Uпр, B | Iпр, mA | Iобр, mkA |");
        for (i=0; i < kol-1; i++)
                for (j=i+1; j < kol; j++)
                        if (mas_Z[i].ipr > mas_Z[j].ipr) {
                                    st = mas_Z[i];
                                    mas_Z[i] = mas_Z[j];
                                    mas_Z[j] = st;
                        }
            for (i=0; i<kol; i++)
            Out(mas_Z[i], Memo1);
    delete []mas_Z;
 
}
if (ComboBox1->ItemIndex == 6)
{
        TZap st, *mas_Z;
        Fz = fopen(File_Zap.c_str(),"rb");
        int D_f = fileno(Fz);               // Находим дескриптор файла
        int len = filelength(D_f);          // Находим размер файла
        int i, j, kol;
        kol = len/size;                 //Количество записей в файле
        mas_Z = new TZap[kol];
// Считываем записи из файла в динамический массив
        for (i=0; i < kol; i++)
            fread((mas_Z+i), size, 1, Fz);
        fclose(Fz);
            Memo1->Lines->Add("----Сортированные сведения по Iобр----");
            Memo1->Lines->Add("| Тип прибора | Uобр max, B | Iпр max, mA | Iпрг, A | fраб, kHz | Uпр, B | Iпр, mA | Iобр, mkA |");
        for (i=0; i < kol-1; i++)
                for (j=i+1; j < kol; j++)
                        if (mas_Z[i].iobr > mas_Z[j].iobr) {
                                    st = mas_Z[i];
                                    mas_Z[i] = mas_Z[j];
                                    mas_Z[j] = st;
                        }
            for (i=0; i<kol; i++)
            Out(mas_Z[i], Memo1);
    delete []mas_Z;
 
}
}

@Gus · 15.12.2012, 21:14

эм, а не проще было запилить шаблон, и не копировать один и тот же по функциональному смыслу слова код?

@shur1kk · 15.12.2012, 21:20 **[ТС]**

ну я как бы понимаю, что одно и тоже, так вот и спрашиваю, как это все сократить?

@SergeyKu · 16.12.2012, 01:46

C++
1
2
3
4
5
int cbii = ComboBox1->ItemIndex;
if(cbii == 0 || cbii == 1 || cbii == 2 || cbii == 3 || cbii == 4 || cbii == 5 || cbii == 6)
  {
    ...
  }

@shur1kk · 16.12.2012, 11:48 **[ТС]**

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

C++
1
2
3
4
5
int cbii = ComboBox1->ItemIndex;
if(cbii == 0 || cbii == 1 || cbii == 2 || cbii == 3 || cbii == 4 || cbii == 5 || cbii == 6)
  {
    ...
  }

ну особо ничего не поменялось же

@nick42 · 16.12.2012, 14:25

Трудно что-либо советовать, не зная структуры TZap. В общем случае, если применять индексацию, возможно стоит написать коротенькую функцию func, возвращающую значение величины из динамического массива по индексам__

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
void __fastcall TForm2::Button3Click(TObject *Sender)
{
    TZap st, *mas_Z;
    int dx = ComboBox1->ItemIndex ;
    String soob[] = { "Uîáð max","Iïð max","Iïðã","fðàá","Uïð","Iïð","Iîáð" };
 
    Fz = fopen(File_Zap.c_str(),"rb");
    int D_f = fileno(Fz);               // Íàõîäèì äåñêðèïòîð ôàéëà
    int len = filelength(D_f);          // Íàõîäèì ðàçìåð ôàéëà
    int i, j, kol;
    kol = len/size;                 //Êîëè÷åñòâî çàïèñåé â ôàéëå
    mas_Z = new TZap[kol];
// Ñ÷èòûâàåì çàïèñè èç ôàéëà â äèíàìè÷åñêèé ìàññèâ
    for (i=0; i < kol; i++) fread((mas_Z+i), size, 1, Fz);
    fclose(Fz);
    Memo1->Lines->Add("----Ñîðòèðîâàííûå ñâåäåíèÿ ïî "+soob[dx]+ "----");
    Memo1->Lines->Add("| Òèï ïðèáîðà | Uîáð max, B | Iïð max, mA | Iïðã, A | fðàá, kHz | Uïð, B | Iïð, mA | Iîáð, mkA |");
 
    for (i=0; i < kol-1; i++)
      for (j=i+1; j < kol; j++)
        if (func(mas_Z,i,dx) > func(mas_Z,j,dx)) {
          st = mas_Z[i];
          mas_Z[i] = mas_Z[j];
          mas_Z[j] = st;
        }
        for (i=0; i<kol; i++) Out(mas_Z[i], Memo1);
    delete []mas_Z;
}

Новые блоги и статьи Все статьи Все блоги /
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Рецензия / Мнение/ Перевод https:/ / **********/ gallery/ thinkpad-x220-tablet-porn-gzoEAjs . . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .

@Gus 26 / 37 / 10 Регистрация: 17.02.2009 Сообщений: 364
	15.12.2012, 21:14
	эм, а не проще было запилить шаблон, и не копировать один и тот же по функциональному смыслу слова код? 0

@shur1kk 0 / 0 / 0 Регистрация: 25.10.2012 Сообщений: 20
	15.12.2012, 21:20 [ТС]
	ну я как бы понимаю, что одно и тоже, так вот и спрашиваю, как это все сократить? 0