Не вызывается нужный конструктор базового класса

@moskitos80 · Регистрация: 04.10.2011

Author24 — интернет-сервис помощи студентам

Здравствуйте. Я пока в процессе изучения С++. Имеется следующая структура классов:

Базовый: Person,
расширяющие его: Student и Manager,
и наследующий их обоих класс Employee

- древо в виде ромба

Вот собственно код:

C++

#include <iostream>
#include <iomanip>
#include <string.h>
 
using namespace std;
 
///////////////////////////////////////////////////////////
class Person {
 
      protected :
          string name;
 
      public :
          Person() // Этого конструктора в начале не было.
          {}
          Person(string nm) :
                name (nm)
          {}
          virtual ~Person()
          {
            cout << "Person destruct" << endl;
          }
 
          virtual void getInfo() = 0;
};
class Student : virtual public Person {
 
      protected :
          int avg;
          int course;
 
      public :
          Student(string nm, int cr = 1, int g = 0) :
                Person(nm),
                avg (g),
                course (cr)
          {}
          ~Student()
          {
              cout << "Student destruct" << endl;
          }
};
class Manager : virtual public Person {
 
      protected :
          float salary;
          string post;
 
      public :
          Manager(string nm, float sal = 0.0f, string pst = "junior") :
                Person(nm),
                post (pst),
                salary (sal)
          {}
          ~Manager()
          {
              cout << "Manager destruct" << endl;
          }
};
class Employee : public Manager, public Student {
 
      public :          
          Employee(string nm) :
                         Manager(nm),
                         Student(nm)
          {}
          ~Employee()
          {
              cout << "Employee destruct" << endl;
          }
          void getInfo()
          {
              cout << "Manager: "      << endl
                   << "name  " << name << endl
                   << "post "  << post << endl;
              if (course > 0) {
                  cout << "It is also a student of " << course << " course " << endl;
              }
              cout << setfill('.') << setw(80) << "" << endl;
          }
};
///////////////////////////////////////////////////////////
 
int main() {
    string data;
 
    cout << "Enter name "; cin >> data; // Ввод имени
    cout << endl;
 
    Person* ptr = new Employee(data);
    ptr->getInfo();
 
    delete ptr;
    return 0;
}

Для того чтобы обратиться к полю name из Employee::getInfo() (строка 75) мне пришлось сделать следующее:

class Manager : virtual public Person ...
class Student : virtual public Person ...

что бы убрать неоднозначность, но после этого в строке 66 (конструктор Employee) компилятор стал требовать явно указать конструктор без аргументов для базового класса Person:

error: no matching function for call to 'Person::Person()'|

когда же я его добавил (строка 15) все стало ок, но теперь при работе программы:

Enter name (вводим: Vasya)

Manager:
name --- ???
post junior
It is also a student of 1 course
...............................................................................

Employee destruct
Student destruct
Manager destruct
Person destruct

Process returned 0 (0x0) execution time : 9.386 s
Press any key to continue.

Нету введённого нами имени! - я понимаю, что для базового класса Person вызывается конструктор без аргументов, который потребовал компилятор, но не понимаю почему он его потребовал, ведь у меня нигде нет ситуации, способствующей вызову Person() и почему не вызывается Person(string nm) - который мне нужен для инициализации поля name

@John Prick · 07.08.2012, 12:39

Не уверен, но по-моему, конструктор Employee должен выглядеть так:

C++

Employee(string nm) :
                         Person(nm),
                         Manager(nm),
                         Student(nm)

Это связано с тем, что виртуальные наследники от Person, классы Manager и Student не инициализируют поля Person. Это должен делать невиртуальный наследник - Employee. В списке инициализации ты не указал Person(nm), поэтому компилятор потребовал от тебя конструктор по умолчанию.

DiffEreD · 07.08.2012, 12:40

Попробуйте

так

C++ (Qt)

class Person {
 
protected :
    string name;
 
public :
    //Person() // Этого конструктора в начале не было.
    //{}
    Person(string nm) :
        name (nm)
    {}
    virtual ~Person()
    {
        cout << "Person destruct" << endl;
    }
 
    virtual void getInfo() = 0;
};
class Student : virtual public Person {
 
protected :
    int avg;
    int course;
 
public :
    Student(string nm, int cr = 1, int g = 0) :
        Person(nm),
        avg (g),
        course (cr)
    {}
    ~Student()
    {
        cout << "Student destruct" << endl;
    }
};
class Manager : virtual public Person {
 
protected :
    float salary;
    string post;
 
public :
    Manager(string nm, float sal = 0.0f, string pst = "junior") :
        Person(nm),
        post (pst),
        salary (sal)
    {}
    ~Manager()
    {
        cout << "Manager destruct" << endl;
    }
};
class Employee : public Manager, public Student {
 
public :          
    Employee(string nm) :
        Manager(nm),
        Student(nm),
        Person(nm)
    {}
    ~Employee()
    {
        cout << "Employee destruct" << endl;
    }
    void getInfo()
    {
        cout << "Manager: "      << endl
            << "name  " << name << endl
            << "post "  << post << endl;
        if (course > 0) {
            cout << "It is also a student of " << course << " course " << endl;
        }
        cout << setfill('.') << setw(80) << "" << endl;
    }
};

@John Prick · 07.08.2012, 12:46

Если что: С.Дьхерст "Скользкие места С++", Совет 53. Инициализация виртуальных базовых классов (стр.124).

@moskitos80 · 07.08.2012, 12:54 **[ТС]**

Сообщение от John Prick

Это связано с тем, что виртуальные наследники от Person, классы Manager и Student не инициализируют поля Person. Это должен делать невиртуальный наследник - Employee.

Отлично! Чётко и по существу - теперь всё ясно. Я предполагал что здесь дело в виртуальных наследниках. Отдельно спасибо за С.Дьхерста

yuron_477 - я бы уже и сам догодался, но всё же спасибо и Вам.

Ещё маленький вопрос: в конструкторах у виртуальных наследников Manager и Student вообще не имеет смысла вызывать конструктор базового класса Person ?

@John Prick · 07.08.2012, 13:02

Сообщение от moskitos80

спасибо за С.Дьхерста

Я опечатался, и вы в след за мной. Стивен Дьюхерст (Stephen Dewhurst), если будете искать.

@moskitos80 · 07.08.2012, 13:13 **[ТС]**

Сообщение от John Prick

Я опечатался, и вы в след за мной. Стивен Дьюхерст (Stephen Dewhurst), если будете искать.

Да-да поисковик меня уже поравил.

А по-поводу того что:

Ещё маленький вопрос: в конструкторах у виртуальных наследников Manager и Student вообще не имеет смысла вызывать конструктор базового класса Person ?

Сам отвечу: по-ходу дела можно и не вызывать, но тогда опять же для них будет вызываться конструктор класса Person без параметров, а так как у базового класса Person уже явно определён конструктор с параметром, то конструктор без параметра тоже потребуется определить явно. Но тем не менее будет работать. Ох уж эти конструкторы

@John Prick 2278 / 1769 / 741 Регистрация: 27.07.2012 Сообщений: 5,253
	07.08.2012, 12:46	4
	Если что: С.Дьхерст "Скользкие места С++", Совет 53. Инициализация виртуальных базовых классов (стр.124). 1

@moskitos80 440 / 97 / 42 Регистрация: 04.10.2011 Сообщений: 346
	07.08.2012, 12:54 [ТС]	5
	Сообщение от John Prick Это связано с тем, что виртуальные наследники от Person, классы Manager и Student не инициализируют поля Person. Это должен делать невиртуальный наследник - Employee. Отлично! Чётко и по существу - теперь всё ясно. Я предполагал что здесь дело в виртуальных наследниках. Отдельно спасибо за С.Дьхерста yuron_477 - я бы уже и сам догодался, но всё же спасибо и Вам. Ещё маленький вопрос: в конструкторах у виртуальных наследников Manager и Student вообще не имеет смысла вызывать конструктор базового класса Person ? 0

@John Prick 2278 / 1769 / 741 Регистрация: 27.07.2012 Сообщений: 5,253
	07.08.2012, 13:02	6
	Сообщение от moskitos80 спасибо за С.Дьхерста Я опечатался, и вы в след за мной. Стивен Дьюхерст (Stephen Dewhurst), если будете искать. 0

@moskitos80 440 / 97 / 42 Регистрация: 04.10.2011 Сообщений: 346
	07.08.2012, 13:13 [ТС]	7
	Сообщение от John Prick Я опечатался, и вы в след за мной. Стивен Дьюхерст (Stephen Dewhurst), если будете искать. Да-да поисковик меня уже поравил. А по-поводу того что: Ещё маленький вопрос: в конструкторах у виртуальных наследников Manager и Student вообще не имеет смысла вызывать конструктор базового класса Person ? Сам отвечу: по-ходу дела можно и не вызывать, но тогда опять же для них будет вызываться конструктор класса Person без параметров, а так как у базового класса Person уже явно определён конструктор с параметром, то конструктор без параметра тоже потребуется определить явно. Но тем не менее будет работать. Ох уж эти конструкторы 0