Загрузка файла в массив

@Blueeyer · Регистрация: 09.10.2013

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

Здравствуйте, помогите, при загрузке из файла в массив - выбивается какой-то мусор... в чем дело?

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
#include<iostream>
#include<string>
#include<fstream>
 
using namespace std;
 
struct Animal {
    int code;
    string name;
    double food;
};
 
enum { ADD = 1, NEW, EXIT };
 
void AddAnimals(Animal *animalDB, int size);
void ShowAnimals(Animal *animalDB, int size);
void Menu();
 
int main(){
    int size;
    cout<<"How much animals you want to add? ";
    cin>>size;
    Animal *animalDB = new Animal[size];
    AddAnimals(animalDB, size);
    ShowAnimals(animalDB, size);
// Открываем существующий (или создаем новый) файл, для записи элементов массива в него
    ofstream updateFile;
    bool menu = true;
    while (menu){
        int choice;
        Menu();
        cin>>choice;
        switch(choice){
        case ADD : updateFile.open("Animals.db", ios::out, ios::binary);
            menu = false;
            break;
        case NEW : updateFile.open("Animals.db", ios::app, ios::binary);
            menu = false;
            break;
        case EXIT : cout<<"Good Bye!"<<endl;
            system("PAUSE");
            return 0;
        default : cout<<"Incorrect choice! Try again."<<endl;
        }
    }
    updateFile.write((char*) animalDB, sizeof(Animal)*size);
    updateFile.close();
//Открываем файл для чтения, если он существует
    ifstream showFile;
    if (showFile){
        showFile.open("Animals.db", ios::binary);
        showFile.seekg(0, ios::end);
        int fileSize = showFile.tellg();
        int arrSize = fileSize/sizeof(Animal);
        cout<<endl<<endl<<fileSize<<endl<<endl;
        Animal *animalDBtest = new Animal[arrSize];
        showFile.read((char*) animalDBtest, sizeof(Animal)*fileSize);
        ShowAnimals(animalDBtest, arrSize);
        showFile.close();
    } else cout<<"Can't find file \"Animals.db\""<<endl;
    system("PAUSE");
    return 0;
}
 
void ShowAnimals(Animal *animalDB, int size){
    for(int i = 0; i < size; i++){
        cout<<animalDB[i].name<<endl;
        cout<<animalDB[i].code<<endl;
        cout<<animalDB[i].food<<endl;
    }
}
 
void AddAnimals(Animal *animalDB, int size){
    for(int i = 0; i < size; i++){
        cout<<"Pet nr."<<i+1<<": "<<endl;
        cout<<endl<<"Enter pets name: ";
        cin>>animalDB[i].name;
        cout<<endl<<"Enter pets code: ";
        cin>>animalDB[i].code;
        cout<<endl<<"Enter how much pet eat per day: ";
        cin>>animalDB[i].food;
    }
}
 
void Menu(){
    cout<<"If you want to create new database - press: \"1\""<<endl;
    cout<<"If you want to add animals in exciting database - press: \"2\""<<endl;
    cout<<"If you want to exit - press: \"3\""<<endl;
    cout<<"Your choice: ";
}

Добавлено через 39 секунд
проблема где-то с 49 по 59 строки

alsav22 · 08.04.2014, 00:32

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

