如何初始化地图的复制构造函数？

Question

我已经为此苦苦思考了至少两个小时，我的知识可能在这里遗漏了一些空白。基本上，我必须为复制构造函数编写函数，无论如何我都无法让它工作。我绝对忽略了这里的一些东西，因为这看起来并不那么复杂。

// Definition default constructor
Members::Members()
{
    members = new map<string, set<string>>;
    numOfMembers = 0;
}

// Definition copy constructor
Members::Members(const Members& mem)
{
    numOfMembers = mem.numOfMembers;
}

我似乎根本无法执行这个程序，它在复制构造函数处停止，我只是不知道从这里去哪里。仍然在 STL 地图周围环绕我的头。将不胜感激任何帮助或解释！谢谢！

给定 Header 函数：

class Members
{
    // Overloaded insertion operator (friend)
    friend std::ostream& operator<<(std::ostream& out, const Members& mem);


public:
    // Default constructor
    Members();

    // Copy constructor
    Members(const Members& mem);

Answer 1

如果您在任何一个构造函数中都不需要任何特殊逻辑，请不要定义它们，默认的会自动放置到位，它们就可以很好地完成工作。

如果你必须定义它们：

// Definition default constructor (note: maps are initialized when you define them)
Members::Members():numOfMembers(0)
{
}

// Definition copy constructor
Members::Members(const Members& mem)
{
    numOfMembers = mem.numOfMembers;
    // if you want the map as well
    members = mem.members;
}

Answer 2

由于它是一个构造，您可能需要使用new 创建members（顺便说一句，这里似乎完全没有理由使用它，请参阅farbiondriven 的答案）。然后你必须从其他类复制members。这是故障：

// Definition copy constructor
Members::Members(const Members& mem)
{
    members = new map<string, set<string>>; // Create memebers
    *members = *mem.members; // Copy
    numOfMembers = mem.numOfMembers;
}

这可以通过使用map 的复制构造一步完成，它在一个步骤中创建和复制（参见the reference 中的编号（3））：

// Definition copy constructor
Members::Members(const Members& mem)
{
    members = new map<string, set<string>>(*mem.members); // Call map's copy constructor
    numOfMembers = mem.numOfMembers;
}

不要忘记delete members; 你的类的析构函数。另外，请注意这是不好的设计：您应该真正使用 farbiondriven 的答案，或者至少使用像 std::unique_ptr 这样的自动指针。