Ошибка при чтении файла

@Alik-Soldier · Регистрация: 01.05.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
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
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
#include "stdafx.h"
#include <iostream>
#include <iomanip>
#include <conio.h>
#include <string.h>
 
using namespace std;
void Menu();
int c=0;                            //Глобальная переменная с - для количества элементов структуры 
int k=0;                            //Глобальная переменная к - для номера увольняемого абонента
struct Worker                       //Структура для абонента
{
        struct FIO
        {
        char surname[50];           //Фамилия 
        char name[50];              //Имя
        char second_name[50];       //Отчество
        };  
    FIO fio;
    char birth[10];                 //Дата рождения
    char sex[30];                   //Пол
    int staj;                       //Стаж работы по специальности 
    char adress[100];               //Домашний адрес   
    char berth[50];                 //Должность
    int salary;                     //Оклад
    };
class Firma                         //Создаю свой класс
{
public:
    Firma();                        //Конструктор    
    void LoadFile();                //Функция для считывания информации из файла
    void New_worker();              //Функция для добавления экземпляра структуры
    void Remove_base();             //Функция для удаления сотрудника
    void See_Base();                //Функция просмотра базы
    void Naity();                   //Функция для поиска по адресу/фамилии
    void Yvolit();                  //Функция для увольнения сотрудника
    void SaveFile();                //Функция для записи телефонной книги в файл
    Worker work[40];                //40 экземпляров структуры Worker 
};
Firma fir;                          //Экземпляр класса
Firma::Firma()                      //Инициализация конструктора
{
}
 
void Firma::LoadFile()              
{   
    FILE *f = fopen("workers.txt", "r");    //Открываем базу 
    if((f=fopen("workers.txt","r"))==NULL)          
        printf("Cannot open file\n");       //Сообщение, если невозможно открыть файл
    char buff[150],*p;
    int i=0;
    while(!feof(f))                         //Цикл продолжается, пока не достигнут конец файла
    {
        fgets (buff, sizeof(buff), f);      //Считывание первой строки файла
        p = strtok (buff," ");              //Указатель на первую строку в файле до первого знака " "
        strcpy (work[i].fio.surname, p);    //Копирование первой строки до " " в массив экземпляра структуры
        p = strtok (NULL, " ");             //Указатель на первую строку в файле до второго знака " "
        strcpy (work[i].fio.name, p);       //Копирование первой строки от " " до " " в массив экземпляра структуры
        p = strtok (NULL, "\n");            //Указатель на первую строку в файле от второго знака " " до конца строки
        strcpy (work[i].fio.second_name, p);//Копирование первой строки от " " до конца строки в массив экземпляра структуры
        fgets (buff, 10, f);                //Считывание второй строки файла
        p = strtok (buff, "\n");            //Указатель на вторую строку в файле
        strcpy (work[i].birth, p);          //Копирование второй строки в массив экземпляра структуры
        fgets(buff, sizeof(buff),f);        //Считывание третьей строки файла
        p = strtok (buff, "\n");            //Указатель на третью строку в файле
        strcpy (work[i].sex, p);            //Копирование третьей строки в массив экземпляра структуры
        fgets(buff, sizeof(buff),f);        //Считывание четвертой строки файла
        work[i].staj = atoi(buff);          //Присваивание переменной экземпляра структуры значения из четвертой строки файла
        fgets(buff, sizeof(buff),f);        //Считывание пятой строки файла
        p = strtok (buff, "\n");            //Указатель на пятую строку в файле
        strcpy (work[i].adress, p);         //Копирование пятой строки в массив экземпляра структуры
        fgets(buff, sizeof(buff),f);        //Считывание шестой строки файла
        p = strtok (buff, "\n");            //Указатель на шестую строку в файле
        strcpy (work[i].berth, p);          //Копирование шестой строки в массив экземпляра структуры
        fgets(buff,sizeof(buff),f);         //Считывание седьмой строки файла
        work[i].salary = atoi(buff);        //Присваивание переменной экземпляра структуры значения из седьмой строки файла
        fgets(buff,sizeof(buff),f);         //Считывание восьмой (пустой) строки файла
        i++;                                
    }
    c=i;                                    //Количество структур в файле           
    fclose(f);                              //Закрываем базу
}
 
