I need help with this error i keep getting about an unmatch brace

[GC2006]

//Error 1 error C2601: 'computeBestHot elBestMonth' : local function definitions are illegal e:\project 3\prj 3b.cpp 90
//Error 2 fatal error C1075: end of file found before the left brace '{' at 'e:\project 3\prj 3b.cpp(68)' was matched e:\project 3\prj 3b.cpp 104

#include <iostream>
#include <fstream>
using namespace std;
const int HOTELS = 5;
const int MONTHS = 12;
void displayGuests(i nt[HOTELS][MONTHS]);
int computeYearlyHo telGuest(int[HOTELS][MONTHS]);
int computeBestHote lInAMonth(int totalguests[HOTELS][MONTHS]);
void computeBestHote lBestMonth(int totalguests[HOTELS][MONTHS], int&, int&);

int main ()
{
ifstream infile;//input file object
infile.open("E: \\hoteldata.txt ");//open the file
int totalguests[HOTELS][MONTHS];
int g[HOTELS][MONTHS];
for (int i = 0; i < 5; i++)
for (int j = 0; j < 12; j++)
infile>>totalgu ests[i][j];
int besthotel;
int bestmonth;

displayGuests(t otalguests);
computeYearlyHo telGuest(totalg uests);
computeBestHote lInAMonth(g);
computeBestHote lBestMonth(tota lguests, besthotel, bestmonth);

return 0;
}
void displayGuests(i nt totalguests[HOTELS][MONTHS])
{

int HOTLES = 5;
int MONTHS = 12;
for (int i=0; i < HOTELS; i++)
{
for (int j=0; j<MONTHS; j++)

cout<<totalgues ts[i][j]<<" ";
cout<<endl;
}

}
int computeYearlyHo telGuest(int totalguests[HOTELS][MONTHS])
{
int num;
int Hotel;
int Months;
int sum;

cout<<"Enter Hotel number 1 - 5: "<<endl;
cin>>num;
switch(num)
{
case 1:
sum = 0;
Hotel = 1;
for (Months = 0; Months < MONTHS; Months++)
{
sum = sum + totalguests[Hotel][Months];
}
cout<<"Yearly Total: "<<sum<<end l;
}

return 0;
}
int computeBestHote lInAMonth(int totalguests[HOTELS][MONTHS])
{
int month;
int hotel;
int bestHotel;
cout<<"Enter the Month:"<<endl;
cin >> month;

switch (month)
{
case 1:
for (int month = 0; month< MONTHS; month++)
{
bestHotel = totalguests[0][month];
for (hotel = 1; hotel < HOTELS; hotel++)
if (bestHotel < totalguests[hotel][month])
bestHotel = totalguests[hotel][month];

cout<<"The best Hotel in January is"<<bestHotel< <endl;
}
return 0;
}
void computeBestHote lBestMonth(int g[HOTELS][MONTHS],int besthotel, int bestmonth)
{
int goodhotel = 0, goodmonth = 0;
for (int i = 0; i<5; i++)
{
for (int j = 0; j<12; j++)
if(g[goodhotel][goodmonth]<g[i][j])
{
goodhotel = i;
goodmonth = j;
}
}
besthotel = goodhotel + 1;
bestmonth = goodmonth + 1;
}

[prakash ravindran]

hi,

The problem is not in the 'computeBestHot elBestMonth' it will work fine. The bug is the 'computeBestHot elInAMonth' function


int computeBestHote lInAMonth(int totalguests[HOTELS][MONTHS])
{
int month;
int hotel;
int bestHotel;
cout<<"Enter the Month:"<<endl;
cin >> month;

switch (month)
{
case 1:
for (int month = 0; month< MONTHS; month++)
{
bestHotel = totalguests[0][month];
for (hotel = 1; hotel < HOTELS; hotel++)
if (bestHotel < totalguests[hotel][month])
bestHotel = totalguests[hotel][month];

cout<<"The best Hotel in January is"<<bestHotel< <endl;
}
return 0;
}


read the code carefully, in this code u had opened a '{' barce for switch statement but that is not closed in right place. So close the brace in correct place that will do....

[thefarmer]

//Error 1 error C2601: 'computeBestHot elBestMonth' : local function definitions are illegal e:\project 3\prj 3b.cpp 90
//Error 2 fatal error C1075: end of file found before the left brace '{' at 'e:\project 3\prj 3b.cpp(68)' was matched e:\project 3\prj 3b.cpp 104

#include <iostream>
#include <fstream>
using namespace std;
const int HOTELS = 5;
const int MONTHS = 12;
void displayGuests(i nt[HOTELS][MONTHS]);
int computeYearlyHo telGuest(int[HOTELS][MONTHS]);
int computeBestHote lInAMonth(int totalguests[HOTELS][MONTHS]);
void computeBestHote lBestMonth(int totalguests[HOTELS][MONTHS], int&, int&);

int main ()
{
ifstream infile;//input file object
infile.open("E: \\hoteldata.txt ");//open the file
int totalguests[HOTELS][MONTHS];
int g[HOTELS][MONTHS];
for (int i = 0; i < 5; i++)
for (int j = 0; j < 12; j++)
infile>>totalgu ests[i][j];
int besthotel;
int bestmonth;

displayGuests(t otalguests);
computeYearlyHo telGuest(totalg uests);
computeBestHote lInAMonth(g);
computeBestHote lBestMonth(tota lguests, besthotel, bestmonth);

return 0;
}
void displayGuests(i nt totalguests[HOTELS][MONTHS])
{

int HOTLES = 5;
int MONTHS = 12;
for (int i=0; i < HOTELS; i++)
{
for (int j=0; j<MONTHS; j++)

cout<<totalgues ts[i][j]<<" ";
cout<<endl;
}

}
int computeYearlyHo telGuest(int totalguests[HOTELS][MONTHS])
{
int num;
int Hotel;
int Months;
int sum;

cout<<"Enter Hotel number 1 - 5: "<<endl;
cin>>num;
switch(num)
{
case 1:
sum = 0;
Hotel = 1;
for (Months = 0; Months < MONTHS; Months++)
{
sum = sum + totalguests[Hotel][Months];
}
cout<<"Yearly Total: "<<sum<<end l;
}

return 0;
}
int computeBestHote lInAMonth(int totalguests[HOTELS][MONTHS])
{
int month;
int hotel;
int bestHotel;
cout<<"Enter the Month:"<<endl;
cin >> month;

switch (month)
{
case 1:
for (int month = 0; month< MONTHS; month++)
{
bestHotel = totalguests[0][month];
for (hotel = 1; hotel < HOTELS; hotel++)
if (bestHotel < totalguests[hotel][month])
bestHotel = totalguests[hotel][month];

cout<<"The best Hotel in January is"<<bestHotel< <endl;
}
return 0;
}
void computeBestHote lBestMonth(int g[HOTELS][MONTHS],int besthotel, int bestmonth)
{
int goodhotel = 0, goodmonth = 0;
for (int i = 0; i<5; i++)
{
for (int j = 0; j<12; j++)
if(g[goodhotel][goodmonth]<g[i][j])
{
goodhotel = i;
goodmonth = j;
}
}
besthotel = goodhotel + 1;
bestmonth = goodmonth + 1;
}

hi there,

i think your error is in within your switch file , co'z you'd always closing it , look at this line:

switch(num)
{
case 1:
sum = 0;
Hotel = 1;
for (Months = 0; Months < MONTHS; Months++)
{
sum = sum + totalguests[Hotel][Months];
}
cout<<"Yearly Total: "<<sum<<end l;
}
return 0;
}

with this line you say that your done already because you'd close the file on your switch case

review your work again and do not close necessary files on your program...

regards,