Connecting Tech Pros Worldwide Help | Site Map

Copy Inherited Objects

  #1  
Old June 27th, 2008, 05:43 PM
timid
Guest
 
Posts: n/a
I'm attempting to learn how to use inheritance in C++, the following
program should print the word 'Orange' on the screen but it prints
'Apple' instead. Can someone explain what is wrong?

#include <iostream>

class Fruit
{
public:
virtual const char* get_name() {return "(nothing)";}
};

class Apple: public Fruit
{
public:
const char* get_name() {return "Apple";}
};

class Orange: public Fruit
{
const char* get_name() {return "Orange";}
};

int main()
{
std::cout << "Test inheritation" << std::endl;

Fruit *myfruit1 = new Apple();
Fruit *myfruit2 = new Orange();
Fruit *myfruit3 = new Apple();

std::cout << "myfruit1 = " << myfruit1->get_name() <<
std::endl; // Prints "Apple"
std::cout << "myfruit2 = " << myfruit2->get_name() <<
std::endl; // Prints "Orange"
std::cout << "myfruit3 = " << myfruit3->get_name() <<
std::endl; // Prints "Apple"

// The following does not work
std::cout << "Now change myfruit3 from Apple to Orange..." <<
std::endl;
*myfruit3 = *myfruit2;

// Prints "Apple" but should print "Orange"
std::cout << "myfruit3 = " << myfruit3->get_name() << std::endl;

// Wait for a keypress
std::cin.get();

delete myfruit1;
delete myfruit2;
delete myfruit3;
}
  #2  
Old June 27th, 2008, 05:43 PM
Christian Hackl
Guest
 
Posts: n/a

re: Copy Inherited Objects


