Questions about Deep Copying with Inheritance

class Goal {	protected:	Character *doer;	Character *giver;	Morality morality; // good, evil, etc	virtual void executeImpl() = 0;	public:	Goal(Character* doer, Character* giver = 0) : doer(doer), giver(giver), finished(false) {}	// These should probably be read-only to the public, thus:	const Morality& getMorality() { return morality; }	virtual bool isFinished() = 0; // yes, really        void Execute() {		if (!isFinished()) { executeImpl(); }	}	// ...};class TakeItem : public Goal {	public:	Item *item;	Move move;	TakeItem(Character* doer, Character* giver, Item* item) : Goal(doer, giver), item(item) {}	virtual void isFinished() { return move.finished; }	void executeImpl() {		move.Execute();		// Why would you use printf()? :( Also, note that I am		// re-distributing the work here; the Character interface must		// change accordingly		cout << doer->name() << "moves to " << doer->position() << "\n";		if (move.finished) {			item->personCarrying = doer;			// Yes, it is possible to make this work with 'item' as			// is, and you probably should do so.			cout << "\n" << doer->name() << " takes item: " << item << "\n\n";		}	}};class NPC : public Character {	Goal *currGoal;	NPC(const NPC&); // do not implement!	NPC& operator=(const NPC&); // do not implement!	public:	NPC() : currGoal(0) {}	~NPC() { delete currGoal; }	void tryToTake(Item* i) {		delete currGoal;		currGoal = new TakeItem(this, 0, i);	}};NPCs.tryToTake(items[indexOfClosestItem]);

class Goal {	// other stuff	public:	virtual Goal* clone() = 0;};class TakeItem {	// other stuff	public:	// clone() is implemented in terms of the copy constructor.	virtual TakeItem* clone() { return new TakeItem(*this); }	// Because a TakeItem object does NOT "own" the pointed at 'doer',	// 'giver' or 'item', but simply "knows about" pre-existing objects via	// those pointers, it is correct for the copy constructor to just copy	// the pointer values. Therefore, as long as the Move copy constructor	// works properly, there is no need to implement a copy constructor	// for TakeItem at all; the default generated one will work.};class NPC : public Character {	Goal *currGoal;	public:	// We ask the 'other' NPC's goal to clone itself, and then initialize	// our goal to be the cloned object.	NPC(const NPC& other) : currGoal(other.currGoal->clone()) {}			NPC& operator=(const NPC& rhs) {		// We use the copy and swap idiom.		NPC other(rhs);		std::swap(currGoal, other.currGoal);		return *this;		// because of the swap, the destructor of 'other' will		// automatically clean up our old Goal, instead of the cloned		// one in the 'other' NPC (which is now our Goal).	}};