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

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

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

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

C++

//-----------------СОРТИРОВАТЬ--------------------------------------------------------
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++

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++

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++

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;
}

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

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

	16.12.2012, 14:25

Опции темы