Constructors: No Matching function for call error

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:

Code:

---

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

Code:

---


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();
}

---

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

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

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.

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

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();
}

---

Quote:

---

Originally Posted by 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.

---

Hit the nail on the head!! Many thanks