Мусор в файлах

@maxim43k · Регистрация: 04.09.2011

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

В файлы попадает какой-то мусор, в чём ошибка?

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
#include <iostream.h>
#include <fstream.h>
#include <string.h>
#include <stdlib.h>
#include <iomanip.h>
#include <conio.h>
 
 struct time_s
{
   int min;
   int sec;
};
 
 struct athlete
{
   char name [20];
   char f_name [20];
   int age;
   time_s time;
};
 
void process(athlete* &all, int count, char *f1_name, char *f2_name);
 
 void main()
{
   athlete *all = NULL;
   char f1_name[20], f2_name[20];
   int count, min;
   count = min = 0;
   clrscr();
   cout << "Введите имя файла F1: ";
   cin >> f1_name;
   cout << "Введите имя файла F2 (бинарный): ";
   cin >> f2_name;
   cout << "Введите количество спортсменов: ";
   cin >> count;
   process(all, count, f1_name, f2_name);
   getch();
}
 
 void process(athlete* &all, int count, char *f1_name, char *f2_name)
{
   int i, n, min = 0;
   all = new athlete[count];
   fstream f1(f1_name, ios::in | ios::out);
   if (f1.fail())
     {
       cout << "F1 Ошибка!" << f1.rdstate() << endl;
     }
   fstream f2(f2_name, ios::in | ios::binary | ios::out);
   if (f2.fail())
     {
       cout << "F2 Ошибка!" << f2.rdstate() << endl;
     }
   cout << "Введите данные о спортсменах\n"
         "(имя, фамилия, возраст, результат в мин. и сек. через пробел):\n";
   for (i = 0; i < count; ++i)
     {
        cout << i + 1 << ". ";
        cin >> all[i].name;
        cin >> all[i].f_name;
        cin >> all[i].age;
        cin >> all[i].time.min;
        cin >> all[i].time.sec;
     }
  
   f1 << "\nПроверка считывания из F2: \n";
   f2.seekg(0,ios::beg);
   for(i=0; i<n; i++)
   f2.write((char *)&all[i], size_d);
   f2.seekp(0, ios::beg);
   for(i=0;i<n;i++)
     {
       f2.read((char *)&all[i], size_d);
       f1 << all[i].name << " ";  
       f1 << all[i].f_name << " ";
       f1 << all[i].age << " ";
       f1 << all[i].time.min << " ";
       f1 << all[i].time.sec << " " << endl;
     }
   for (int k = 0; k < count; k++)
   if (k == 0) min = k;
   else if (all[k].age < all[min].age) min = k;
         cout << "\nСамый юный участник:\n";
         cout << "Порядковый номер в реестре: " << min 
                << "\nИмя: " << all[min - 1].name 
                << "\nФамилия: " << all[min - 1].f_name
                << "\nВозраст: " << all[min - 1].age 
                << "\nРезультат: " << all[min - 1].time.min 
                << " мин. " << all[min - 1].time.sec << " сек. ";
 
   ofstream file("F2.bin", ios::out | ios::binary);
   file.write((char *)&all[min], sizeof(athlete)); 
   f1.close();
   f2.close();
}

accept · 09.09.2011, 10:40

size_d - это что ?
сначала выведи на экран то, что собрался в файл выводить

@-=ЮрА=- · 09.09.2011, 10:45

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

f2.write((char *)&all[i], size_d);

size_d - не определён, читаешь левое число символов, вместо size_d вставь sizeof(athlete)

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

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

f2.read((char *)&all[i], size_d);

- тоже

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

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

file.write((char *)&all[min], sizeof(athlete));

- тут уже правильно писал

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

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

for(i=0;i<n;i++)

cpp(69) : warning C4700: local variable 'n' used without having been initialized - как вообще, что то могло писаться, запись просто не происходила!!!

@maxim43k · 09.09.2011, 10:51 **[ТС]**

Аха, сейчас в файл пишет уже нормально, но ниже данных появляется текстовый мусор под 3 Мб

@-=ЮрА=- · 09.09.2011, 10:56

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

Аха, сейчас в файл пишет уже нормально, но ниже данных появляется текстовый мусор под 3 Мб

-

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

ofstream file("F2.bin", ios::out | ios::binary);

