Syntax Error comparing private class data

[ishakarthika]

how can i compare the a private variable of a class and a value in the column of a text file.

there is a syntax error in my code while comparing. senario is i am getting bus details like busno, source , destination,typ e of bus, price/head . I should compare the busno which i get through object to the bus no which is present in a .txt file. and contents in the text file is as the format below.
|1488|xxx|uuuu| A/c|150.
|1422|mmm|oooo| nonA/C|900.

#include<iostream>
#include<iomanip>
#include<fstream>
#include<string>
class add
{
private:
     int bus_no;
     int route_no;
     string source;
     string dest;
     int time;
     int ticket_price;
     string type;
 
public:
void enter_bus()
{
        char temp[1000],busno[5];
        string type("A/C");
        cout<<"Enter Bus_No::";
        cin>>busno;
        bus_no=string(busno);
        cout<<"Enter Bus_Route::";
        cin>>route_no;
        cout<<"Enter Source::";
        cin>>source;
        cout<<"Enter Destination::";
        cin>>dest;
        cout<<"Enter Time(Hrs:Min)::";
        cin>>time;
        cout<<"Enter Ticket_Price::";
cin>>ticket_price;
       // Type:
        cout<<"Type of Bus (A/C or NonA/C)::";
       // cout<<"Enter Type::";
        cin>>type;
}
 void get_busno()
{
  bus_no;
}
 
void display__bus()
{
 
                cout<<"Bus details"<<endl;
                cout<<"|"<<bus_no<<"|";
                <<setw(10)<<route_no<<"|";
                <<setw(15)<<source<<"|";
                <<setw(15)<<dest<<"|";
                <<setw(8)<<time<<"|";
                <<setw(6)<<type<<"|";
                <<setw(3)<<no_of_seats<<"|";
                <<setw(4)<<ticket_price<<endl;
}
 
void intofile__bus()
{
 
 
                cout<<"|"<<bus_no<<"|";
                <<setw(10)<<route_no<<"|";
                                                <<setw(15)<<source<<"|";
                <<setw(15)<<dest<<"|";
                <<setw(8)<<time<<"|";
                <<setw(6)<<type<<"|";
                <<setw(3)<<no_of_seats<<"|";
                <<setw(4)<<ticket_price<<endl;
}
};
void main
{
 
        add obj;
        fstream file;
        file.open("Bus.txt",std::ios_base::ate | std::ios_base::in | std::ios_base::out );
        file.seekg(0, std::ios_base::end);
        unsigned long length = file.tellg();
        int obj_length=sizeof(obj);
        int p=length/obj_length;
        int n;
        int option;
        char choice;
        int flag=0;
        add obj2[p];
        file.seekg(0,ios::beg);
        cout<<"Want to enter details in the file (y/n)?"<<endl;
        cin>>choice;
        while((choice=='y') || (choice=='Y'))
        {
                 cout<<"Enter the option 1 for adding details:"<<endl;
                cin>>option;
                switch (option)
                {
                 case 1: obj.enter_bus();
for(n=0;n<=p;n++)
                        {
                           file.read((char *)&obj2[n],sizeof(obj2[n]));
                           if(strcmp(obj2[n].get_busno(),obj.get_busno())==0)
                             {
                               cout<<"Sorry the bus already exist"<<endl;
                               flag=2;
                               break;
                            }
                            else
                            {
                              flag=1;
                            }
                         }
 
                default: cout<<"Please enter the option 1 for adding details"<<endl;
                }
 
                if(flag==1)
                {
                   obj.enter_bus();
                   obj.intofile_bus();
                   file.write((char *)&obj,sizeof(obj));
                   cout<<"Record inserted successfully...";
                }
               else
                {
                 cout<<"Want to enter the details again(y/n)?"<<endl;
                 cin>>choice;
                }
            }

[ishakarthika]

Please read our posting guidelines taking note of the sections on choosing a thread title and composing your post (using code tags round the posted code).


You have posted code and said you have a syntax error, this is not enough you should post the exact error you are getting (copy and past it) and you need to tell us on what line of code it is appearing.

However in general

1. You have declared main as returning void. This invokes undefined behaviour, all bets are now off your program could do anything. Examining any further errors after this one is pointless.

Main MUST be declared as either

int main()

or

int main(int argc, char **argp)

that is it must return int.

2. Private data is only accessible inside the class. I don't see you calling any class methods so I assume your syntax error is attempting to access private data. You will need to provide access to the data by either creating a method to return the data, creating a method to do the comparison for you, changing the access specifier of the data being declared (but not this last one is really a hack).

yeah
i have did but again error.

declared a function to get bus no
public:
void get_bus_no()
{
cout<<bus_no;
}

while getting the bus_no i used a temporary variable busno converting it into string , assignin it to bus_no and declared bus_no as string in private.

cout<<"Enter Bus_No::";
cin>>busno;
bus_no=string(b usno);

i have declared bus_no as private.
private:
string bus_no;

but still error in this line

if(strcmp(strin g(obj2[n].get_bus_no()), obj.get_bus_no( ))==0)

[Banfa]

declared a function to get bus no
public:
void get_bus_no()
{
cout<<bus_no;
}

This doesn't get the bus number it prints it to the console!

if(strcmp(strin g(obj2[n].get_bus_no()), obj.get_bus_no( ))==0)

What type is strcmp expecting for the parameters being passed to it? What type is get_bus_no() returning? Are these types compatible?

[Savage]

Again,your function is of void type and void functions can't return any value.What you did is just printed the value of bus_no to the screen,but you need to return that value to the calling function.So declare your get_busno() function to return the same type of which is bus_no made.

EDIT:Also bus_no is a int,but it's the string you need to return.

[pootle]

yeah

if(strcmp(strin g(obj2[n].get_bus_no()), obj.get_bus_no( ))==0)

When you declare your get_bus_no() method so:

std::string& get_bus_no() const {
   return bus_no;
}
 

Then you do not need strcmp anymore. The strcmp function is only used for comparing char*. You can find out all about standard library strings and string comparison / manipulation on the internet.