#include <iostream> #include <string> using namespace std; class Date { friend class person; public: int year,month,day; friend ostream & operator <<( ostream&os, const Date&date ) { os<<date.year<<"_"<<date.month<<"_"<<date.day; return os; } Date(int year=2000,int month=1,int day=1):year(year),month(month),day(day) {} void print() { cout<<year<<","<<month<<","<<day; } }; class person { public: string Name; string Num; Date brith; person(string Name="张三", string Num="123456789", int year=2000,int month=1,int day=1): Name(Name),Num(Num),brith(year,month,day) {} string GetName() { return Name; } string GetNum() { return Num; } Date Getbrith() { return brith; } }; bool compare(person&p1,person&p2) { if(person&p1.year>person&p2.year) { return true; } else { return false; } } if(compare(person&p1.year,person&p2.year)) { cout<<"p1的年龄大"<<endl; } else { cout<<"p2的年龄大"<<end; } class student: private person { private: int StuNum; }; class teacher:public person { private: Date Time; }; class CTA :public student,public teacher { private: int No; }; int main() { person p1("张三","123456789",2000,1,1); person p2("李四","123456789",2001,1,1); cout<<"序号"<<endl; cout<<" 1 "<<endl; cout<<" "<<p1.GetName()<<endl; cout<<" "<<p1.GetNum()<<endl; cout<<" "<<p1.Getbrith()<<endl; cout<<" 2 "<<endl; cout<<" "<<p2.GetName()<<endl; cout<<" "<<p2.GetNum()<<endl; cout<<" "<<p2.Getbrith()<<endl; }有什么问题
时间: 2024-04-26 17:21:57 浏览: 56
该代码存在以下问题:
1. 在比较函数 `compare` 中,使用了 `person&p1.year` 和 `person&p2.year`,这个语法是错误的,应该是 `p1.brith.year` 和 `p2.brith.year`。
2. 类 `student` 和 `teacher` 的继承方式应该改为 `public`,否则无法访问基类中的成员函数和数据成员。
3. 类 `CTA` 继承了 `student` 和 `teacher`,但是这两个类都继承了 `person`,这就导致 `CTA` 中有两份 `person` 的数据成员,这样会导致访问不确定性和冗余。应该将 `student` 和 `teacher` 改为继承自一个共同的基类,这个基类中包含 `person` 类型的数据成员和成员函数。
4. 在 `main` 函数中,没有输出序号 3。
5. 在输出日期时,应该使用 `brith.print()` 而不是 `cout<<brith`,因为 `brith` 是一个 `Date` 类型的对象,没有定义 `<<` 运算符重载。
6. 在输出年龄大小时,应该使用 `p1.Getbrith().year` 和 `p2.Getbrith().year` 来获取出生年份,然后进行比较,而不是使用 `compare` 函数。
7. 类 `CTA` 中的数据成员 `No` 没有被使用,可能是不必要的。
阅读全文