Question

今天在我的课堂上，我们进行了继承练习。我们编写了一个继承自Person类继承的Student类的UnderGrad类。

Person为班级提供了3个变量名称，地址和生日。

学生有身份证号码，专业和学位类型。

最后，本科生还有一所高中，并获得了学分。

当我为每个类编写头文件时，默认构造函数看起来像这样：

   //snippet from UnderGrad.h
 Undergrad(string nm = "", string add = "", string bday = "", int id = 0, string maj = "", string degtyp = "", string hs = "", int cred = 0) : Student(nm,add,bday,id,maj,degtyp) {

     highSchool = hs;
     credits = cred;
};


//snippet from Student.h
Student(string nm = "", string add = "", string bday = "", int id = 0, string maj = "", string degtyp = "") : Person(nm,add,bday){
     stuId = id;
     major = maj;
     degreeType = degtyp;
};

//snippet from Person.h
Person(string nm = "", string add = "", string bday = ""){
     name = nm;
     address = add;
     bDay = bday;
};

我的教授说这是不可读的，并不是真的这样做。

我想知道，以这种方式创建默认构造函数有什么问题？有什么不对的吗？有没有更好的方法呢？

Answer 1

如果没有代码上下文的好处，我会对这些构造函数进行一些更改。

我会考虑将构造函数的定义移动到单独的源文件中。这将允许更改定义，而不会触发重新编译其余代码。对于小程序，或非常简单的构造函数，如果没有标题也不会改变，这可能不值得付出努力。
我会更改构造函数的定义以使用成员初始化列表，而不是在构造函数体中分配成员。（请参阅Als的答案以获得解释）。
我会删除函数定义末尾的分号，完全不需要它们。
我会将长构造函数分成几行，以避免过长的代码行（长行可能不适合您的编辑器，并且很难执行差异和合并）。
< / LI>
您的头文件中似乎有using namespace ::std或using ::std::string。这是不好的做法，因为这意味着包含您的标头的代码会应用此using。这可能会使客户端代码的含义发生变化，或使客户端代码变得模糊不清并无法编译。我会删除using，而是使用std::string的限定名称。

这些变化付诸实施：

//snippet from UnderGrad.h
Undergrad(
    std::string nm = "",
    std::string add = "",
    std::string bday = "",
    int id = 0,
    std::string maj = "",
    std::string degtyp = "",
    std::string hs = "",
    int cred = 0) :
        Student(nm,add,bday,id,maj,degtyp),
        highSchool(hs),
        credits(cred)
{
}

//snippet from Student.h
Student(
    std::string nm = "",
    std::string add = "",
    std::string bday = "",
    int id = 0,
    std::string maj = "",
    std::string degtyp = "") :
      Person(nm,add,bday),
      stuId(id),
      major(maj),
      degreeType(degtyp)
{
}

//snippet from Person.h
Person(std::string nm = "", std::string add = "", std::string bday = "") :
    name(nm), address(add), bDay(bday)
{
}

Answer 2

语义上很好，但你可以使用成员字符串的初始化器而不是分配每个成员字符串。

在风格上，你的线条有点长，但这就是我所能解决的所有问题。

默认构造函数都封装在头文件中

2 个答案: