Сортировка массив структур: qsort выдает Expression syntax

@razer89 · Регистрация: 20.06.2010

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

Имеется код для сортировки массива структур:

C
1
qsort (base, n, sizeof(people), namecmp);

C
1
2
3
4
int namecmp (const void * a, const void * b)
{
    return strcmp (((people*)a).FIO, ((people*)b).FIO);
}

При вызове функции namecmp пишет: E2188 Expression syntax. Не могу понять, в чем ошибка?

go · 14.06.2012, 21:01

razer89, перезаписать в новый файл все, кроме тех, что нужно удалить.

@razer89 · 15.06.2012, 00:03 **[ТС]**

Вот, выкладываю код, полностью рабочий. Программа свои функции выполняет нормально, по моему мнению на этом можно остановиться (хотя и можно было добавить всяческих проверок на открытие файла, вводимых значений и т.д.). Выслушаю любую критику, замечания и предложения. Всем еще раз огромное спасибо за оказанное содействие!

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
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
//---------------------------------------------------------------------------
FILE *database;
typedef struct PEOPLE {char FIO [25]; float s2; int amount;} people_t;
 
void view_all();
void edit_item();
void sort();
int namecmp(const void* a, const void* b);
int s2cmp(const void* a, const void* b);
int amountcmp(const void* a, const void* b);
 
int main()
{
    int t;
 
    if ((database = fopen ("database.db", "rb+")) == NULL)
    {
        printf ("File not found! Create new? [Y/N] \n");
        char n [1];
        scanf ("%s", n);
        if (n[0] == 'y')
        {
            database = fopen ("database.db", "wb+");
            system ("cls");
        }
        else
        {
            t = 10;
        }
    }
    fclose (database);
 
    if (t != 10)
    {
 
        do
        {
            printf ("[1]View All\t[2]Edit Item\t[3]Sort\t[0]Exit\n");
            scanf ("%d", &t);
            switch (t) {
                case 1: system("cls");
                    view_all();
                    break;
 
                case 2: system("cls");
                    edit_item();
                    break;
                case 3: system("cls");
                    sort();
                    break;
            }
        }
        while (t != 0);
    }
}
 
void view_all()
{
    people_t people;
    database = fopen ("database.db", "rb");
    fread (&people, sizeof (people_t), 1, database);
    printf ("FIO \t\tSquare \t\tAmount\n");
    while (!feof(database))
    {
        printf ("%s \t\t%f \t%i \n", people.FIO, people.s2, people.amount);
        fread (&people, sizeof (people_t), 1, database);
    }
    fclose (database);
    system ("pause"); system ("cls");
}
 
void edit_item()
{
    int s = 0;
    people_t people;
 
    printf ("[1]Add Item\t[2]Delete Item\n");
    scanf ("%d", &s);
 
    do
    {
        switch (s)
        {
            case 1:
            system ("cls");
            database = fopen ("database.db", "ab");
            puts ("Input FIO");
            scanf ("%s", &people.FIO);
            puts ("Input square");
            scanf ("%f", &people.s2);
            puts ("Input amount");
            scanf ("%i", &people.amount);
            system ("cls");
            fwrite (&people, sizeof(people_t), 1, database);
            fclose (database);
            s = 0;
            break;
 
            case 2:
            int i = 0;
            system("cls");
            printf ("Select a number to delete\n");
            database = fopen ("database.db", "rb");
            fread (&people, sizeof (people_t), 1, database);
            printf ("#\tFIO \t\tSquare \t\tAmount\n");
            people_t base[100];
            while (!feof(database))
            {
                strcpy (base[i].FIO, people.FIO);
                base[i].s2 = people.s2;
                base[i].amount = people.amount;
                printf ("%d\t\%s \t\t%f \t%i \n", i + 1, people.FIO, people.s2, people.amount);
                fread (&people, sizeof (people_t), 1, database);
                i++;
            }
            fclose (database);
            int d;
            scanf ("%d", &d);
            int num = i;
            if (d <= i)
            {
                for (i = d - 1; i < num; i++)
                {
                    strcpy (base[i].FIO, base[i + 1].FIO);
                    base[i].s2 = base[i + 1].s2;
                    base[i].amount = base[i + 1].amount;
                }
                database = fopen ("database.db", "wb+");
                for (i = 0; i < (num - 1); i++)
                {
                    strcpy (people.FIO, base[i].FIO);
                    people.s2 = base[i].s2;
                    people.amount = base[i].amount;
                    fwrite (&people, sizeof(people_t), 1, database);
                    printf ("%d\t\%s \t\t%f \t%i \n", i + 1, people.FIO, people.s2, people.amount);
                }
                fclose (database);
                s = 0;
            }
            else printf ("Number is not correct!");
            system ("pause");
            system ("cls");
            break;
        }
    }
    while (s != 0);
}
 
void sort()
{
    people_t people;
    int i = 0;
    int c;
    people_t base [100];
    database = fopen ("database.db", "rb");
    fread (&people, sizeof (people_t), 1, database);
 
    while (!feof(database))
    {
        strcpy (base[i].FIO, people.FIO);
        base[i].s2 = people.s2;
        base[i].amount = people.amount;
        fread (&people, sizeof (people_t), 1, database);
        i++;
    }
 
    printf ("Sort by: \n[1]FIO \t[2]Square \t[3]Amount\n");
    scanf ("%d", &c);
    int n = i;
    switch (c)
    {
        case 1: system("cls");
        qsort (base, n, sizeof(people), namecmp);
        for (i = 0; i < n; i++)
        {
            printf ("%s\t%f\t%i\n", base[i].FIO, base[i].s2, base[i].amount);
        }
        system ("pause");
        system ("cls");
        break;
 
        case 2: system("cls");
        qsort (base, n, sizeof(people), s2cmp);
        for (i = 0; i < n; i++)
        {
            printf ("%s\t%f\t%i\n", base[i].FIO, base[i].s2, base[i].amount);
        }
        system ("pause");
        system ("cls");
        break;
 
        case 3: system("cls");
        qsort (base, n, sizeof(people), amountcmp);
        for (i = 0; i < n; i++)
        {
            printf ("%s\t%f\t%i\n", base[i].FIO, base[i].s2, base[i].amount);
        }
        system ("pause");
        system ("cls");
        break;
 
    }
}
 
