C / C++ / MFC Discussion Boards

error C2228 "must have class/struct/union"

alaaan7318-Dec-12 4:55

18-Dec-12 4:55

in Flight.h i have this prototype :

bool reserveSeat(Passenger psngr, bool smoker);

and i have to do the implementation for it , i try to do it but i got an error :
"

error message :
task b\flight.cpp(15): error C2228: left of '.setPassportNumber' must have class/struct/union
1>type is ''unknown-type''
1> b\flight.cpp(16): error C2065: 'passenger' : undeclared identifier
1>b\flight.cpp(16): error C2228: left of '.setContactNumber' must have class/struct/union
1>type is ''unknown-type''
1> b\flight.cpp(17): error C2228: left of '.smokerSeat' must have class/struct/union

and here is my try

bool Flight::reserveSeat(Passenger psngr, bool smoker)
{
   Passenger.setPassengerName(psngr.getPassengerName());
   passenger.setPassportNumber(psngr.getPassportNumber());
   passenger.setContactNumber(psngr.getContactNumber());
   reservations.smokerSeat=smoker;

}

note that Flight have an object of Reservation and here's the Reservation constuctor :
Reservation.cpp

Reservation::Reservation(Passenger& psngr, bool smoke)
	
{
   passenger.setPassengerName(psngr.getPassengerName());
   passenger.setPassportNumber(psngr.getPassportNumber());
   passenger.setContactNumber(psngr.getContactNumber());
   smokerSeat=smoke;
}//end consreuctor

and here's the whole Flight Header

#include "reservation.h"
#include "okReservation.h"
#include "waitReservation.h"
class Flight{
	
private:
	int waitingListMax;
	int seats;
	int seatsCount;
	Reservation** reservations;
	
public:
	Flight(int capacity, int waitingMax);
	bool reserveSeat(Passenger psngr, bool smoker);
	
	//Searches for the reservation with the given reservation number, and 	//deletes it. Uses the confirmReservation function if the reservation to be 	//deleted was an OK one 
	void cancelReservation(string resNum);
	
	//Deletes the given waiting reservation after creating a new OK reservation 	//for the same passenger. Returns a pointer to the newly created reservation
	okReservation confirmReservation(waitReservation* res); 
	void printWaitingList();
	void printReservationList();

	};

Re: error C2228 "must have class/struct/union"

C^hris L^osinger18-Dec-12 5:19

C^hris L^osinger

18-Dec-12 5:19

alaaan73 wrote:
Passenger.setPassengerName(psngr.getPassengerName());
passenger.setPassportNumber(psngr.getPassportNumber());

as far as i can tell, you have not declared any variables named "Passenger" or "passenger".

reservations.smokerSeat=smoker;

you have declared reservations as a pointer to a pointer to a Reservation.

Reservation** reservations;

so you can't use "reservations.smokerSeat". you would have to declare it as:

Reservation reservations;

etc..

image processing toolkits | batch image processing

Re: error C2228 "must have class/struct/union"

Richard MacCutchan18-Dec-12 7:09

Richard MacCutchan

18-Dec-12 7:09

You have:

C++

bool Flight::reserveSeat(Passenger psngr, bool smoker)
{
   Passenger.setPassengerName(psngr.getPassengerName());
   passenger.setPassportNumber(psngr.getPassportNumber());
   passenger.setContactNumber(psngr.getContactNumber());
   reservations.smokerSeat=smoker;
 
}

which makes little sense. you are using both Passenger (with upper case P), and passenger (with lower case p) to refer to something. But what, why would you try to set the value of an item in an object from the value taken from that object? Similarly the statement

C++

reservations.smokerSeat=smoker;

make little sense as reservations is a pointer to a pointer (of something). I think you are a little confused about your classes and how they connect to each other.

One of these days I'm going to think of a really clever signature.

Re: error C2228 "must have class/struct/union"

Stefan_Lang21-Dec-12 1:59

Stefan_Lang

21-Dec-12 1:59

You can call a class method either by using the '.' operator on an instance of that class, e. g.

C++

psngr.SetPassengerName("SomeName");

Or by using the '::' operator on the class, if it is a static method, e. g.:

C++

Passenger::SetPassengerName("SomeName");

I don't think either makes sense in this case however.

Your mistakes have already been pointed out, but from the information you gave us it's hard to derive what is actually needed to fix them. Looking at the declaration of class Flight, I suspect that reservations is supposed to be a list of, well, reservations. The prototype of cancelReservation(string resNum) indicates that you are supposed to use a string representing the reservation number as a key for retrieving a specific reservation. So your reservations may in fact be organized as a sorted tree - but that is something you have to find out for yourself.

In any case, the method reserveSeat() should create a new object of type Reservation and add it to that list. Probably you should apply these method calls on that new object in order to properly initialize it. The declaration of class Reservation should shed more light on this.

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.