void Firma::New_worker()
{
    char buf[150],*p;
    cout<<"Enter telephon subscriber surname, name and second name"<<endl;  //ФИО сотрудника
    _flushall();                            //Очищает буфер перед вводом сотрудника
    cin.getline(buf,150,'\n');
    p = strtok(buf, " \n");
    strcpy(work[c].fio.surname, p);
    p = strtok(NULL, " \n");
    strcpy(work[c].fio.name, p);
    p = strtok(NULL, "\n\0");
    strcpy(work[c].fio.second_name, p);
    cout<<"Enter date of birth DD/MM/YY"<<endl;                 //Дата рождения сотрудника
    cin.getline(work[c].birth, sizeof(work[c].birth), '\n');            
    cout<<"Enter sex of worker"<<endl;
    cin.getline(work[c].sex, sizeof(work[c].sex), '\n');
    cout<<"Enter experience of worker"<<endl;
    cin>>work[c].staj;
    _flushall();
    cout<<"Enter telephon subscriber adress:"<<endl;            //Адрес сотрудника
    cin.getline(work[c].adress, sizeof(work[c].adress), '\n');
    cout<<"Enter berth of worker"<<endl;                        //Должность
    cin.getline(work[c].berth, sizeof(work[c].berth), '\n');
    cout<<"Enter salary, $:"<<endl;                             //Оклад
    cin>>work[c].salary;
    c++;                                    //Добавление 1 экземпляра структуры
}
void Firma::Yvolit()
{
    char sname[50], nm[50], snm[50];
    bool Yv = false;                        //Флажок для поиска и удаления абонента
    cout<<"Enter worker surname, name and second name, who you want to discharge"<<endl;//Уволить
    _flushall();                    
    cin.getline(sname, sizeof(sname), 32);
    cin.getline(nm, sizeof(nm), 32);
    cin.getline(snm, sizeof(snm));
    for(int i=0; i<c; i++)
    {
        char s = strcmp (sname, work[i].fio.surname);
        char n = strcmp (nm, work[i].fio.name);
        char sn = strcmp (snm, work[i].fio.second_name);
        if (s==0 && n==0 && sn==0)
        {
            k=i;
            Yv = true;
            cout<<"worker is discharged"<<endl;
        }
    }
    if(Yv==true)
    {
        --c;
        for(int j=k; j<c; j++)
            work[j]=work[j+1];
    }
    else
        cout<<"There is no such worker"<<endl;
}
void Firma::Naity()
{
    bool find=false;                    //Флажок для поиска абонента
    char berth[50];
    _flushall();
    cout<<"Enter worker berth"<<endl;   //Должность
    cin.getline(berth,50,'\n');
    for(int i=0; i<c; i++)
    {
        char fnd = strcmp(berth, work[i].berth);
            if(fnd==0)
            {
                find=true;
                printf("%s %s %s\n", work[i].fio.surname, work[i].fio.name, work[i].fio.second_name);
                printf("\n");
            }
    }
    if(find==false)
        cout<<" Telephon subscriber isn't find"<<endl;
}
void Firma::Remove_base()
{ 
    c=0;                //Количество экземпляров структуры, которое было в файле, приравниваем к 0
    cout<<"Base is removed"<<endl;
}
int main()
{
    fir.LoadFile();
    Menu(); 
    fir.SaveFile();
    return 0;
}
void Menu()
{
    int control;                            //Переменная для реакции на клавиатуру
    cout<<"Choose menu position"<<endl;     //Выбрать позицию из меню
    cout<<"\nMENU:\n1. New worker \n2. Discharge worker \n3. Find workers \n4. See base \n5. Remove base \n6. Exit\n"<<endl;
    cin>>control;
    for(;;)
    {
        if(control==1)
            fir.New_worker();
        else
        if(control==2)
            fir.Yvolit();
        else
        if(control==3)
            fir.Naity();
        if(control==4)
            fir.See_Base();
        else
        if(control==5)
            fir.Remove_base();
        if (control==6)break;
        else
        if(control==0 || control>6)
            cout<<"You press incorrect button. Please repeat"<<endl;    //Нажата неверная клавиша, повторить
        cout<<"Choose menu position"<<endl;
        cin>>control;
    }
}
void Firma::See_Base()
{
    _flushall();
    bool se = false;
    char fam[50];
    cout<<"Enter surname worker"<<endl;
    cin.getline(fam, sizeof(fam), '\n');
    for(int i=0; i<c; i++)
    {
        if(strcmp(fam, work[i].fio.surname)==0)
        {
            printf("%s %s %s\n",work[i].fio.surname,work[i].fio.name,work[i].fio.second_name);
            printf("%s\n",work[i].birth);
            printf("%s\n",work[i].sex);
            printf("%d\n",work[i].staj);
            printf("%s\n",work[i].adress);
            printf("%s\n",work[i].berth);
            printf("%d\n",work[i].salary);
            printf("\n");
            se = true;
        }
    }
    if(se = false)
        cout<<"There is no such worker"<<endl;
}
void Firma::SaveFile()
{
    FILE *f;                        
    f = fopen("workers.txt", "w");          //Открытие базы
    for(int i=0; i<c; i++)                  //Цикл для записи сотрудников в базу
    {
        fprintf(f,"%s %s %s\n",work[i].fio.surname,work[i].fio.name,work[i].fio.second_name);
        fprintf(f,"%s\n",work[i].birth);
        fprintf(f,"%s\n",work[i].sex);
        fprintf(f,"%d\n",work[i].staj);
        fprintf(f,"%s\n",work[i].adress);
        fprintf(f,"%s\n",work[i].berth);
        fprintf(f,"%d\n",work[i].salary);
        if(i<c-1) fprintf(f,"\n");
    }
    fclose(f);                              //Закрытие базы
}

