не работает qsort

@sergeu90 · Регистрация: 17.10.2009

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

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
#include <iostream>
#include  <math.h>
#include <iomanip>
#include <string.h>
using namespace std;
 
struct MARSH
{
    char nac_punkt[100];
    char konec_punkt[100];
    int   nomer;
    int   dlina;
}*p1[50];
int comp(const void* a, const void* b)
{
    const  MARSH* k = (const  MARSH*)a;
    const  MARSH* m = (const  MARSH*)b;
    int s = ((k ->dlina) - (m ->dlina));
       
    return s;
}
void  main(void)
{
    FILE *f1;
    setlocale(LC_ALL,"Russian");
    if((f1=fopen("2.dat", "rb+"))==NULL) {
        printf("Ошибка при открытии файла.\n");
        exit(1);
    }
 
 
    for(int i=0;i<5;i++)
    {
        MARSH *pmarh=new MARSH[i+1];
 
        if (pmarh != NULL) cout << "Память успешно выделена" << endl; 
        else cout << "Ошибка выделения памяти" << endl; 
        fread(&pmarh[i],sizeof(MARSH),1,f1);
        p1[i]=&pmarh[i];
        cout<<i+1<<"-aя запись"<<endl;
        cout<<"начальный пункт  "<<pmarh[i].nac_punkt<<endl;
        cout<<"название конечного пункта  "<<pmarh[i].konec_punkt<<endl;
        cout<<"длина  "<<pmarh[i].dlina<<endl;
        cout<<"номер   "<<pmarh[i].nomer<<endl;    
    }
    cout<<"Сортировка"<<endl;
    cout<<endl;
    cout<<endl;
    cout<<endl;
    cout<<endl;
    cout<<endl;
    cout<<endl;
    cout<<endl;
    qsort(p1, 5, sizeof(p1[0]), comp);
    for(int i=0;i<5;i++)
    {      
        cout<<i+1<<"-aя запись"<<endl;
        cout<<"начальный пункт  "<<p1[i]->nac_punkt<<endl;
        cout<<"название конечного пункта  "<<p1[i]->konec_punkt<<endl;
        cout<<"длина  "<<p1[i]->dlina<<endl;
        cout<<"номер   "<<p1[i]->nomer<<endl   
    }
    fclose(f1);
    system("PAUSE");
}

не работает qsort нужным образом тоесть не сортирует

Добавлено через 14 минут
посмотрите пожалуйста код почему qsort не хочет сортировать

@gooseim · 11.03.2012, 17:25

qsort(p1, 5, sizeof(p1[0]), comp);

В первом параметре вы передаете MARSH**, а надо MARSH*.

@sergeu90 · 11.03.2012, 17:26 **[ТС]**

а как исправить это?

@Xind · 11.03.2012, 17:29

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

MARSH *pmarh=new MARSH[i+1];

В цикле каждый раз выделяется память. Это так задумано?

@gooseim · 11.03.2012, 17:30

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
struct MARSH
{
  char nac_punkt[100];
  char konec_punkt[100];
  int   nomer;
  int   dlina;
}p1[50];
...
for(int i=0;i<5;i++)
        {
 
          fread(&p1[i],sizeof(MARSH),1,f1);
                        
                  cout<<i+1<<"-aя запись"<<endl;
                        cout<<"начальный пункт  "<<p1[i].nac_punkt<<endl;
                    cout<<"название конечного пункта  "<<p1[i].konec_punkt<<endl;
                        cout<<"длина  "<<p1[i].dlina<<endl;
                        cout<<"номер   "<<p1[i].nomer<<endl;
}

и далее в том же духе

@sergeu90 · 11.03.2012, 17:36 **[ТС]**

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
for(int i=0;i<5;i++)
       {
           if (p1[i]->dlina<p1[i+1]->dlina)
           {
               MARSH *p2;
             p2=p1[i];
            p1[i]=p1[i+1];
            p1[i+1]=p2;
 
 
           }
 
}

пробывал так все равно

Добавлено через 1 минуту
мне через массив указателей надо сделать по заданию

@gooseim · 11.03.2012, 17:41

