Printable View
Hey everyone, I am having some difficulties knowing how to assign city to the list. Can anyone help? Thanks!
Code:class DestinationList
{
private:
int num; // number of objects in the list
Destination list[MAX_CITIES]; // array of destinations
public:
int Add(string city)
{
num = 0;
list[num] = city;
num++;
}
};
You don't wan to set num to zero every time you call add. You want it to start off as zero.
That's the job of a "constructor", hint hint.
Yes, initialise num to 0 just once, in the constructor.
Well when I made the constructor and assigned num to zero it gives me this error "Cannot assign values of type string to a variable of type Destination"
That seems about right.
What's the relationship between a destination and a city?
Perhaps your destinationlist add function should accept a destination, not a city.
Post the code, and the exact error messages!
Jim
That is the exact error message XD I have to use some limited compiler for some reason. I have the code posted at the top already too, the only difference is that a put in a constructor.
Well the city is the destination of where the shipment has to be dropped off. Would accepting a destination not a city still be a string anyways?
Hey, it's your definition of Destination, not mine you have to worry about.
What's the difference between
Destination list[MAX_CITIES]; // array of destinations
and
String list[MAX_CITIES]; // array of destinations
Presumably, you created a class called Destination, with an obvious member called say city, which is a string.
But what else?
string city
int package_count
float total_weight
are the members that are present in Destination right now.
you could try
KurtCode:list[num].city = city;
Hey that works! Thanks! Don't really know why I didn't try that yet! XD
I'm suprised. Usually classes don't have too many public data members.
Next time it would be a good idea to show the declaration of the class in question if you really want help.
Kurt
This spells trouble. First off, you should probably be using std::vector instead of an array here. Use std::vector::push_back.
And, as you have clearly demonstrated, a city is not a destination. You need to add some way to create a destination from a city. Probably using a constructor in the Destination class.
That seems more like a 'Consignment' to me.Quote:
Originally Posted by XNOViiCE
One thing about OO design and programming is giving your objects accurate and meaningful names.
No one here would have even guessed that it had anything to do with packages until you said so.