- не вижу чтобы закрывался этот поток, file.close(); не забыл???

@maxim43k · 09.09.2011, 11:00 **[ТС]**

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

f1.close();
f2.close();

А это не закрывает поток?

@Deviaphan · 09.09.2011, 11:03

Сообщение от -=ЮрА=-

не вижу чтобы закрывался этот поток, file.close(); не забыл???

В данном случае даже f1.close() и f2.close() вызывать не обязательно... Норм тут всё с закрытием.

@-=ЮрА=- · 09.09.2011, 11:04

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

А это не закрывает поток?

- ты же ещё 3-й поток создал,

C++
1
ofstream file("F2.bin", ios::out | ios::binary);

неужели не видишь что не закрыл его

@Deviaphan · 09.09.2011, 11:05

Тем более, что f1.close(); f2.close() не правильно с концептуальной точки зрения. Нужно писать f2.close(); f1.close(). Т.е. так, как это будет происходить при естественном закрытии потоков при помощи деструктора - в порядке, обратном созданию.

@-=ЮрА=- · 09.09.2011, 11:08

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

В данном случае даже f1.close() и f2.close() вызывать не обязательно... Норм тут всё с закрытием.

- 3-й поток закрыть надо!Любишь работать с открытыми потоками - работай, рано или поздно наступишь на грабли из за того что их не завершаешь!

Добавлено через 3 минуты
maxim43k, вбей file.close + проверь чтобы здесь

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

for (int k = 0; k < count; k++)
* *if (k == 0) min = k;
* *else if (all[k].age < all[min].age) min = k;

min не превысило count, иначе в файл будешь писать неинициализированную структуру all[min]

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

file.write((char *)&all[min], sizeof(athlete));

@Deviaphan · 09.09.2011, 11:09

Сообщение от -=ЮрА=-

3-й поток закрыть надо!

Он закрывается автоматически. Делать нужно только то, что нужно. Не более. Вызывать метод, который будет вызван автоматически на следующей строчке - излишне.
Но я с тобой полностью согласен, открытые потоки оставлять нельзя.

@-=ЮрА=- · 09.09.2011, 11:11

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

Он закрывается автоматически. Делать нужно только то, что нужно. Не более. Вызывать метод, который будет вызван автоматически на следующей строчке - излишне.
Но я с тобой полностью согласен, открытые потоки оставлять нельзя.

Хорошо открой вконце мэйна F2.bin для чтения...

@Deviaphan · 09.09.2011, 11:14

Сообщение от -=ЮрА=-

Хорошо открой вконце мэйна F2.bin для чтения...

Поток file закрывается при выходе из process. Если у тебя он остался открыт, то перестань уже использовать морально устаревшие, не корректные библиотеки.

Добавлено через 40 секунд
Упс. Это ТС их использует...

@maxim43k · 09.09.2011, 11:15 **[ТС]**

Пишу так, в итоге ошибка и всё закрывается

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
#include <iostream.h>
#include <fstream.h>
#include <string.h>
#include <stdlib.h>
#include <iomanip.h>
#include <conio.h>
 
 struct time_s
{
   int min;
   int sec;
};
 
 struct athlete
{
   char name [20];
   char f_name [20];
   int age;
   time_s time;
};
 
