произвольный доступ к файлам

[StudentPolitech]

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

#include<iostream.h>
#include<conio.h>
#include <fstream.h>

struct proga
{
int nomer;
int ocenka;
char familia[10];
};
proga a;
proga b;
proga c;
int main()
{
clrscr();
ofstream f1("a.bar",ios::binary);
int kol=0;
for(int i=1; i<=10; i++)
{
kol=kol++;
cout<<kol<<" "<<"student"<<endl;
//cout<<"Vvedite nomer"<<endl;
a.nomer=kol;
cout<<"Vvedite familiy"<<endl;
cin>>a.familia;
cout<<"Vvedite ocenky"<<endl;
cin>>a.ocenka;
clrscr();
f1.write((char *) &a, sizeof (a));}
f1.close();

ifstream f2("a.bar",ios::binary);

for( i=1;i<=10;i++)
{
f2.read((char*) &a, sizeof( a));
cout<<"Nomer "<<a.nomer<<endl;
cout<<"Familia "<<a.familia<<endl;
cout<<"Ocenka "<<a.ocenka<<endl;
cout<<endl;
}
f2.close();
getch();
clrscr();


ifstream f3("a.bar", ios::ate |ios::in | ios::binary);
ofstream f4("a.bar", ios::ate| ios:ut| ios::binary);



for(int j=0; j<9; j++)
{
f3.open("a.bar", ios::ate | ios::binary);
f4.open("a.bar", ios::ate| ios::binary);
//f3.seekg(0);
//f4.seekp(0);
for(i=0;i<9; i++)
{
f3.seekg((i)*sizeof(a));
f3.read((char*) &a, sizeof a);
f3.read((char*) &b, sizeof b);
if(a.ocenka<b.ocenka)
{
c.nomer=a.nomer;
for(int k=0;k<10;k++) {
c.familia[k]=a.familia[k]; }
c.ocenka=a.ocenka;

a.nomer=b.nomer;
for(k=0;k<10;k++)
{
a.familia[k]=b.familia[k];
}
a.ocenka=b.ocenka;
b.nomer=c.nomer;
for( k=0;k<10;k++){
b.familia[k]=c.familia[k]; }
b.ocenka=c.ocenka;
}
//f3.seekg((i)*sizeof(a));
f4.seekp((i)*sizeof(a));
f4.write((char *) &a, sizeof(a));
f4.write((char *) &b, sizeof(b));
}
//f3.seekg(0);
//f4.seekp(0);
f3.close();
f4.close();
}

cout<<"sortirovka okonchena"<<endl;
getch();
clrscr();
f2.open("a.bar",ios::binary);
for( i=0;i<10;i++)
{
f2.read((char*) &a, sizeof(a));
//f2.seekg(0);
//if(f2.eof()){cout<<"file pustoy"<<endl; getch();clrscr(); return 0;}

cout<<"Nomer "<<a.nomer<<endl;
cout<<"Familia "<<a.familia<<endl;
cout<<"Ocenka "<<a.ocenka<<endl;
cout<<endl;
}
f2.close();
getch();

return 0;
}

Модератор: тег CODE

[Sazary]

А что оно делать-то должно?
Из того, что успел заметить: у вас локальные переменные используются и дальше в программе (после того, как уже перестанут существовать). Это касается практически всех циклов. Или сделайте их глобальными, или объявляйте каждый раз в циклах.

Если программе нужен какой-то исходный файл (с данными, которые он считывает), то приложите его.

[ROD]

Нет. "Данный файл" программа создает сама. (ну это тут не столь важно).

Вот выражаю мысль по нормальному.

Дан файл. В файле записанны структуры
struct student

{
int nomer;
int ocenka;
char familia[10];
};

В файле таких структур 10(десять).

Необходимо отсортировать студентов по убыванию их оценки (5,4,3,2,1).
Целиком считывать файл в оперативку нельзя.


Проьблема возникает в функции сортировки. В результате в файле оказывается черт знает что.

Вот программа, которую мы, вместе с моим коллегой, составили:

#include <iostream.h>
#include <conio.h>
#include <fstream.h>

