我不断收到链表的分段错误

Question

该程序旨在生成一个由用户输入的n个数字组成的随机链表，但在尝试打印链表时出现了分段错误。

该程序在显示链表之前正常运行。

#include <iostream>

class node
{
public:
    // TYPEDEF
    typedef double value_type;

    // CONSTRUCTOR
    node(
         const value_type& init_data = value_type(),
         node* init_link = NULL
         )
    { data_field = init_data; link_field = init_link; }

    // Member functions to set the data and link fields:
    void set_data(const value_type& new_data) { data_field = new_data; }
    void set_link(node* new_link)             { link_field = new_link; }

    // Constant member function to retrieve the data:
    value_type data() const { return data_field; }

    // Constant member functions to retreive the link:
    node* linker() const          { return link_field; }

private:
    value_type data_field;
    node* link_field;
};

int myrand(int)
{
    return(1 + rand() %(1000 - 1 +1));
}

void print_linked_list(node*& head_ptr, node*& print_ptr, size_t n)
{
    for (size_t i =1 ; i <= n ; i++) {
        head_ptr = new node(myrand(n), head_ptr);
    }
    std::cout << "Unsorted List: " << std::endl;
    for (print_ptr = head_ptr; print_ptr !=NULL; print_ptr = print_ptr->linker()) {
        std::cout <<  print_ptr->data() << " ";
    }
}

int main()
{
    size_t n;
    srand(time(NULL));

    node* head_ptr;
    node* print_ptr;

    std::cout << "Please input a number" << std::endl;
    std::cin >> n;

    print_linked_list(head_ptr, print_ptr, n);

    return 0;
}

- larman

Answer 1

你正在尝试访问一个未初始化的指针。你将未初始化的变量head_ptr传递给print_linked_list函数，然后在创建第一个节点时使用它作为下一个节点的指针。这意味着条件print_ptr != NULL永远不会满足。

可以通过在main函数中声明head_ptr时将其设置为NULL来解决此问题。

- Kyle A

---

顺便提一下，你还有一个内存泄漏问题，因为你从未删除使用 new 创建的节点。 - Kyle A

Answer 2

head_ptr没有被初始化为NULL。

因此，第一个创建的节点将得到一个垃圾指针作为其link_field。

因此，当您的打印代码尝试遍历链接列表时，最终会遇到垃圾指针，并进入永远不会回来的领域。

- Sam Varshavchik

---

如果您使用nullptr，那就太好了。至少可以促进更多惯用的方式。 :-) - WhiZTiM