void process(athlete* &all, int count, char *f1_name, char *f2_name);
 
 void main()
{
   athlete *all = NULL;
   char f1_name[20], f2_name[20];
   int count, min;
   count = min = 0;
   clrscr();
   cout << "Введите имя файла F1: ";
   cin >> f1_name;
   cout << "Введите имя файла F2 (бинарный): ";
   cin >> f2_name;
   cout << "Введите количество спортсменов: ";
   cin >> count;
   process(all, count, f1_name, f2_name);
   getch();
}
 
 void process(athlete* &all, int count, char *f1_name, char *f2_name)
{
   int i, n, min = 0;
   all = new athlete[count];
   fstream f1(f1_name, ios::in | ios::out);
   if (f1.fail())
     {
       cout << "F1 Ошибка!" << f1.rdstate() << endl;
     }
   fstream f2(f2_name, ios::in | ios::binary | ios::out);
   if (f2.fail())
     {
       cout << "F2 Ошибка!" << f2.rdstate() << endl;
     }
   cout << "Введите данные о спортсменах\n"
         "(имя, фамилия, возраст, результат в мин. и сек. через пробел):\n";
   for (i = 0; i < count; ++i)
     {
        cout << i + 1 << ". ";
        cin >> all[i].name;
        cin >> all[i].f_name;
        cin >> all[i].age;
        cin >> all[i].time.min;
        cin >> all[i].time.sec;
     }
  
   f1 << "\nПроверка считывания из F2: \n";
   f2.seekg(0,ios::beg);
   for(i=0; i<n; i++)
   f2.write((char *)&all[i], sizeof(athlete));
   f2.seekp(0, ios::beg);
   for(i=0;i<n;i++)
     {
       f2.read((char *)&all[i], sizeof(athlete));
       f1 << all[i].name << " ";  
       f1 << all[i].f_name << " ";
       f1 << all[i].age << " ";
       f1 << all[i].time.min << " ";
       f1 << all[i].time.sec << " " << endl;
     }
   for (int k = 0; k < count; k++)
   if (k == 0) min = k;
   else if (all[k].age < all[min].age) min = k;
         cout << "\nСамый юный участник:\n";
         cout << "Порядковый номер в реестре: " << min 
                << "\nИмя: " << all[min - 1].name 
                << "\nФамилия: " << all[min - 1].f_name
                << "\nВозраст: " << all[min - 1].age 
                << "\nРезультат: " << all[min - 1].time.min 
                << " мин. " << all[min - 1].time.sec << " сек. ";
 
   ofstream file("F2.bin", ios::out | ios::binary);
   file.close();
   file.write((char *)&all[min], sizeof(athlete)); 
   
   f2.close();
   f1.close();
}

@Deviaphan · 09.09.2011, 11:16

maxim43k - перестань использовать морально устаревшие, не корректные компиляторы и их библиотеки!

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

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

Пишу так

93-94 местами поменяй.)

@-=ЮрА=- · 09.09.2011, 11:18

maxim43k, ввёл 1 запись и в коде записал так

C++
1
2
3
4
5
ofstream file("F2.bin", ios::out | ios::binary);
   file.write((char *)&all[min], sizeof(athlete)); 
   f1.close();
   f2.close();
   file.close();

На миниатюре скрин записанного в файлы, как видишь нет ничего лишнего, если слушаешь Deviaphan то я умываю руки...

@maxim43k · 09.09.2011, 11:18 **[ТС]**

Всё равно ошибка

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
#include <iostream.h>
#include <fstream.h>
#include <string.h>
#include <stdlib.h>
#include <iomanip.h>
#include <conio.h>
 
 struct time_s
{
   int min;
   int sec;
};
 
 struct athlete
{
   char name [20];
   char f_name [20];
   int age;
   time_s time;
};
 
void process(athlete* &all, int count, char *f1_name, char *f2_name);
 
 void main()
{
   athlete *all = NULL;
   char f1_name[20], f2_name[20];
   int count, min;
   count = min = 0;
   clrscr();
   cout << "Введите имя файла F1: ";
   cin >> f1_name;
   cout << "Введите имя файла F2 (бинарный): ";
   cin >> f2_name;
   cout << "Введите количество спортсменов: ";
   cin >> count;
   process(all, count, f1_name, f2_name);
   getch();
}
 
 void process(athlete* &all, int count, char *f1_name, char *f2_name)
{
   int i, n, min = 0;
   all = new athlete[count];
   fstream f1(f1_name, ios::in | ios::out);
   if (f1.fail())
     {
       cout << "F1 Ошибка!" << f1.rdstate() << endl;
     }
   fstream f2(f2_name, ios::in | ios::binary | ios::out);
   if (f2.fail())
     {
       cout << "F2 Ошибка!" << f2.rdstate() << endl;
     }
   cout << "Введите данные о спортсменах\n"
         "(имя, фамилия, возраст, результат в мин. и сек. через пробел):\n";
   for (i = 0; i < count; ++i)
     {
        cout << i + 1 << ". ";
        cin >> all[i].name;
        cin >> all[i].f_name;
        cin >> all[i].age;
        cin >> all[i].time.min;
        cin >> all[i].time.sec;
     }
  
   f1 << "\nПроверка считывания из F2: \n";
   f2.seekg(0,ios::beg);
   for(i=0; i<n; i++)
   f2.write((char *)&all[i], sizeof(athlete));
   f2.seekp(0, ios::beg);
   for(i=0;i<n;i++)
     {
       f2.read((char *)&all[i], sizeof(athlete));
       f1 << all[i].name << " ";  
       f1 << all[i].f_name << " ";
       f1 << all[i].age << " ";
       f1 << all[i].time.min << " ";
       f1 << all[i].time.sec << " " << endl;
     }
   for (int k = 0; k < count; k++)
   if (k == 0) min = k;
   else if (all[k].age < all[min].age) min = k;
         cout << "\nСамый юный участник:\n";
         cout << "Порядковый номер в реестре: " << min 
                << "\nИмя: " << all[min - 1].name 
                << "\nФамилия: " << all[min - 1].f_name
                << "\nВозраст: " << all[min - 1].age 
                << "\nРезультат: " << all[min - 1].time.min 
                << " мин. " << all[min - 1].time.sec << " сек. ";
 
   ofstream file("F2.bin", ios::out | ios::binary);
   file.write((char *)&all[min], sizeof(athlete)); 
   file.close();
   f2.close();
   f1.close();
}

