双向链表，我似乎可以创建一个新节点，我离我有多远？

Question

该程序假设是一个双向链表，提示用户

1.Insert" "2. Delete"<<endl<< "3. Display"<<endl<< "4. Sum"<<endl<< "5. Average"<<endl<< "6. Exit" From the list above what would you like to do next?"

问题：在执行时，在我为“值”输入双精度值后，程序冻结了一点然后退出

//- 输入提示

cout<<"\\nWhat value would you like to insert to the list: "; cin>>value; l.insertAtEnd(value); break;

//--- 函数定义 ---

 void List::insertAtEnd(double & x) { List::NodePointer ptr; ptr = new List::Node(last, x ); last->next = ptr; last = ptr; count ++; sum += x; }

//--- 原型

包括

    // #include"Node.h" 
>     
>     using namespace std;
>     
>     #ifndef LIST
>     #define LIST
>     
>     typedef double ElementType;
>     
>     class List
>     {
>         public:
>             ...
>             void insertAtEnd(ElementType & x); //insert a value x on the end of the list
>             ...
>         
>     private:
>             ...
>             
>             class Node
>             {
>                 public:
>                 ElementType data;
>                 Node *prev;
>                 Node *next;
>                 //--- Node constructor
>                 /*-------------------------------------------------------------------
>                     Precondition:  None.
>                     Postcondition: A Node has been constructed with value in its data 
>                         part and its next part set to link (default 0).
>                     -------------------------------------------------------------------*/
>                 Node(Node *prevNodePtr,ElementType value, Node *link = 0)
>     
>                     : prev(prevNodePtr) ,data(value), next(link)
>                 {}
>             };
>     
>             typedef Node * NodePointer;
>             NodePointer last;
>             NodePointer first;
>     
>     };
>     
>     ostream & operator<<(ostream & out, const List & s);
>     
>     #endif // !LIST

请帮忙...

Answer 1

我不认为你离得很远。 我看到的主要问题是您没有处理List类中的first指针。

当您像这样向列表末尾添加一个值时，要添加到列表中的第一个值是一种特殊情况，因为您还必须更新第first指针。 您的代码不会这样做。

您的代码的另一个不寻常之处是您拥有这些count和sum变量。 似乎您正在尝试保持列表长度和列表总和的运行总数。 这并没有错，但这是一件不寻常的事情。 通常，您会在需要时计算这些值，而不是一直计算。 此外，如果您继续这样做，那么您需要将count和sum添加到您的列表类中。 目前，您在一个变量中有一个列表，它是不同变量中的长度和总和。 这是糟糕的设计，因为变量之间没有密切关联。 想象一下，你有两个列表。 那么你将如何保持count和sum变量的更新？ 就我个人而言，我只想删除处理count和sum的代码，只需在需要时计算这些数量。