C++重载<<运算符问题

Question

我对 C++ 相当陌生，但对指针、取消引用等非常满意。我对类的

#include <string>
#include <iostream>

using namespace std;

class Person
{

private:

 string _name;
 Person* _manager;

public:

 Person(string name, Person *manager);
 Person(string name);

 friend ostream &operator<<(ostream &stream, Person &p);

};


Person::Person(string name, Person *manager)
{
 _name = name;
 _manager = manager;
}

Person::Person(string name)
{
 _name = name;
}

ostream &operator<<(ostream &stream, Person &p)
{
 Person* mgr = p._manager;

 stream << p._name << std::endl;
 stream << mgr->_name << std::endl;
 return stream;
}


int main()
{
 Person *pEmployee = new Person("John Doe Employee");
 Person *pManager = new Person("John Doe Manager", pEmployee);

 cout << *pEmployee;
 cout << *pManager;

 return 0;
}

Answer 1

在只有一个参数的构造函数中，您需要将 _manager 设置为 NULL/0。在您的 operatorname。就目前而言，您正在取消引用一个未初始化的指针。

Person::Person(string name)
{
    _name = name;
    _manager = 0;
}

ostream &operator<<(ostream &stream, Person &p)
{
    Person* mgr = p._manager;

    stream << p._name << std::endl;
    if (mgr)
        stream << mgr->_name << std::endl;
    return stream;
}

还有许多其他事情可以做得更好，例如在参数上使用 const 引用和使用构造函数初始化列表，但它们不会成为问题的原因。您还应该解决传递给构造函数的管理器对象的所有权问题，以确保它不会被双重删除。

Answer 2

你的 _manager 指针在构造第一个 Person 实例时没有初始化，因此在你的操作符

Answer 3

您对operator<< 的实现不会检查_manager 成员是否有效。如果没有经理，您应该通过将指针设置为0 来明确这一点。否则指针值是未定义的，访问它会使你的程序崩溃。

(1) 如果没有在构造函数中提供，您的Person 类应该将_manager 设置为0（空指针）：

Person::Person(string name) : _name(name), _manager(0)
{
}

(2) 在您的operator<< 中，在取消引用之前检查指针：

if (mgr) {
    stream << mgr->_name << std::endl;
}

更好的代码的一些提示：

(1) 将您的函数参数更改为接受const string& 而不是string。这样，在调用函数/构造函数时不会复制字符串，而是作为常量引用传递。

(2) 让您的operator<< 也接受const 对Person 的引用会更简洁，因为它不会/不应该修改Person。这样，您也可以在有常量Person 的地方使用该运算符。

Answer 4

Person::Person(string name)
{
    _name = name;
}

你的经理是谁？

Person::Person(string name) : _manager(NULL)
{
    _name = name;
}

ostream &operator<<(ostream &stream, Person &p)
{
 Person* mgr = p._manager;

 stream << p._name << std::endl;
 if (mgr != NULL) { stream << mgr->_name << std::endl; }
 return stream;
}

Answer 5

您的 Person 实例 *pEmployee 没有设置 _manager。这可能是一个 NULL 指针。