Constructors: No Matching function for call error

[Emilio Martin]

Hi,

This is probably gonna be something really simple! but please help..

The problem seems to be with the Registration constructor. I get a candidates are Person::Person(QString,QString,QString)

I thought that was what I was doing! Umm, there are no default constructors

I have the following headers:

class Person
{
public:

Person(QString,QString,QString);
QString getName();
QString getEmail();
QString getAffiliation();
QString toString();
private:
QString m_Name;
QString m_Email;
QString m_Affiliation;

};
class Registration: public Person
{
public:
Registration(QString,QString,QString);
Person getAttendee();
QDate getBookingDate();
double calculateFee();
QString toString()const;
private:
Person m_Attendee;
QDate m_BookingDate;
static const double STANDARD_FEE;
};

and the constructors look like this

Person::Person(QString n,QString e,QString a)
{
m_Name=n;
m_Email=e;
m_Affiliation=a;
}Registration::Registration(QString n,QString e,QString a) : m_Attendee(n,e,a)
{
m_BookingDate=QDate::currentDate();
}

[Elysia]

Based on your code, I suggest:

• That you indent properly
• That you use proper names (not n, e, etc)
• That you read up on how initializer lists work
• That you read SourceForge.net: Do not remove parameter names - cpwiki

[jimblumberg]

Also when you get compiler errors post the complete error messages, exactly as they appear in your development environment. These messages have important information to aid in location and repair of the problem.

Jim

[hk_mp5kpdw]

Your Registration class is both derived from and contains a Person object? I would think you'd want one or the other but not both. Change Registration class so it is not derived from a Person object and you should be OK.

[Emilio Martin]

Hi,

- Sorry indentations disappeared with copy and paste..
- I agree on the param names, the question was using them
- I have been reading up on initializer lists, but guess I'm missing something?
- point taken on parameter names

Here is updated code

classPerson
{
public:

Person(QStringname,QStringemail,QStringaffiliation);
QStringgetName();
QStringgetEmail();
QStringgetAffiliation();
QStringtoString();
private:
QStringm_Name;
QStringm_Email;
QStringm_Affiliation;

};
classRegistration:publicPerson
{
public:
Registration(QStringname,QStringemail,QStringaffiliation);
PersongetAttendee();
QDategetBookingDate();
doublecalculateFee();
QStringtoString()const;
private:
Personm_Attendee;
QDatem_BookingDate;
staticconstdoubleSTANDARD_FEE;
};
Person::Person(QStringname,QStringemail,QStringaffiliation)
{
m_Name=name;
m_Email=email;
m_Affiliation=affiliation;
}

Registration::Registration(QStringname,QStringemail,QStringaffiliation):m_Attendee(name,email,affiliation)
{
m_BookingDate=QDate::currentDate();
}

[Emilio Martin]

Your Registration class is both derived from and contains a Person object? I would think you'd want one or the other but not both. Change Registration class so it is not derived from a Person object and you should be OK.

Hit the nail on the head!! Many thanks