Прежде всего, я думаю, что у вас имеется опечатка в определении данного конструктора
Couple(const Person& she, Person& he){
cout<<he.name;;
};
Либо вы имеете в виду
Couple(const Person& she, const Person& he){
^^^^^^^^^^^^^^^^
cout<<he.name;;
};
либо
Couple(Person& she, Person& he){
^^^^^^^^^^^
cout<<he.name;;
};
Иначе определение класса выглядит крайне запутанным.
Кроме того, если в определениях параметров конструктора присутствует квалификатор const
, то соответствующий член класса либо указатель wife
, либо указатель husband
, либо оба указателя также должны иметь квалификатор const
. В противном случае компилятор выдаст диагностическое сообщение.
На мой взгляд У вас есть противоречие в дизайне и использовании классов.
Когда вы создаете объект в динамической области памяти, как в данном случае
Couple *pcouple1 = new Couple (mary,*pjohn);
то предполагается, что время жизни объекта будет простираться за пределы того блока, где объявлен указатель pcouple1
, или где существуют инициализаторы mary
и *pjohn
, Иначе для этих классов не имеет никакого смысла создавать объекты в динамической области памяти, если время жизни объектов ограничивается тем блоком, где определены соответствующие указатели.
А это чревато неопределенным поведением программы, когда объекты, на которых в классе Couple
указывают соответствующие члены данных, будут удалены раньше, чем сам объект, содержащий указатели.
Ниже приведена программа, которая демонстрирует это неопределенное поведение.
#include <iostream>
#include <cstring>
class Person
{
public:
explicit Person( const char *name )
: name( std::strcpy( new char[std::strlen( name ) + 1], name ) )
{
}
~Person()
{
std::cout << "Person::~Person()" << std::endl;
delete [] name;
name = "Hello undefined behaviour!";
}
char *name;
};
class Parent
{
public:
Parent( const Person &person ) : person( &person ){}
const Person *person;
};
int main()
{
Parent *p;
{
Person person( "user2420249" );
p = new Parent( person );
std::cout << p->person->name << std::endl;
}
std::cout << p->person->name << std::endl;
delete p;
}
На консоль будет выдано:
user2420249
Person::~Person()
Hello undefined behaviour!
Как видно из вывода, объект Person
был уже удален, в то время как объект Parent
еще живой.
Более того, в качестве аргумента конструктора класса Parent
вообще может быть задан временный объект, так как константная ссылка может быть связана с временным объектом.
Поэтому в вашем случае лучше в классе Couple
создавать копии аргументов, а не просто присваивать указателям адреса аргументов, переданных по ссылке.
Такой подход, когда непосредственно указателям присваиваются адреса аргументов, имеет смысл только в том случае, когда время жизни создаваемого объекта не больше времени жизни объектов аргументов, и когда создаваемый объект не является владельцем своих подобъектов.
strdup
?