Thread: Template Problem

Okay here's my dillema. I'm creating a class template, called RentalSystem. This RentalSystem is supposed to accept either a Car class or a Motorcycle class, with each having 3 derived classes for more specific types ( i.e. economy, luxury, midsize, etc. ). Now the way I have it setup is that a function within RentalSystem parses through the command file, and recieves each command. If a vehicle is supposed to be added, the vehicle type is compared with that of the 3 derived classes and the vehicle pointer is set to the address of the derived class. The problem is, now that I'm being asked to use a template, the project requirements prohibit me from explicity making any reference to Car or Motorcycle classes inside the template, i.e. it all has to be <T>. So what I can't figure out is how I'm supposed to create this derived object within the RentalSystem template without explicity creating a Economy object, etc. Sorry if that was unclear, if so I can clarify.

Thanks,
Chris

Based on an instructor's suggestion, I'm going to parse the command file in main as opposed to the RentalSystem class. So at this point, what I'm trying to do is simply read in the car type from the command file in main, and send a derived class pointer to a function that will add it to my vector of vehicle pointers. I.e. main would be

Code:

NewCar* Car = new NewCar;
NewLuxury Luxury;
while ( inStream !eof() )
{
    if ( command == add_car )
   {
      inStream >> carType;

      if ( carType == Luxury )
      {
           NewCar == &Luxury;
      }
    }
}

Then I would send NewCar to a member function of RentalSystem called AddVehicle, which would take the object and place it into the private data member; a vector of <T*>. Would this work? I wasn't sure if there'd be any scope issues with it.

BTW, do you use AIM Daved? Would be easier than posting back and forth. If you use it, my AIM is in my user profile.

Sounds good. You might have to do it like this so that the AddCar variable doesn't go out of scope:

Code:

Car* newCar = 0;
if ( type == ECONOMY )
{
    newCar = new Economy;
}
else if ( type == LUXURY )
{
    newCar = new Luxury;
}
else if ( type == MIDSIZE )
{
    newCar = new Midsize;
}
// Add newCar to your vector here.

That assumes that Economy, Luxury, and Midsize all derive from Car.

Yes, they all derive from Car. My plan was to do something like

Code:

Car* newCar = 0;
if ( type == ECONOMY )
{
    newCar = new Economy;
     Rental.AddVehicle ( newCar );
}
else if ( type == LUXURY )
{
    newCar = new Luxury;
     Rental.AddVehicle ( newCar );
}
else if ( type == MIDSIZE )
{
    newCar = new Midsize;
     Rental.AddVehicle ( newCar );
}

Since the car pointer vector is a private data member of the Rental class. That should still work, right?

That's fine. If you ever have time to come back to this you should consider using a Factory method instead, so you could do it all in one piece of code instead of duplicating code in the if and else blocks.

Code:

// Defined somewhere to return proper derived class.
newCar* CreateNewCar(CAR_TYPE type);
 
...
 
// Later just call the function ...
Rental.AddVehicle(CreateNewCar(type));

Like I said, don't worry about that now, it is just a slightly better design, but if you have time come back to it and understand why it is better.

Thanks, I'll give it a shot. I'm not quite to the point where I can test it, because I'm still transfering things from the RentalSystem to main, cleaning it up in functions, etc; the dirty work. I'll have to check out Factory methods once the semester is over. 3 more weeks