Программа рабочая...Но есть одна проблема: если файл workers.txt пустой, то при запуске программы ошибка появляется...Как исправить данную ошибку?
Прилагаю файл:
http://slil.ru/27711657

@FunDuck · 31.05.2009, 20:54

C++
1
2
FILE *f = fopen("workers.txt", "r");    //Открываем базу 
        if((f=fopen("workers.txt","r"))==NULL)// Ты файл повторно открыть пытаешся

@Gravity · 31.05.2009, 20:58

Мало того, что открываешь повторно, так еще и продолжаешь работу функции даже если файл открыть не удалось. При таком условии

C++
1
2
if((f=fopen("workers.txt","r"))==NULL)              
        printf("Cannot open file\n");           //Сообщение, если невозможно открыть файл

надо делать return из функции, чтобы не работать с NULL'ом.

@Alik-Soldier · 31.05.2009, 21:05 **[ТС]**

Да,я не прав...

C++
1
2
3
4
5
6
FILE *f;    
    if((f=fopen("workers.txt","r"))==NULL)  
    {
        printf("Cannot open file\n");       
        exit(1);
    }

Но после этого изменения все та же ошибка...

@FunDuck · 31.05.2009, 21:19

После каждого fgetc надо проверять на окончание файла.

@Gravity · 31.05.2009, 21:23

Ну так у тебя LoadFile написан вобще без проверок на то, что читают функции из файла. Если strtok где-то не найдет пробела в строке или '\n', то вернет опять же NULL, а ты хочешь записать его в буфер рабочих.

@Alik-Soldier · 31.05.2009, 21:40 **[ТС]**

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
while(!feof(f))                         //Цикл продолжается, пока не достигнут конец файла
    {
        fgets (buff, sizeof(buff), f);      //Считывание первой строки файла
        if(strtok(buff, control)==0)
        {
            cout<<"Base is empty"<<endl;
            break;
        }
        else
        {
        p = strtok (buff," ");              //Указатель на первую строку в файле до первого знака " "
        strcpy (work[i].fio.surname, p);    //Копирование первой строки до " " в массив экземпляра структуры
        p = strtok (NULL, " ");             //Указатель на первую строку в файле до второго знака " "
        strcpy (work[i].fio.name, p);       //Копирование первой строки от " " до " " в массив экземпляра структуры
        p = strtok (NULL, "\n");            //Указатель на первую строку в файле от второго знака " " до конца строки
        strcpy (work[i].fio.second_name, p);//Копирование первой строки от " " до конца строки в массив экземпляра структуры
        fgets (buff, 10, f);                //Считывание второй строки файла
        p = strtok (buff, "\n");            //Указатель на вторую строку в файле
        strcpy (work[i].birth, p);          //Копирование второй строки в массив экземпляра структуры
        fgets(buff, sizeof(buff),f);        //Считывание третьей строки файла
        p = strtok (buff, "\n");            //Указатель на третью строку в файле
        strcpy (work[i].sex, p);            //Копирование третьей строки в массив экземпляра структуры
        fgets(buff, sizeof(buff),f);        //Считывание четвертой строки файла
        work[i].staj = atoi(buff);          //Присваивание переменной экземпляра структуры значения из четвертой строки файла
        fgets(buff, sizeof(buff),f);        //Считывание пятой строки файла
        p = strtok (buff, "\n");            //Указатель на пятую строку в файле
        strcpy (work[i].adress, p);         //Копирование пятой строки в массив экземпляра структуры
        fgets(buff, sizeof(buff),f);        //Считывание шестой строки файла
        p = strtok (buff, "\n");            //Указатель на шестую строку в файле
        strcpy (work[i].berth, p);          //Копирование шестой строки в массив экземпляра структуры
        fgets(buff,sizeof(buff),f);         //Считывание седьмой строки файла
        work[i].salary = atoi(buff);        //Присваивание переменной экземпляра структуры значения из седьмой строки файла
        fgets(buff,sizeof(buff),f);         //Считывание восьмой (пустой) строки файла
        i++;                                
        }
    }