timid wrote:
Quote:
#include <iostream>
>
class Fruit
{
public:
virtual const char* get_name() {return "(nothing)";}
Use std::string rather than char*.

Also, a function such as "get_name" should usually be declared const.
Quote:
};
>
class Apple: public Fruit
{
public:
const char* get_name() {return "Apple";}
};
>
class Orange: public Fruit
{
const char* get_name() {return "Orange";}
};
>
int main()
{
std::cout << "Test inheritation" << std::endl;
>
Fruit *myfruit1 = new Apple();
Fruit *myfruit2 = new Orange();
Fruit *myfruit3 = new Apple();
>
[...]
>
// The following does not work
std::cout << "Now change myfruit3 from Apple to Orange..." <<
std::endl;
*myfruit3 = *myfruit2;
* derefences the pointer, yielding the object it's pointing to.
Consequently, in your example you don't assign the _pointers_, you
assign the _objects_ they are pointing to. That's a huge difference.
Assigning an Orange to an Apple obviously does not do anything because
you did not specify anything to happen in case of such an assignment.
The invisible assignment operator automatically generated by the
compiler is empty. In other words, the Apple is completely unaffected by
the operation.


If you want to assign a pointer to another pointer, you have to write:

myfruit3 = myfruit2;

However, in your program this would have disastrous results because you
would lose the old value of myfruit3, which means you cannot delete your
second Apple anymore. (You'd have to save the old value somewhere, for
example in a temporary pointer variable.)

What do you try to achieve by having myfruit3 point to a different object?


--
Christian Hackl
  #3  
Old June 27th, 2008, 05:43 PM
timid
Guest
 
Posts: n/a

re: Copy Inherited Objects


On May 17, 10:49 pm, Christian Hackl <ha...@sbox.tugraz.atwrote:
Quote:
timid wrote:
Quote:
#include <iostream>
>
Quote:
class Fruit
{
public:
virtual const char* get_name() {return "(nothing)";}
>
Use std::string rather than char*.
>
Also, a function such as "get_name" should usually be declared const.
>
>
>
Quote:
};
>
Quote:
class Apple: public Fruit
{
public:
const char* get_name() {return "Apple";}
};
>
Quote:
class Orange: public Fruit
{
const char* get_name() {return "Orange";}
};
>
Quote:
int main()
{
std::cout << "Test inheritation" << std::endl;
>
Quote:
Fruit *myfruit1 = new Apple();
Fruit *myfruit2 = new Orange();
Fruit *myfruit3 = new Apple();
>
Quote:
[...]
>
Quote:
// The following does not work
std::cout << "Now change myfruit3 from Apple to Orange..." <<
std::endl;
*myfruit3 = *myfruit2;
>
* derefences the pointer, yielding the object it's pointing to.
Consequently, in your example you don't assign the _pointers_, you
assign the _objects_ they are pointing to. That's a huge difference.
Assigning an Orange to an Apple obviously does not do anything because
you did not specify anything to happen in case of such an assignment.
The invisible assignment operator automatically generated by the
compiler is empty. In other words, the Apple is completely unaffected by
the operation.
>
If you want to assign a pointer to another pointer, you have to write:
>
myfruit3 = myfruit2;
>
However, in your program this would have disastrous results because you
would lose the old value of myfruit3, which means you cannot delete your
second Apple anymore. (You'd have to save the old value somewhere, for
example in a temporary pointer variable.)
>
What do you try to achieve by having myfruit3 point to a different object?
>
--
Christian Hackl
Thank's for the help so far, it would appear that I need to learn a
lot more.

I'm going to try & rewrite this program so it'll work.
  #4  
Old June 27th, 2008, 05:43 PM
=?ISO-8859-1?Q?Marcel_M=FCller?=
Guest
 
Posts: n/a

re: Copy Inherited Objects


Hi!

Christian Hackl schrieb:
Quote:
Quote:
>class Fruit
>{
>public:
> virtual const char* get_name() {return "(nothing)";}
>
Use std::string rather than char*.
Normally I would agree, but in case of /const/ char* there is usually no
problem. Only the use of char* for strings in C++ applications is nearly
always a risk. On the other hand, using std::string for this (and
similar) purposes is a significant runtime overhead. Only if you already
have a std::string it is more advisable to return std::string, or if
this causes no threading or aliasing problem const std::string&.
Quote:
Also, a function such as "get_name" should usually be declared const.
That's obviously true.

Quote:
If you want to assign a pointer to another pointer, you have to write:
>
myfruit3 = myfruit2;
>
However, in your program this would have disastrous results because you
would lose the old value of myfruit3, which means you cannot delete your
second Apple anymore. (You'd have to save the old value somewhere, for
example in a temporary pointer variable.)
>
What do you try to achieve by having myfruit3 point to a different object?
I think the OP is seeking for something like a clone method.


Marcel
  #5  
Old June 27th, 2008, 05:43 PM
James Kanze
Guest
 
Posts: n/a

re: Copy Inherited Objects


On 17 mai, 23:11, timid <visicalcena...@googlemail.comwrote:
Quote:
I'm attempting to learn how to use inheritance in C++, the
following program should print the word 'Orange' on the screen
but it prints 'Apple' instead. Can someone explain what is
wrong?
Quote:
#include <iostream>
Quote:
class Fruit
{
public:
virtual const char* get_name() {return "(nothing)";}
};
Quote:
class Apple: public Fruit
{
public:
const char* get_name() {return "Apple";}
};
Quote:
class Orange: public Fruit
{
const char* get_name() {return "Orange";}
};
Quote:
int main()
{
std::cout << "Test inheritation" << std::endl;
Quote:
Fruit *myfruit1 = new Apple();
Fruit *myfruit2 = new Orange();
Fruit *myfruit3 = new Apple();
Quote:
std::cout << "myfruit1 = " << myfruit1->get_name() <<
std::endl; // Prints "Apple"
std::cout << "myfruit2 = " << myfruit2->get_name() <<
std::endl; // Prints "Orange"
std::cout << "myfruit3 = " << myfruit3->get_name() <<
std::endl; // Prints "Apple"
Quote:
// The following does not work
std::cout << "Now change myfruit3 from Apple to Orange..." <<
std::endl;
*myfruit3 = *myfruit2;
And what do you expect this to do? You cannot change the type
of an existing object. In general, when inheritence is
involved, you don't support assignment and copy.
Quote:
// Prints "Apple" but should print "Orange"
std::cout << "myfruit3 = " << myfruit3->get_name() << std::endl;
No. Should print "Apple". But a lot of typical implementations
of the assignment operator (for more complicated types) will
result in undefined behavior. The base class for an inheritence
hierarchy should normally forbid assignment, so that the
situation can't arrive, even accidentally.

--
James Kanze (GABI Software) email:james.kanze@gmail.com
Conseils en informatique orientée objet/
Beratung in objektorientierter Datenverarbeitung
9 place Sémard, 78210 St.-Cyr-l'École, France, +33 (0)1 30 23 00 34
  #6  
Old June 27th, 2008, 05:43 PM
James Kanze
Guest
 
Posts: n/a

re: Copy Inherited Objects


On 18 mai, 11:29, Marcel Müller <news.5.ma...@spamgourmet.comwrote:
Quote:
I think the OP is seeking for something like a clone method.
Or the letter/envelope idiom. Or perhaps just simply some
information on OO design; he hasn't really expressed a concrete
need for assignment.

--
James Kanze (GABI Software) email:james.kanze@gmail.com
Conseils en informatique orientée objet/
Beratung in objektorientierter Datenverarbeitung
9 place Sémard, 78210 St.-Cyr-l'École, France, +33 (0)1 30 23 00 34
Closed Thread


Similar Threads
Thread Thread Starter Forum Replies Last Post
Templated containers of inherited objects George Exarchakos answers 21 February 20th, 2007 03:35 AM
Synthesized and non-inherited class members lovecreatesbea...@gmail.com answers 14 November 17th, 2006 09:15 PM
Is 'everything' a refrence or isn't it? KraftDiner answers 161 January 22nd, 2006 11:35 PM
explicit call to copy constructor and operator = needed trying_to_learn answers 8 July 22nd, 2005 11:02 PM