0

我正在学校做一项作业,我们应该为我们自己的 List 类创建一个构造函数,该构造函数将初始化列表作为参数。

这就是我希望能够做到的:

List ourList {1, 2, 3};

这是我到目前为止所拥有的:

List::List(std::initializer_list<int> il)
{
  head_ = copy(il.begin(), il.end());
}

List_Node* List::copy(std::initializer_list<int>::iterator begin,
                      std::initializer_list<int>::iterator end)
{
  if(begin == end)
    return nullptr;

  List_Node* new_list = new List_Node(*begin);

  List_Node* node = copy(begin++, end);
  new_list->next_ = node;
  return new_list;
}

在我看来,这应该很好用。但是,当我尝试初始化 ( List list {1,2,3};) 时,我得到一个段错误。有人可以解释一下我在这里做错了什么吗?

4

1 回答 1

6
List_Node* node = copy(begin++, end);

这将copy使用相同的参数再次调用,永远递归并且永远不会完成。

您应该能够通过使用调试器查看它崩溃的位置来判断这一点,并且您会看到对 的调用有数百个List::copy,而不是您预期的三个调用。

++begin不想begin++

于 2014-03-24T13:37:07.303 回答