вот такая проверка не пойдет?

После каждого fgetc надо проверять на окончание файла.

После fgets?И как?И что это даст?

@FunDuck · 31.05.2009, 21:45

Я имел ввиду :

C++
1
2
fgets (buff, sizeof(buff), f); 
if (feof(f))break;

@Gravity · 31.05.2009, 21:47

Сообщение от Alik-Soldier

вот такая проверка не пойдет?

Сам проверь, твоя же прога.

@Alik-Soldier · 31.05.2009, 21:54 **[ТС]**

Спасибо)))Вы мне очень помогли)))

Добавлено через 6 минут 2 секунды

Сам проверь, твоя же прога.

Не пойдет...

@Gravity · 31.05.2009, 21:55

Значит ставь и на остальные стртоки.

Новые блоги и статьи Все статьи Все блоги /
Загрузка PNG с альфа-каналом на SDL3 для Android: с помощью SDL3_image 8Observer8 27.01.2026 Содержание блога SDL3_image - это библиотека для загрузки и работы с изображениями. Эта пошаговая инструкция покажет, как загрузить и вывести на экран смартфона картинку с альфа-каналом, то есть с. . .	влияние грибов на сукцессию anaschu 26.01.2026 Бифуркационные изменения массы гриба происходят тогда, когда мы уменьшаем массу компоста в 10 раз, а скорость прироста биомассы уменьшаем в три раза. Скорость прироста биомассы может уменьшаться за. . .	Воспроизведение звукового файла с помощью SDL3_mixer при касании экрана Android 8Observer8 26.01.2026 Содержание блога SDL3_mixer - это библиотека я для воспроизведения аудио. В отличие от инструкции по добавлению текста код по проигрыванию звука уже содержится в шаблоне примера. Нужно только. . .	Установка Android SDK, NDK, JDK, CMake и т.д. 8Observer8 25.01.2026 Содержание блога Перейдите по ссылке: https:/ / developer. android. com/ studio и в самом низу страницы кликните по архиву "commandlinetools-win-xxxxxx_latest. zip" Извлеките архив и вы увидите. . .
Вывод текста со шрифтом TTF на Android с помощью библиотеки SDL3_ttf 8Observer8 25.01.2026 Содержание блога Если у вас не установлены Android SDK, NDK, JDK, и т. д. то сделайте это по следующей инструкции: Установка Android SDK, NDK, JDK, CMake и т. д. Сборка примера Скачайте. . .	Использование SDL3-callbacks вместо функции main() на Android, Desktop и WebAssembly 8Observer8 24.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .	моя боль iceja 24.01.2026 Выложила интерполяцию кубическими сплайнами www. iceja. net REST сервисы временно не работают, только через Web. Написала за 56 рабочих часов этот сайт с нуля. При помощи perplexity. ai PRO , при. . .	Модель сукцессии микоризы anaschu 24.01.2026 Решили писать научную статью с неким РОманом

@FunDuck 692 / 383 / 51 Регистрация: 22.01.2009 Сообщений: 1,135
	31.05.2009, 21:19
	После каждого fgetc надо проверять на окончание файла. 0

@Gravity 577 / 571 / 65 Регистрация: 29.01.2009 Сообщений: 1,274
	31.05.2009, 21:23
	Ну так у тебя LoadFile написан вобще без проверок на то, что читают функции из файла. Если strtok где-то не найдет пробела в строке или '\n', то вернет опять же NULL, а ты хочешь записать его в буфер рабочих. 0

@Gravity 577 / 571 / 65 Регистрация: 29.01.2009 Сообщений: 1,274
	31.05.2009, 21:55
	Значит ставь и на остальные стртоки. 0