C++
1
2
3
4
...
ifstream showFile;
if (showFile){
...

Это не проверка существования файла. Проверяйте после открытия.

Добавлено через 1 минуту

C++
1
2
3
ifstream showFile("Animals.db", ios::binary);
if (showFile){
...

@Blueeyer · 08.04.2014, 00:41 **[ТС]**

да, случайно ошибся, но почему в массив записывается какой-то мусор?

alsav22 · 08.04.2014, 00:50

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

да, случайно ошибся, но почему в массив записывается какой-то мусор?

То есть, код изменили, ошибки открытия файла нет?

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

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

showFile.read((char*) animalDBtest, sizeof(Animal)*fileSize);

Тут размер неправильный.

@Blueeyer · 08.04.2014, 00:52 **[ТС]**

С этим всё нормально. Вот я записал в файл 2 переменных типа Animal, у меня создался бинарный файл с ними, затем я снова запустил программу, указал, что хочу добавить 0 животных, затем загрузилось всё и вот с 49 по 59 строку оно должно из файла загрузить в новый массив эти 2 переменные. Но загружается мусор, вроде, если вообще загружается

Почему не правильный? sizeof(Animal) умножается на кол-во переменных этого типа, которое находится в файле. Или я что-то не так понимаю?

alsav22 · 08.04.2014, 00:53

Нужно или fileSize, или sizeof(Animal)*arrSize.

Добавлено через 1 минуту

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

С этим всё нормально.

Ничего там не нормально. fileSize - всё количество байт в файле. Вы его ещё умножаете на размер структуры.

@Blueeyer · 08.04.2014, 00:56 **[ТС]**

Ничего не изменилось...

alsav22 · 08.04.2014, 00:59

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

Ничего не изменилось...

Ничего страшного, ошибки постепенно убираются. Покажите код с исправлениями.

@Blueeyer · 08.04.2014, 01:00 **[ТС]**

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
#include<iostream>
#include<string>
#include<fstream>
 
using namespace std;
 
struct Animal {
    int code;
    string name;
    double food;
};
 
enum { ADD = 1, NEW, EXIT };
 
void AddAnimals(Animal *animalDB, int size);
void ShowAnimals(Animal *animalDB, int size);
void Menu();
 
int main(){
    int size;
    cout<<"How much animals you want to add? ";
    cin>>size;
    Animal *animalDB = new Animal[size];
    AddAnimals(animalDB, size);
    ShowAnimals(animalDB, size);
// Открываем существующий (или создаем новый) файл, для записи элементов массива в него
    ofstream updateFile;
    bool menu = true;
    while (menu){
        int choice;
        Menu();
        cin>>choice;
        switch(choice){
        case ADD : updateFile.open("Animals.db", ios::out, ios::binary);
            menu = false;
            break;
        case NEW : updateFile.open("Animals.db", ios::app, ios::binary);
            menu = false;
            break;
        case EXIT : cout<<"Good Bye!"<<endl;
            system("PAUSE");
            return 0;
        default : cout<<"Incorrect choice! Try again."<<endl;
        }
    }
    updateFile.write((char*) animalDB, sizeof(Animal)*size);
    updateFile.close();
//Открываем файл для чтения, если он существует
    ifstream showFile;
    showFile.open("Animals.db", ios::binary);
    if (showFile){
        showFile.seekg(0, ios::end);
        int arrSize = showFile.tellg()/sizeof(Animal);
        Animal *animalDBtest = new Animal[arrSize];
        showFile.read((char*) animalDBtest, showFile.tellg());
        ShowAnimals(animalDBtest, arrSize);
        showFile.close();
    } else cout<<"Can't find file \"Animals.db\""<<endl;
    system("PAUSE");
    return 0;
}
 
void ShowAnimals(Animal *animalDB, int size){
    for(int i = 0; i < size; i++){
        cout<<animalDB[i].name<<endl;
        cout<<animalDB[i].code<<endl;
        cout<<animalDB[i].food<<endl;
    }
}
 
void AddAnimals(Animal *animalDB, int size){
    for(int i = 0; i < size; i++){
        cout<<"Pet nr."<<i+1<<": "<<endl;
        cout<<endl<<"Enter pets name: ";
        cin>>animalDB[i].name;
        cout<<endl<<"Enter pets code: ";
        cin>>animalDB[i].code;
        cout<<endl<<"Enter how much pet eat per day: ";
        cin>>animalDB[i].food;
    }
}
 
void Menu(){
    cout<<"If you want to create new database - press: \"1\""<<endl;
    cout<<"If you want to add animals in exciting database - press: \"2\""<<endl;
    cout<<"If you want to exit - press: \"3\""<<endl;
    cout<<"Your choice: ";
}

alsav22 · 08.04.2014, 01:05

Только сейчас увидел string в структуре. Для структур, с такими полями, бинарные методы записи/чтения не годятся.

Добавлено через 1 минуту

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

showFile.seekg(0, ios::end);

А обратно, курсор, перед чтением вернуть?

@Blueeyer · 08.04.2014, 01:05 **[ТС]**

тоесть необходим массив чаров или как ещё можно это всё сделать?

Добавлено через 35 секунд
alsav22, упс

alsav22 · 08.04.2014, 01:06

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

тоесть необходим массив чаров или как ещё можно это всё сделать?

Или массив char, или текстовые методы использовать.

@Blueeyer · 08.04.2014, 01:08 **[ТС]**

а можете помочь исправить это?

alsav22 · 08.04.2014, 01:08

Вот тут тему почитайте:
Запись и чтение структуры с файла

@Blueeyer · 08.04.2014, 01:12 **[ТС]**

спасибо большое, переставил указатель назад и всё заработало, странно, но даже со string

alsav22 · 08.04.2014, 01:13

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

а можете помочь исправить это?

Тут особо помогать нечего. Для записи string, int, double - использовать <<, для чтния int, double - >>, для чтания string - getline().

Добавлено через 1 минуту

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

странно, но даже со string

Нет, это кривой код будет. Когда будет работать (когда адреса в памяти совпадут), когда не будет.

@Blueeyer · 08.04.2014, 01:15 **[ТС]**

тоесть мне string name поменять на char name[30] к примеру?

alsav22 · 08.04.2014, 01:16

Дело в том, что в памяти под string (которую вы записываете в файл), обычно, нет самой строки (она где-нибудь в динамической памяти).

Добавлено через 20 секунд

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

тоесть мне string name поменять на char name[30] к примеру?

Да.

@Blueeyer · 08.04.2014, 01:17 **[ТС]**

спасибо, исправил

Новые блоги и статьи Все статьи Все блоги /
Вывод данных через динамический список в справочнике Maks 01.04.2026 Реализация из решения ниже выполнена на примере нетипового справочника "Спецтехника" разработанного в конфигурации КА2. Задача: вывести данные из ТЧ нетипового документа. . .	Функция заполнения текстового поля в реквизите формы документа Maks 01.04.2026 Алгоритм из решения ниже реализован на нетиповом документе "ВыдачаОборудованияНаСпецтехнику" разработанного в конфигурации КА2, в дополнении к предыдущему решению. На форме документа создается. . .	К слову об оптимизации kumehtar 01.04.2026 Вспоминаю начало 2000-х, университет, когда я писал на Delphi. Тогда среди программистов на форумах активно обсуждали аккуратную работу с памятью: нужно было следить за переменными, вовремя. . .	Идея фильтра интернета (сервер = слой+фильтр). Hrethgir 31.03.2026 Суть идеи заключается в том, чтобы запустить свой сервер, о чём я если честно мечтал давно и давно приобрёл книгу как это сделать. Но не было причин его запускать. Очумелые учёные напечатали на. . .
Модель здравосоХранения 6. ESG-повестка и устойчивое развитие; углублённый анализ кадрового бренда anaschu 31.03.2026 В прикрепленном документе раздумья о том, как можно поменять модель в будущем	10 пpимет, которые всегда сбываются Maks 31.03.2026 1. Чтобы, наконец, пришла маршрутка, надо закурить. Если сигарета последняя, маршрутка придет еще до второй затяжки даже вопреки расписанию. 2. Нaдоели зима и снег? Не надо переезжать. Достаточно. . .	Перемещение выделенных строк ТЧ из одного документа в другой Maks 31.03.2026 Реализация из решения ниже выполнена на примере нетипового документа "ВыдачаОборудованияНаСпецтехнику" с единственной табличной частью "ОборудованиеИКомплектующие" разработанного в конфигурации КА2. . . .	Functional First Web Framework Suave DevAlt 30.03.2026 Sauve. IO Апнулись до NET10. Из зависимостей один пакет, работает одинаково хорошо как в режиме проекта так и в интерактивном режиме. из сложностей - чисто функциональный подход. Решил. . .

@Blueeyer 12 / 12 / 7 Регистрация: 09.10.2013 Сообщений: 222
	08.04.2014, 00:41 [ТС]
	да, случайно ошибся, но почему в массив записывается какой-то мусор? 0

@Blueeyer 12 / 12 / 7 Регистрация: 09.10.2013 Сообщений: 222
	08.04.2014, 00:52 [ТС]
	С этим всё нормально. Вот я записал в файл 2 переменных типа Animal, у меня создался бинарный файл с ними, затем я снова запустил программу, указал, что хочу добавить 0 животных, затем загрузилось всё и вот с 49 по 59 строку оно должно из файла загрузить в новый массив эти 2 переменные. Но загружается мусор, вроде, если вообще загружается Почему не правильный? sizeof(Animal) умножается на кол-во переменных этого типа, которое находится в файле. Или я что-то не так понимаю? 0

@Blueeyer 12 / 12 / 7 Регистрация: 09.10.2013 Сообщений: 222
	08.04.2014, 00:56 [ТС]
	Ничего не изменилось... 0

@Blueeyer 12 / 12 / 7 Регистрация: 09.10.2013 Сообщений: 222
	08.04.2014, 01:05 [ТС]
	тоесть необходим массив чаров или как ещё можно это всё сделать? Добавлено через 35 секунд alsav22, упс 0

@Blueeyer 12 / 12 / 7 Регистрация: 09.10.2013 Сообщений: 222
	08.04.2014, 01:08 [ТС]
	а можете помочь исправить это? 0

alsav22 5500 / 4895 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	08.04.2014, 01:08
	Сообщение было отмечено Blueeyer как решение Решение Вот тут тему почитайте: Запись и чтение структуры с файла 1

@Blueeyer 12 / 12 / 7 Регистрация: 09.10.2013 Сообщений: 222
	08.04.2014, 01:12 [ТС]
	спасибо большое, переставил указатель назад и всё заработало, странно, но даже со string 0

@Blueeyer 12 / 12 / 7 Регистрация: 09.10.2013 Сообщений: 222
	08.04.2014, 01:15 [ТС]
	тоесть мне string name поменять на char name[30] к примеру? 0

@Blueeyer 12 / 12 / 7 Регистрация: 09.10.2013 Сообщений: 222
	08.04.2014, 01:17 [ТС]
	спасибо, исправил 0

Загрузка файла в массив

Решение