problam with initializer

We did this program in class and i wanted see how will it run on CodeBlocks.But i got an error saying "expected initializer before 'void' on line 24".This is the first time i have this error.I try searching for a solution but didnt find one that will help me explain and figure out this problem.
This is a problem

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++;
}
}

---

Please indent your code properly, and watch out for semi-colons.

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++;
        }
}

---

You still didn't correct his second comment .. reread.

Quote:

---

and watch out for semi-colons

---

and then look at your error

Quote:

---

expected initializer before 'void' on line 24

---

and think what do initializer's end with?

and the indentation is not much improved sorry - don't write {code after braces for one thing

The best thing to do is adhere to some common style, such as Allman. This makes the code much easier to read.
Your code's readability is improved, but I'm afraid it could be a lot better.

This was clearly all written in one go without compiling along the way. That's not how to program. You need to do a little bit at a time, and compile often, especially when you're still a beginner.
E.g http://cboard.cprogramming.com/c-pro...t-process.html