Невнимательно посмотрел.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
for(int i=0;i<5;i++)
 
       {
                  p1[i]=new MARSH;
 
                   if (p1[i]!= NULL) cout << "Память успешно выделена" << endl; 
   else cout << "Ошибка выделения памяти" << endl; 
 
          fread(p1[i],sizeof(MARSH),1,f1);
                       
                  cout<<i+1<<"-aя запись"<<endl;
                        cout<<"начальный пункт  "<<p1[i]->nac_punkt<<endl;
                    cout<<"название конечного пункта  "<<p1[i]->konec_punkt<<endl;
                        cout<<"длина  "<<p1[i]->dlina<<endl;
                        cout<<"номер   "<<p1[i]->nomer<<endl;
         
}

@sergeu90 · 11.03.2012, 18:01 **[ТС]**

сделал наконецто своим методом кому надо возьмите

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
#include <iostream>
#include  <math.h>
#include <iomanip>
#include <string.h>
using namespace std;
 
 
 
 
struct MARSH
{
  char nac_punkt[100];
  char konec_punkt[100];
  int   nomer;
  int   dlina;
}*p1[50];
 
 
int comp(const void* a, const void* b)
{
        const  MARSH* k = (const  MARSH*)a;
        const  MARSH* m = (const  MARSH*)b;
        int s = ((k ->dlina) - (m ->dlina));
        
        return s;
}
 
void  main(void)
{
 
     FILE *f1;
    setlocale(LC_ALL,"Russian");
if((f1=fopen("2.dat", "rb+"))==NULL) {
    printf("Ошибка при открытии файла.\n");
    exit(1);
  }
 
 
for(int i=0;i<5;i++)
 
       {
           MARSH *pmarh=new MARSH[i+1];
            //p1[i]=new MARSH;
           if (pmarh != NULL) cout << "Память успешно выделена" << endl; 
   else cout << "Ошибка выделения памяти" << endl; 
 
          fread(&pmarh[i],sizeof(MARSH),1,f1);
            p1[i]=&pmarh[i];
          cout<<i+1<<"-aя запись"<<endl;
            cout<<"начальный пункт  "<<pmarh[i].nac_punkt<<endl;
            cout<<"название конечного пункта  "<<pmarh[i].konec_punkt<<endl;
            cout<<"длина  "<<pmarh[i].dlina<<endl;
            cout<<"номер   "<<pmarh[i].nomer<<endl;
    
    
}
cout<<"Сортировка"<<endl;
cout<<endl;
cout<<endl;
cout<<endl;
cout<<endl;
cout<<endl;
cout<<endl;
cout<<endl;
 
 for(int i = 0; i < 5 - 1; ++i) 
    {            
        for(int j = 0; j < 5 - 1; ++j) 
        {     
            if (p1[j+1]->dlina>p1[j]->dlina)
            {
           MARSH *p2;
             p2=p1[j+1];
            p1[j+1]=p1[j];
            p1[j]=p2;
            }
        }
    }
 
//qsort(p1, 5, sizeof(p1[0]), comp);
 
 
 
 
 
 
for(int i=0;i<5;i++)
 
       {
           
          cout<<i+1<<"-aя запись"<<endl;
            cout<<"начальный пункт  "<<p1[i]->nac_punkt<<endl;
            cout<<"название конечного пункта  "<<p1[i]->konec_punkt<<endl;
            cout<<"длина  "<<p1[i]->dlina<<endl;
            cout<<"номер   "<<p1[i]->nomer<<endl;
    
    
}
 
 
 
 
 
 
 
 
 
 
 
    
 
       fclose(f1);
    system("PAUSE");
 
}

@gooseim · 11.03.2012, 18:07

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

MARSH *pmarh=new MARSH[i+1];

Помедитируйте над этой строкой. 2 вам поставят.

@Xind · 11.03.2012, 18:13

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

//qsort(p1, 5, sizeof(p1[0]), comp);

Не по теме:

Что-то новенькое. Чтобы решить проблему нужно поставить в комментарии?!
Вот это да!..

@sergeu90 · 11.03.2012, 18:16 **[ТС]**

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

@Xind · 11.03.2012, 18:18

sergeu90, у вас еще в первом цикле творится непонятно что "MARSH *pmarh=new MARSH[i+1];"

Не по теме:

Ну ладно. Решили так решили. Хозяин барин.

@sergeu90 · 11.03.2012, 18:36 **[ТС]**

почему непонятно выделяю память

@gooseim · 11.03.2012, 18:39

sergeu90, зачем вы создаете массив, если вам нужен только один элемент?

@Xind · 11.03.2012, 18:46

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

почему непонятно выделяю память