@-=ЮрА=- · 09.09.2011, 11:21

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

Всё равно ошибка

- сейчас прогоню твой код, введу 3 участника, жди...

@Deviaphan · 09.09.2011, 11:21

Не по теме:

Borland C++ for DOS... ну вашу ж мать!

@-=ЮрА=- · 09.09.2011, 11:23

D:\Мои документы\ЮрА\Личное\PROJECTS\t2\t2.cpp( 88) : warning C4700: local variable 'n' used without having been initialized

Добавлено через 1 минуту
maxim43k, 72-ая строка n - не инициализировано, поставь count вместо n

Новые блоги и статьи Все статьи Все блоги /
Знаешь почему 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. Задача: реализовать контроль корректности заполнения дат назначения. . .	Архитектура слоя интернета для сервера-слоя. Hrethgir 11.04.2026 В продолжение https:/ / www. cyberforum. ru/ blogs/ 223907/ 10860. html Знаешь что я подумал? Раз мы все источники пишем в голове ветки, то ничего не мешает добавить в голову такой источник, который сам. . .	Подстановка значения реквизита справочника в табличную часть документа Maks 10.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеПерсонала", разработанного в конфигурации КА2. Задача: при выборе сотрудника (справочник Сотрудники) в ТЧ документа. . .	Очистка реквизитов документа при копировании Maks 09.04.2026 Алгоритм из решения ниже применим как для типовых, так и для нетиповых документов на самых различных конфигурациях. Задача: при копировании документа очищать определенные реквизиты и табличную. . .

accept 4866 / 3287 / 468 Регистрация: 10.12.2008 Сообщений: 10,570
	09.09.2011, 10:40
	size_d - это что ? сначала выведи на экран то, что собрался в файл выводить 1

@maxim43k 0 / 0 / 0 Регистрация: 04.09.2011 Сообщений: 106
	09.09.2011, 10:51 [ТС]
	Аха, сейчас в файл пишет уже нормально, но ниже данных появляется текстовый мусор под 3 Мб 0

@Deviaphan Делаю внезапно и красиво 1313 / 1228 / 72 Регистрация: 22.03.2011 Сообщений: 3,744
	09.09.2011, 11:05
	Тем более, что f1.close(); f2.close() не правильно с концептуальной точки зрения. Нужно писать f2.close(); f1.close(). Т.е. так, как это будет происходить при естественном закрытии потоков при помощи деструктора - в порядке, обратном созданию. 0

@Deviaphan
	09.09.2011, 11:21
	Не по теме: Borland C++ for DOS... ну вашу ж мать! 0

@-=ЮрА=- 6614 / 4256 / 401 Регистрация: 08.08.2009 Сообщений: 10,325 Записей в блоге: 24
	09.09.2011, 11:23
	D:\Мои документы\ЮрА\Личное\PROJECTS\t2\t2.cpp( 88) : warning C4700: local variable 'n' used without having been initialized Добавлено через 1 минуту maxim43k, 72-ая строка n - не инициализировано, поставь count вместо n 0