How about now ?Has the code become a little more easy to read(I need a second opinion)?
Code:
#include<iostream>
#include<string>
#include<fstream>
#include<stdlib.h>
using namespace std;
const int N=50;
const string name="koli.dat";
struct kola{
string kusha;
string marka;
int godina;
double cena;
fstream fp;
};
void add_kola();
void reading (kola mas[],int &br);
void output(kola mas[],int br);
int search(kola mas[],int br,string mar);
void sort (kola mas[],int br,string mar,int y);
void menu()
void main()
{kola car[N],int br(0);
int ans;
do{menu;
do{cin>>ans;
}
}while(a<1||a>6);
switch(ans){
case 1:add_kola();break;
case 2:reading (car,br);break;
case 3:output(car,br);break;
case 4:if(tarsene(car,br,"Pego"))
cout<<endl<<"Found";
else
cout<<endl<<"Not Found";
case 5:sort(car,br,,"Pego",2008);break;
case 6:cout<<endl<<"End";
exit(1);
}
while(ans!=6);
}
void add_kola()
{kola k;
cout<<endl<<"Add carshop";
cin>>k.kusha;
cout<<endl<<"Add marka";
cin>>k.marka;
cout<<endl<<"Add price";
cin>>k.cena;
fp.open(name,ios::binary;ios::app);
if(fp.fail))
{
cout<<endl<<"Error";
exit(0);
}
{
fp.write((char*)&k,sizeof(kola))
fp.close));
}
}
void reading (kola mas[],int &br)
{fp.open (name,ios::binary;ios::ate);
br=fp.tellg()/sizeof(kola);
fp.seeg(ol;ios::beg);
fp.read(dae*) mas,br*sizeof(kola))l
fp.close();
}
void output(kola mas[],int br)
{for(i=0;i<br;i++)
cout<"Carshop"<<mas.kusha[i].;
cout<"Brand"<<mas.[i].marka;
cout<"Year"<<mas.[i].godina;
cout<"Price"<<mas.[i].cena;
}
int search(kola mas[],int br,string mar)
{for (int i=0;i<br;i++)
if (mas[i].marka=mar)
return 1;
return 0;
}
void sort(kola mas[],int br,string max,int y)
{kola kk[N];int l(0);
for(int i=0;i<br;i++)
if(mas[i].mark=mar&&mas[i].godina==y)
{kk[l]=mas[i];
l++;
}
}