使代码更小以实现多重继承

Question

我编写了一个非常小的程序，您可以在其中输入您是男孩还是女孩，然后它会打印出一个声明。我的主要问题是，从我的代码中，除了从基类复制和粘贴之外，还有什么更简单的方法可以为女性编写代码。 这是我的代码

#include <iostream>
#include <string>


class Man{
protected:
    std::string name;
public:
    void getInfo(std::string hName){
        name = hName;
    }
    void showInfo(){
        std::cout << "Your name is: " << name << std::endl;
        std::cout << "And you are a MAN" << std::endl;
    }
};

class Women{ //is there an easier way to write this         
protected:
    std::string fem_name;
public:
    void getfemInfo(std::string fhName){
        fem_name = fhName;
    }
    void showfemaleInfo(){
        std::cout << "Your name is: " << fem_name << std::endl;
        std::cout << "And you are a Women" << std::endl;    
    }
};

class Human:public Man, public Women{
public:
    Human(){}
};

int main(){
    //local variables
    std::string choice;
    std::string tName;
    //declaring objects 
    Human person;
    //user interface
    std::cout << "Please enter you name: ";
    std::cin >> tName;
    std::cout << "Are you a [boy/girl]: ";
    std::cin >> choice;
    //if handler
    if (choice == "boy"){
        person.getInfo(tName);
        person.showInfo();
    }else if(choice == "girl"){
        person.getfemInfo(tName);
        person.showfemaleInfo();
    }
    system("pause");
    return 0;
}

当我尝试从类Man 派生类Woman 时，它使person.getInfo(tName) 和person.showInfo() 模棱两可。这是为什么？以及如何使这段代码更小（针对女性）。

Answer 1

你把它弄反了——公共继承代表 IS-A 关系。所以你的用法是说“每个人都是男人，每个人都是女人。”这真的行不通。公共继承应该永远被用作“在一个地方获得所有功能”的便利。这就是组合（或者最坏的情况是非公共继承）的用途。

不过，您的情况是使用继承的一个很好的例子。它只需要遵循其自然定义（每个男人是人；每个女人都是人）。也就是说，将Human 设为基类并在那里实现所有共享功能。像这样的：

class Human
{
protected:
  std::string name;

  virtual std::string getGenderString() const = 0;

public:
  virtual ~Human() {}  //dtor must be virtual to be usable as a polymorphic base class

  void getInfo(std::string hName)
  { name = hName; }

  void showInfo() const
  {
    std::cout << "Your name is: " << name << '\n';
    std::cout << "And you are a " << getGenderString() << std::endl;
  }
};


class Man : public Human
{
protected:
  virtual std::string getGenderString() const
  { return "MAN"; }
};


class Woman : public Human
{
protected:
  virtual std::string getGenderString() const
  { return "WOMAN"; }
};


int main(){
    //local variables
    std::string choice;
    std::string tName;
    //declaring objects 
    Human *person = NULL;
    //user interface
    std::cout << "Please enter you name: ";
    std::cin >> tName;
    std::cout << "Are you a [boy/girl]: ";
    std::cin >> choice;
    //if handler
    if (choice == "boy"){
        person = new Man();
    }else if(choice == "girl"){
        person = new Woman();
    }
    person->getInfo(tName);
    person->showInfo();
    system("pause");
    delete person;
    return 0;
}

以上代码使用纯虚函数（必须在派生类中实现）来获取适当的性别字符串。其他所有内容对两种性别都是通用的，所以它在基类中。

请注意，有很多好的做法可以添加到代码中，但我不想过多地混淆这个问题。我不知道您是否可以访问 C++11 功能，所以我没有使用任何功能。将上述（工作）代码转变为好的代码的最佳选择是选择good C++ book。

Answer 2

目前 Human 类确实代表两种性别，这在现实世界中是不正确的。它应该是相反的方式：一个 Human 类用作基础类，持有名称并提供可以在 Woman/Man 类中被覆盖的虚拟方法，因为它实际上是：成人：P 人类要么是男人或一个女人。目前的设计真的没有多大意义。

另外，name 是男女通用的属性，和操作它的方法一样，所以应该去基类。你可以选择模板方法设计模式，所以最终你的代码可以这样改进：

class Human
{
  std::string name;
  virtual std::string GetGender() const = 0;
public:
    // virtual d-tor

    Human(const std::string& humanName)
      : name(humanName) {}

    void showInfo() const 
    {
        std::cout << "Your name is: " << name << std::endl;
        std::cout << "And you are a " << GetGender() << std::endl;
    }
};

class Man : public Human{
  /*virtual*/ std::string GetGender() const { return "MAN"; }
public:
  Man(const std::string& name)
    : Human(name) {}
};

class Woman : public Human{
  /*virtual*/ std::string GetGender() const { return "WOMAN"; }
public:
  Woman(const std::string& name)
    : Human(name) {}
};

然后你可以在对指向特定派生类的基类指针进行操作时轻松打印所需的属性：

humanObject->showInfo();