Почему зацикливается программа после выполнения функции загрузки?

@alex75 · Регистрация: 23.02.2015

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

Вот моя функция, после её выполнения программа зацикливается.

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
ITEM* zagruzka(ITEM  *head)
{
ITEM *cur;
FILE *f;
cur=(ITEM*)malloc(sizeof(ITEM));
if((f=fopen("C:/TURBOC3/BIN/park.dat","rb"))==NULL)
{
printf("Errors");
getch();
}
cur=head;
fseek(f,0,SEEK_SET);
while(!feof(f))
{
fscanf(f,"%s",cur->name);
fseek(f,1,SEEK_CUR);
fscanf(f,"%s",cur->kat);
fseek(f,1,SEEK_CUR);
fscanf(f,"%d",cur->gv);
fseek(f,1,SEEK_CUR);
fscanf(f,"%f",cur->weight);
fseek(f,1,SEEK_CUR);
fscanf(f,"%f",cur->dvig);
fseek(f,1,SEEK_CUR);
cur->num=num;
num++;
cur=cur->next;
}
fclose(f);
return head;
}

Байт · 24.04.2015, 12:27

alex75, Вот что бросается в глаза.
1. После неудачного открытия файла и сообщения об ошибке вы продолжаете с файлом работать. А его указатель f = NULL
2. зачем fseek(f, 0...) после открытия файла? Он и так стоит на начале.
3. cur = malloc(...) и тут же cur = head. Паять ваша выделенная повисла в безвоздушном пространстве.
4. cur = cur->next. А что в поле next ?
5. Зачем возвращается return head ? Он и так вполне известен вызывающей программе.
6. Зачем вы смещаетесь на 1 байт после чтения каждого числа (fseek(f, 1, SEEK_CUR)) - так надо?
Общее впечатление от вашего кода - очень слабая теоретическая(и практическая) подготовка

@alex75 · 24.04.2015, 21:11 **[ТС]**

Вот я немного изменил функцию. Вся суть функции в том, чтобы загрузить из файла в список данные, данные записаны в формате "name kat gv weight dvig"(name и kat слова, gv целое число, а weight и dvig вещественные числа) и так n раз. Помимо этой функции в программе существует функция добавления элементов в список. Курсор я перемещаю, чтобы двигаться по файлу и считывать данные. Вся проблема в том, что при выполнении функции программа зацикливается.

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
ITEM* zagruzka(ITEM  *head)
{
ITEM *cur;
FILE *f;
if((f=fopen("C:/TURBOC3/BIN/park.dat","rb"))==NULL)
{ puts("Error");
getch();
return 0;
}
cur=(ITEM*)malloc(sizeof(ITEM));
if(cur==NULL)
{
puts("Error memory");
getch();
return 0;
}
cur=head;
 
if(head!=NULL)
    {while(cur->next!=NULL)
        {cur=cur->next;
        }
}
 
while(!feof(f))
{
fscanf(f,"%s",cur->name);
fseek(f,1,SEEK_CUR);
fscanf(f,"%s",cur->kat);
fseek(f,1,SEEK_CUR);
fscanf(f,"%d",cur->gv);
fseek(f,1,SEEK_CUR);
fscanf(f,"%f",cur->weight);
fseek(f,1,SEEK_CUR);
fscanf(f,"%f",cur->dvig);
fseek(f,1,SEEK_CUR);
cur->num=num;
num++;
cur=cur->next;
}
fclose(f);
return head;
}

Новые блоги и статьи Все статьи Все блоги /
Модель заражения группы наркоманов alhaos 17.04.2026 Условия задачи сформулированы тут Суть: - Группа наркоманов из 10 человек. - Только один инфицирован ВИЧ. - Колются одной иглой. - Колются раз в день. - Колются последовательно через. . .	Мысли в слух. Про "навсегда". kumehtar 16.04.2026 Подумалось тут, что наверное очень глупо использовать во всяких своих установках понятие "навсегда". Это очень сильное понятие, и я только начинаю понимать край его смысла, не смотря на то что давно. . .	My Business CRM MaGz GoLd 16.04.2026 Всем привет, недавно возникла потребность создать CRM, для личных нужд. Собственно программа предоставляет из себя базу данных клиентов, в которой можно фиксировать звонки, стадии сделки, а также. . .	Знаешь почему 90% людей редко бывают счастливыми? kumehtar 14.04.2026 Потому что они ждут. Ждут выходных, ждут отпуска, ждут удачного момента. . . а удачный момент так и не приходит.
Фиксация колонок в отчете СКД Maks 14.04.2026 Фиксация колонок в СКД отчета типа Таблица. Задача: зафиксировать три левых колонки в отчете. Процедура ПриКомпоновкеРезультата(ДокументРезультат, ДанныеРасшифровки, СтандартнаяОбработка) / / . . .	Настройки VS Code Loafer 13.04.2026 { "cmake. configureOnOpen": false, "diffEditor. ignoreTrimWhitespace": true, "editor. guides. bracketPairs": "active", "extensions. ignoreRecommendations": true, . . .	Оптимизация кода на разграничение прав доступа к элементам формы Maks 13.04.2026 Алгоритм из решения ниже реализован на нетиповом документе, разработанного в конфигурации КА2. Задачи, как таковой, поставлено не было, проделанное ниже исключительно моя инициатива. Было так:. . .	Контроль заполнения и очистка дат в зависимости от значения перечислений Maks 12.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеПерсонала", разработанного в конфигурации КА2. Задача: реализовать контроль корректности заполнения дат назначения. . .

Байт Диссидент 27714 / 17332 / 3810 Регистрация: 24.12.2010 Сообщений: 38,978
	24.04.2015, 12:27
	alex75, Вот что бросается в глаза. 1. После неудачного открытия файла и сообщения об ошибке вы продолжаете с файлом работать. А его указатель f = NULL 2. зачем fseek(f, 0...) после открытия файла? Он и так стоит на начале. 3. cur = malloc(...) и тут же cur = head. Паять ваша выделенная повисла в безвоздушном пространстве. 4. cur = cur->next. А что в поле next ? 5. Зачем возвращается return head ? Он и так вполне известен вызывающей программе. 6. Зачем вы смещаетесь на 1 байт после чтения каждого числа (fseek(f, 1, SEEK_CUR)) - так надо? Общее впечатление от вашего кода - очень слабая теоретическая(и практическая) подготовка 0