struct proga//это структура, типа "студент"
{
int nomer;
int ocenka;
char familia[10];
};
proga a;
proga b;
proga c;
int main()
{
clrscr();
ofstream f1("a.bar",ios::binary);
int kol=0;//файл, который "дан" создаем и заполняем тут.
for(int i=1; i<=10; i++)
{
kol=kol++;
cout<<kol<<" "<<"student"<<endl;
//cout<<"Vvedite nomer"<<endl;
a.nomer=kol;
cout<<"Vvedite familiy"<<endl;
cin>>a.familia;
cout<<"Vvedite ocenky"<<endl;
cin>>a.ocenka;
clrscr();
f1.write((char *) &a, sizeof (a));}
f1.close();

ifstream f2("a.bar",ios::binary);//выводим его

for(int i=1;i<=10;i++)
{
f2.read((char*) &a, sizeof( a));
cout<<"Nomer "<<a.nomer<<endl;
cout<<"Familia "<<a.familia<<endl;
cout<<"Ocenka "<<a.ocenka<<endl;
cout<<endl;
}
f2.close();
getch();
clrscr();

//начинаем сортировку по убыванию
ifstream f3("a.bar", ios::ate |ios::in | ios::binary);
ofstream f4("a.bar", ios::ate| ios::out| ios::binary);


for(int j=0; j<9; j++)
	{

	for(int i=0;i<9; i++)
		{
		f3.seekg((i)*sizeof(a), ios::beg);
		f3.read((char*) &a, sizeof a);
		f3.read((char*) &b, sizeof b);
		if(a.ocenka<b.ocenka)
		{
		c.nomer=a.nomer;
		for(int k=0;k<10;k++)
      	{
			c.familia[k]=a.familia[k];
         }
      c.ocenka=a.ocenka;
		a.nomer=b.nomer;
		for(int k=0;k<10;k++)
			{
			a.familia[k]=b.familia[k];
			}
		a.ocenka=b.ocenka;
   	b.nomer=c.nomer;
      for(int k=0;k<10;k++)
      	{
			b.familia[k]=c.familia[k];
         }
		b.ocenka=c.ocenka;
		}
   f3.seekg(i*sizeof(a), ios::beg);
	f4.seekp(i*sizeof(a), ios::beg);
	f4.write((char *) &a, sizeof(a));
	f4.write((char *) &b, sizeof(b));
	}
	f3.seekg(0, ios::beg);
	f4.seekp(0, ios::beg);
	}
f3.close();
f4.close();

cout<<"sortirovka okonchena"<<endl;
//ну а дальше выводим все на экран, и видим не верные результаты.
getch();
clrscr();
f2.open("a.bar",ios::binary);
for(int i=0;i<10;i++)
{
f2.read((char*) &a, sizeof(a));
f2.seekg(0);
if(f2.eof()){cout<<"file pustoy"<<endl; getch();clrscr(); return 0;}

cout<<"Nomer "<<a.nomer<<endl;
cout<<"Familia "<<a.familia<<endl;
cout<<"Ocenka "<<a.ocenka<<endl;
cout<<endl;
}
f2.close();
getch();

return 0;
 }

[offtop]Только тема задания, не ввод-вывод, а , скорее, произвольный доступ.[/offtop]

[Sazary]

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

#include <iostream.h>
#include <conio.h>
#include <fstream.h>

struct proga//это структура, типа "студент"
{
int nomer;
int ocenka;
char familia[10];
};

const int N=4;

int main()
{
proga a,b;
fstream f;
f.open("a.bar",ios::binary|ios::out);
int kol=0;//файл, который "дан" создаем и заполняем тут.
for(int i=0; i<N; i++)
{
 kol++;
 cout<<kol<<" "<<"student"<<endl;
 a.nomer=kol;
 cout<<"Vvedite familiy"<<endl;
 _flushall();
 cin.getline(a.familia,10,'\n');
 cout<<"Vvedite ocenky"<<endl;
 cin>>a.ocenka;
 
 f.seekp(i*sizeof(struct proga),ios::beg);
 f.write((char*)&a, sizeof(struct proga));
 }
f.close();

f.open("a.bar",ios::binary|ios::in);//выводим его

cout<<"\n------------------------------\n\n";
for(int i=0;i<N;i++)
{
f.seekg(i*sizeof(struct proga),ios::beg);
f.read((char*)&a, sizeof(struct proga));
//cout<<"Nomer "<<a.nomer<<endl;
//cout<<"Familia "<<a.familia<<endl;
//cout<<"Ocenka "<<a.ocenka<<endl;
//cout<<endl;
cout<<a.nomer<<"\t"<<a.familia<<"\t"<<a.ocenka<<endl;
}
f.close();

getch();
cout<<"\n=====================\n\n";

cout<<endl;
//начинаем сортировку по убыванию

f.open("a.bar", ios::in | ios::out| ios::binary);

int k;
int tmp;
proga c;
for(int i=0; i<N-1; i++)
	{
 f.seekg(i*sizeof(struct proga),ios::beg);
 f.read((char*)&a, sizeof(struct proga)); 
	k = i;
 c = a;
 for(int j=i+1;j<N; j++)
		{  
		 f.seekg(j*sizeof(struct proga),ios::beg);
		 f.read((char*)&b,sizeof(struct proga));
   if(b.ocenka>c.ocenka) 
   {
    k = j;
    c = b;
   }
  }
		if(k!=i)
		 {
    f.seekp(i*sizeof(struct proga),ios::beg);
    f.write((char*)&c,sizeof(struct proga));
    f.seekp(k*sizeof(struct proga),ios::beg);
    f.write((char*)&a,sizeof(struct proga));
   }	
  
	}
f.close();


cout<<"sortirovka okonchena"<<endl;

getch();


f.open("a.bar",ios::binary|ios::in);
for(int i=0;i<N;i++)
{
 f.seekg(i*sizeof(struct proga),ios::beg);
 f.read((char*)&a, sizeof(struct proga));

//cout<<"Nomer "<<a.nomer<<endl;
//cout<<"Familia "<<a.familia<<endl;
//cout<<"Ocenka "<<a.ocenka<<endl;
 cout<<a.nomer<<"\t"<<a.familia<<"\t"<<a.ocenka<<endl;
 }
f.close();

getch();

return 0;
}

[ROD]

sizeof(struct proga)

Вопрос по поводу этой записи. Почему нельзя написать просто sizeof(proga) ?

P.S.

К стати, вывод красиво сделан.

[Sazary]

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

Почему нельзя написать просто sizeof(proga) ?

Можно Но так более...наглядно что-ли.

[StudentPolitech]

Извините вопрос зачем вот это _flushall(); если можно было просто написать cin>>familia;

[Sazary]

_flushall() - очистка буферов.
В принципе, можете и так написать..