Смотри, в 1-м цикле у тебя при i = 0 происходит

C++
1
new MARSH[1];

т.е. в памяти 1 объект
ты к нему обратился &pmarh[0] пока все нормально.

но затем у тебя посходит в следующей итерации

C++
1
new MARSH[2];

т.е. 2 объекта и заметь предыдущие данные остались в памяти
а при i = 1 у тебя работает только одни объект, остальное выделенное не используется.
и так далее каждый раз выделяется все больше неиспользуемой памяти, непонятно зачем?

вынеси за цикл

C++
1
MARSH *pmarh=new MARSH[i+1];

и замени на

C++
1
MARSH *pmarh=new MARSH;

а в цикле обращайся к одному объекту pmarh
или еще лучше используй только p1[i]

@gooseim · 11.03.2012, 18:49

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

@sergeu90 · 11.03.2012, 18:53 **[ТС]**

спасибо учту

Добавлено через 1 минуту
MARSH *pmarh=new MARSH если так то повреждениче кучи ошибка

@gooseim · 11.03.2012, 18:55

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
for(int i=0;i<5;i++)
    {
        MARSH *pmarh=new MARCH;
 
        if (pmarh != NULL) cout << "Память успешно выделена" << endl; 
        else cout << "Ошибка выделения памяти" << endl; 
        fread(pmarh,sizeof(MARSH),1,f1);
        p1[i]=pmarh;
        cout<<i+1<<"-aя запись"<<endl;
        cout<<"начальный пункт  "<<pmarh->nac_punkt<<endl;
        cout<<"название конечного пункта  "<<pmarh->konec_punkt<<endl;
        cout<<"длина  "<<pmarh->dlina<<endl;
        cout<<"номер   "<<pmarh->nomer<<endl; 
    }

@sergeu90 · 11.03.2012, 19:00 **[ТС]**

а вот хочу освободить память в конце так оно не возможно только если в цикле освобождать а я хочу после цикла освободитть

@gooseim · 11.03.2012, 19:21

В самом конце.

C++
1
2
3
4
for(int i=0;i<5;i++)
{
  delete p1[i];
}

Новые блоги и статьи Все статьи Все блоги /
Символьное дифференцирование igorrr37 13.02.2026 / * Логарифм записывается как: (x-2)log(x^2+2) - означает логарифм (x^2+2) по основанию (x-2). Унарный минус обозначается как ! */ #include <iostream> #include <stack> #include <cctype>. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .

@sergeu90 28 / 29 / 5 Регистрация: 17.10.2009 Сообщений: 739
	11.03.2012, 17:26 [ТС]
	а как исправить это? 0

@sergeu90 28 / 29 / 5 Регистрация: 17.10.2009 Сообщений: 739
	11.03.2012, 18:16 [ТС]
	ну это вы уже преувеличили я сделал просто по другому сортировку пузырьком 0

@Xind 277 / 150 / 25 Регистрация: 05.11.2011 Сообщений: 429 Записей в блоге: 1
	11.03.2012, 18:18
	sergeu90, у вас еще в первом цикле творится непонятно что "*MARSH pmarh=new MARSH[i+1];" Не по теме:** Ну ладно. Решили так решили. Хозяин барин. 0

@sergeu90 28 / 29 / 5 Регистрация: 17.10.2009 Сообщений: 739
	11.03.2012, 18:36 [ТС]
	почему непонятно выделяю память 0

@gooseim 516 / 421 / 92 Регистрация: 23.09.2010 Сообщений: 1,165
	11.03.2012, 18:39
	sergeu90, зачем вы создаете массив, если вам нужен только один элемент? 0

@gooseim 516 / 421 / 92 Регистрация: 23.09.2010 Сообщений: 1,165
	11.03.2012, 18:49
	Xind, выносить как раз не надо, иначе в p1 будут все одинаковые элементы = последнему считанному. 1

@sergeu90 28 / 29 / 5 Регистрация: 17.10.2009 Сообщений: 739
	11.03.2012, 18:53 [ТС]
	спасибо учту Добавлено через 1 минуту MARSH *pmarh=new MARSH если так то повреждениче кучи ошибка 0

@sergeu90 28 / 29 / 5 Регистрация: 17.10.2009 Сообщений: 739
	11.03.2012, 19:00 [ТС]
	а вот хочу освободить память в конце так оно не возможно только если в цикле освобождать а я хочу после цикла освободитть 0