Форум программистов, компьютерный форум CyberForum.ru

Мусор в файлах - C++

Восстановить пароль Регистрация
 
 
Рейтинг: Рейтинг темы: голосов - 12, средняя оценка - 4.92
maxim43k
0 / 0 / 0
Регистрация: 04.09.2011
Сообщений: 106
09.09.2011, 10:23     Мусор в файлах #1
В файлы попадает какой-то мусор, в чём ошибка?

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();
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
09.09.2011, 10:23     Мусор в файлах
Посмотрите здесь:

C++ Мусор в строках
Мусор C++
C++ Мусор в строке
C++ Мусор в масивах char
Мусор в переменной C++
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
accept
4838 / 3237 / 165
Регистрация: 10.12.2008
Сообщений: 10,682
09.09.2011, 10:40     Мусор в файлах #2
size_d - это что ?
сначала выведи на экран то, что собрался в файл выводить
-=ЮрА=-
Заблокирован
Автор FAQ
09.09.2011, 10:45     Мусор в файлах #3
Цитата Сообщение от 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
0 / 0 / 0
Регистрация: 04.09.2011
Сообщений: 106
09.09.2011, 10:51  [ТС]     Мусор в файлах #4
Аха, сейчас в файл пишет уже нормально, но ниже данных появляется текстовый мусор под 3 Мб
-=ЮрА=-
Заблокирован
Автор FAQ
09.09.2011, 10:56     Мусор в файлах #5
Цитата Сообщение от maxim43k Посмотреть сообщение
Аха, сейчас в файл пишет уже нормально, но ниже данных появляется текстовый мусор под 3 Мб
-
Цитата Сообщение от maxim43k Посмотреть сообщение
ofstream file("F2.bin", ios::out | ios::binary);
- не вижу чтобы закрывался этот поток, file.close(); не забыл???
maxim43k
0 / 0 / 0
Регистрация: 04.09.2011
Сообщений: 106
09.09.2011, 11:00  [ТС]     Мусор в файлах #6
Цитата Сообщение от maxim43k Посмотреть сообщение
f1.close();
f2.close();
А это не закрывает поток?
Deviaphan
Делаю внезапно и красиво
Эксперт C++
 Аватар для Deviaphan
1283 / 1217 / 50
Регистрация: 22.03.2011
Сообщений: 3,744
09.09.2011, 11:03     Мусор в файлах #7
Цитата Сообщение от -=ЮрА=- Посмотреть сообщение
не вижу чтобы закрывался этот поток, file.close(); не забыл???
В данном случае даже f1.close() и f2.close() вызывать не обязательно... Норм тут всё с закрытием.
-=ЮрА=-
Заблокирован
Автор FAQ
09.09.2011, 11:04     Мусор в файлах #8
Цитата Сообщение от maxim43k Посмотреть сообщение
А это не закрывает поток?
- ты же ещё 3-й поток создал,
C++
1
ofstream file("F2.bin", ios::out | ios::binary);
неужели не видишь что не закрыл его
Deviaphan
Делаю внезапно и красиво
Эксперт C++
 Аватар для Deviaphan
1283 / 1217 / 50
Регистрация: 22.03.2011
Сообщений: 3,744
09.09.2011, 11:05     Мусор в файлах #9
Тем более, что f1.close(); f2.close() не правильно с концептуальной точки зрения. Нужно писать f2.close(); f1.close(). Т.е. так, как это будет происходить при естественном закрытии потоков при помощи деструктора - в порядке, обратном созданию.
-=ЮрА=-
Заблокирован
Автор FAQ
09.09.2011, 11:08     Мусор в файлах #10
Цитата Сообщение от 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
Делаю внезапно и красиво
Эксперт C++
 Аватар для Deviaphan
1283 / 1217 / 50
Регистрация: 22.03.2011
Сообщений: 3,744
09.09.2011, 11:09     Мусор в файлах #11
Цитата Сообщение от -=ЮрА=- Посмотреть сообщение
3-й поток закрыть надо!
Он закрывается автоматически. Делать нужно только то, что нужно. Не более. Вызывать метод, который будет вызван автоматически на следующей строчке - излишне.
Но я с тобой полностью согласен, открытые потоки оставлять нельзя.
-=ЮрА=-
Заблокирован
Автор FAQ
09.09.2011, 11:11     Мусор в файлах #12
Цитата Сообщение от Deviaphan Посмотреть сообщение
Он закрывается автоматически. Делать нужно только то, что нужно. Не более. Вызывать метод, который будет вызван автоматически на следующей строчке - излишне.
Но я с тобой полностью согласен, открытые потоки оставлять нельзя.
Хорошо открой вконце мэйна F2.bin для чтения...
Deviaphan
Делаю внезапно и красиво
Эксперт C++
 Аватар для Deviaphan
1283 / 1217 / 50
Регистрация: 22.03.2011
Сообщений: 3,744
09.09.2011, 11:14     Мусор в файлах #13
Цитата Сообщение от -=ЮрА=- Посмотреть сообщение
Хорошо открой вконце мэйна F2.bin для чтения...
Поток file закрывается при выходе из process. Если у тебя он остался открыт, то перестань уже использовать морально устаревшие, не корректные библиотеки.

Добавлено через 40 секунд
Упс. Это ТС их использует...
maxim43k
0 / 0 / 0
Регистрация: 04.09.2011
Сообщений: 106
09.09.2011, 11:15  [ТС]     Мусор в файлах #14
Пишу так, в итоге ошибка и всё закрывается

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
Делаю внезапно и красиво
Эксперт C++
 Аватар для Deviaphan
1283 / 1217 / 50
Регистрация: 22.03.2011
Сообщений: 3,744
09.09.2011, 11:16     Мусор в файлах #15
maxim43k - перестань использовать морально устаревшие, не корректные компиляторы и их библиотеки!

Добавлено через 32 секунды
Цитата Сообщение от maxim43k Посмотреть сообщение
Пишу так
93-94 местами поменяй.)
-=ЮрА=-
Заблокирован
Автор FAQ
09.09.2011, 11:18     Мусор в файлах #16
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
0 / 0 / 0
Регистрация: 04.09.2011
Сообщений: 106
09.09.2011, 11:18  [ТС]     Мусор в файлах #17
Всё равно ошибка

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

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();
}
-=ЮрА=-
Заблокирован
Автор FAQ
09.09.2011, 11:21     Мусор в файлах #18
Цитата Сообщение от maxim43k Посмотреть сообщение
Всё равно ошибка
- сейчас прогоню твой код, введу 3 участника, жди...
Deviaphan
09.09.2011, 11:21
  #19

Не по теме:

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

MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
09.09.2011, 11:23     Мусор в файлах
Еще ссылки по теме:

C++ Мусор в массиве
Мусор и оператор new C++
Мусор в конце строки C++

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
-=ЮрА=-
Заблокирован
Автор FAQ
09.09.2011, 11:23     Мусор в файлах #20
D:\Мои документы\ЮрА\Личное\PROJECTS\t2\t2.cpp(88) : warning C4700: local variable 'n' used without having been initialized

Добавлено через 1 минуту
maxim43k, 72-ая строка n - не инициализировано, поставь count вместо n
Yandex
Объявления
09.09.2011, 11:23     Мусор в файлах
Ответ Создать тему
Опции темы

Текущее время: 15:54. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru