继承和构造函数语法造成的一个bug

一

• 出错误的代码
  开发环境： visual studio 2017 社区版

#include "pch.h"
#include <iostream>
#include <string>
#include <list>
using namespace std;

//公司类 抽象类或接口
class Company
{
public:
    Company() {}
    Company(string str)
        :name(str) {}
    virtual void Add(Company* c) = 0;  //增加
    virtual void Remove(Company* c) = 0;  //移除
    virtual void Display(int depth) = 0;  //显示
    virtual void LineOfDuty() = 0;  //履行职责  
    inline bool operator==(const Company& company) const
    {
        return this->name == company.name;
    }
protected:
    string name;
};

class ConcreteCompany : public Company
{
public:
    ConcreteCompany(string name) : Company(name)
    {
        children = new list<Company*>;
    }
    void Add(Company* c) override
    {
        children->push_back(c);
    }
    void Remove(Company* c) override  //删除子节点
    {

        for (list<Company*>::iterator it = children->begin(); it != children->end(); ++it)
        {
            if (*(*it) == *c)
            {
                children->erase(it);
                break;
            }
        }
    }
    void Display(int depth) override  //显示
    {
        for (int ix = 0; ix < depth; ++ix)
        {
            cout << "-";
        }
        cout << this->name << endl;
        for (list<Company*>::iterator it = children->begin(); it != children->end(); ++it)
        {
            (*it)->Display(depth + 2);
        }
    }
    virtual void LineOfDuty() override  //履行职责
    {
        for (list<Company*>::iterator it = children->begin(); it != children->end(); ++it)
        {
            (*it)->LineOfDuty();
        }
    }
private:
    list<Company*>* children;
};

//人力资源部 树叶节点
class HRDepartment : public Company
{
public:
    HRDepartment(string str) : Company(name) {}
    void Add(Company* c) override  //增加
    {

    }
    void Remove(Company* c) override  //移除
    {

    }
    void Display(int depth) override  //显示
    {
        for (int ix = 0; ix < depth; ++ix)
        {
            cout << "-";
        }
        cout << name << endl;
    }
    void LineOfDuty() override  //履行职责  
    {
        cout << this->name << " 员工招聘培训管理" << endl;
    }
};

class FinanceDepartment : public Company
{
public:
    
    FinanceDepartment(string str) : Company(name) {}
    void Add(Company* c) override  //增加
    {

    }
    void Remove(Company* c) override  //移除
    {

    }
    void Display(int depth) override  //显示
    {
        for (int ix = 0; ix < depth; ++ix)
        {
            cout << "-";
        }
        cout << name << endl;
    }
    void LineOfDuty() override  //履行职责  
    {
        cout << this->name << " 公司财务收支管理" << endl;
    }
};

int main()
{
    Company* root = new ConcreteCompany("北京总公司");
    root->Add(new HRDepartment("总公司人力资源部"));
    root->Add(new FinanceDepartment("总公司财务部"));

    ConcreteCompany* comp = new ConcreteCompany("上海华东分公司");
    comp->Add(new HRDepartment("华东分公司人力资源部"));
    comp->Add(new FinanceDepartment("华东分公司财务部"));
    root->Add(comp);

    ConcreteCompany* comp1 = new ConcreteCompany("南京办事处");
    comp1->Add(new HRDepartment("南京办事处人力资源部"));
    comp1->Add(new FinanceDepartment("南京办事处财务部"));
    comp->Add(comp1);

    ConcreteCompany* comp2 = new ConcreteCompany("杭州办事处");
    comp2->Add(new HRDepartment("杭州办事处人力资源部"));
    comp2->Add(new FinanceDepartment("杭州办事处财务部"));
    comp->Add(comp2);

    cout << "\n结构图" << endl;
    root->Display(1);

    cout << "\n职责：" << endl;
    root->LineOfDuty();

    return 0;
}

编译上述代码，VS2017提示错误

一开始这个错误让我百思不得其解，因此将同样的代码放到Linux下编译调试。g++提示的错误如下：

二

• 错误的解决
  经过排查发现，错误出现在下面这两行代码

HRDepartment(string str) : Company(name) {}
FinanceDepartment(string str) : Company(name) {}

其中HRDepartment类和FinanceDepartment类均是继承自Company类，这两个类的构造函数初始化Company类中的成员name。正确的写法应该是

HRDepartment(string str) : Company(str) {}
FinanceDepartment(string str) : Company(str) {}

三 知识补充

• 派生类不能直接访问基类的私有成员，而必须通过基类方法进行访问。具体的说，派生类构造函数必须使用基类构造函数
• 创建派生对象时，程序首先创建基类对象，这意味着基类对象应当在程序进入派生类构造函数之前被创建。C++使用成员初始化列表语法来完成这种工作。
• 看下面一个例子

class TableTennisPlayer
{
private:
    string firstname;
    string lastname;
    bool hasTable;
public:
    TableTennisPlayer(const string &fn = "none",
        const string &ln = "none", bool ht = false);
};
//基类的构造函数
TableTennisPlayer::TableTennisPlayer(const string & fn, const string & ln, bool ht)
    :firstname(fn),lastname(ln),hasTable(ht)
{

}

class RatedPlayer : public TableTennisPlayer
{
public:
    RatedPlayer(unsigned int r = 0, const string& fn = "none",
        const string& ln = "none", bool ht = false);
private:
    unsigned int rating; //派生类需要一个数据成员存储比分
};
//子类的构造函数
RatedPlayer::RatedPlayer(unsigned int r, const string & fn, const string & ln, bool ht)
    :TableTennisPlayer(fn,ln,ht)
{
    this->rating = r;
}

int main()
{
    return 0;
}

先看RatedPlayer的一个构造函数

//子类的构造函数
RatedPlayer::RatedPlayer(unsigned int r, const string & fn, const string & ln, bool ht)
    :TableTennisPlayer(fn,ln,ht)
{
    this->rating = r;
}

其中TableTennisPlayer(fn,ln,ht)是成员初始化列表，它是可执行的代码，调用TableTennisPlayer构造函数。例如，程序中包含以下声明

RatedPlayer rplayerrl(1140,"Mallory","Duck",true);

则RatedPlayer 构造函数将把实参”Mallory","Duck",true赋给形参fn,ln,ht，然后将这些参数作为实参传递给TableTennisPlayer（父类）的构造函数，后者将创建一个嵌套TableTennisPlayer对象，并将数据“Mallory","Duck",true存储在该对象中。然后，程序进入RatedPlayer（子类）构造函数体，完成RatedPlayer 对象的创建，并将参数r的值赋为rating成员。可以参考下图。

• 将参数传递给基类构造函数