int namecmp (const void * a, const void * b)
{
    return strcmp (((people_t*)a)->FIO, ((people_t*)b)->FIO);
}
 
int s2cmp (const void* a, const void* b)
{
    float d = ((people_t*)a)->s2 - ((people_t*)b)->s2;
    return ( d < 0.0 ) ? -1 : ( d > 0.0 ) ? 1 : 0;
}
 
int amountcmp (const void* a, const void* b)
{
    return ((people_t*)a)->amount - ((people_t*)b)->amount;
}
 
 
//---------------------------------------------------------------------------

Добавлено через 1 минуту
easybudda, к вам еще небольшой вопрос. Не могли бы вы пояснить смысл строки:

C
1
return ( d < 0.0 ) ? -1 : ( d > 0.0 ) ? 1 : 0;

@easybudda · 15.06.2012, 01:17

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

Не могли бы вы пояснить смысл строки

Ну а как бы ещё два вещественных числа сравнить так, чтобы вернуть целое результатом? Не приводить же разницу к целому.

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

C
1
2
char n [1]; 
scanf ("%s", n);

Это не правильно. В n помещается 1 символ, а scanf после него ещё и 0 запишет. А если пользователь решит вместо одной буквы y/n развёрнутый ответ дать? В любом случае выход за границу массива...
Пункт edit мрачен, вызывает вопрос - это вообще работает? Готов на слово поверить, что как бы да, но выглядит жутковато... При чтении в массив (размер массива так и не удосужились вычислить, а я показывал, как это делается) из каких соображений нужно читать из файла в переменную, а потом из неё в очередной элемент массива всё копировать? За чем вообще нужна ещё одна переменная, если их и так уже аж 100 штук заведено?
Ну и самое главное - работать всё это будет (если будет) крайне медленно. Практически на каждое действие файл заново открывается и перечитывается. Проще все данные в памяти держать, а в файл по команде сбрасывать. Вообще работа напрямую с файлами - вещь достаточно сложная и делается совсем по-другому.
Короче, будь я преподом - больше трёх баллов в жизнь бы не поставил, и то при условии, что оно у меня на глазах скомпилируется и заработает.
Вы уж не обижайтесь...

@razer89 · 15.06.2012, 02:48 **[ТС]**

easybudda, спасибо за честную критику! =))

Новые блоги и статьи Все статьи Все блоги /
Транскрипция 55-минутного видео через Whisper: WhisperDesktop облажался, спас Google Colab[ anaschu 01.06.2026 Понадобилось получить текст из свежезагруженного видео на YouTube. Казалось бы, задача на пять минут. Заняла полтора часа. Делюсь опытом — может кому пригодится последовательность решений. . . .	21 мат мед. Планы на развитие модели здравоСохранения anaschu 01.06.2026 AnyLogic: план развития симуляционной модели рабочего коллектива — динамический абсентеизм, реальные данные, три сценария сравнения Продолжаю серию постов о дискретно-событийной модели рабочего. . .	20. Мат мед. Абсентеизм как отдельный тип простоя anaschu 29.05.2026 Апдейт модели: исправленные баги, абсентеизм и новые механизмы Продолжаю развивать ранее описанную модель рабочего коллектива на AnyLogic. За последние несколько дней был проведён серьёзный. . .	19. здоровье, усталость и психотип работника влияют на производительность предприятия, и наоборот, производительность на здоровье, усталось и психотип anaschu 28.05.2026 Дискретно-событийная модель рабочего коллектива на AnyLogic: здоровье, выгорание, психотипы и микростимуляция Привет, коллеги. Хочу поделиться итогами нескольких недель работы над симуляционной. . .
"Прокси" для последовательного порта Eddy_Em 28.05.2026 Эту штуку написал я достаточно давно. Но сейчас вот понадобилось настроить датчик грозы, но при этом не отключать его от "метеодемона". Соответственно, надо запустить этот "прокси": метеодемон будет. . .	Рефакторинг программы уравнивания. Massaraksh7 26.05.2026 Пример по предыдущей записи в блоге. Но, надо заметить, что, во-первых, там оптимизация не только математики, но и работы с базой данных, и с графами, а во-вторых, это ещё не всё.	Использование TThread в Lazarus для математических вычислений. Massaraksh7 25.05.2026 Производя рефакторинг своих программ на предмет ускорения их работы, обратил внимание на такой аспект, как сокращение времени матвычислений. Дело в том, что приходится работать с большими матрицами. . .	Модель здравосохранения 18. Чем здоровее работник, тем быстрее выгорает anaschu 24.05.2026 Имитационная модель корпоративного здравоохранения: что показывает математика Сегодня в модели рабочего коллектива на AnyLogic появились три новые механики — выгорание через накопленную усталость,. . .

go 3646 / 1378 / 243 Регистрация: 16.04.2009 Сообщений: 4,526
	14.06.2012, 21:01
	razer89, перезаписать в новый файл все, кроме тех, что нужно удалить. 0

@razer89 15 / 15 / 0 Регистрация: 20.06.2010 Сообщений: 142
	15.06.2012, 02:48 [ТС]
	easybudda, спасибо за честную критику! =)) 0