void retLast(ListNode* list1) { while (list1->next != nullptr) { list1 = list1->next; } }为何返回的还是最初的头节点
时间: 2023-12-10 09:39:32 浏览: 90
这是因为函数`retLast`中只是对`list1`进行了遍历,但并没有将最后一个节点返回。如果你想要返回最后一个节点,应该将遍历到的最后一个节点返回,而不是只对`list1`进行操作。可以修改函数代码如下:
```c++
ListNode* retLast(ListNode* list1) {
while (list1->next != nullptr) {
list1 = list1->next;
}
return list1;
}
```
这样,函数就会返回最后一个节点的指针了。
相关问题
#include <iostream> #include "listnode.h" using namespace std; /*struct listNode { listNode * prev, * next; int val; listNode(): val(0), prev(nullptr), next(nullptr){} listNode(int v, listNode *p, listNode *n): val(v), prev(p), next(n) { if (prev != nullptr) prev->next = this; if (next != nullptr) next->prev = this; } };*/ class OrderedList { protected: listNode * root; public: OrderedList(){root=new listNode();} ~OrderedList(){ listNode* cur=root; if(cur->prev!=nullptr)cur=cur->prev; if(cur!=nullptr){listNode*tmp=cur;cur=cur->next;delete tmp;} delete cur;} virtual void insert(int val)=0; void printList() const { listNode* cur=root; while(cur->prev!=nullptr)cur=cur->prev; while(cur!=nullptr){cout<<cur->val<<' ';cur=cur->next;} cout<<endl; } }; class AscendOrderedList:public OrderedList { public: AscendOrderedList(){root=new listNode();} ~AscendOrderedList(){ listNode* cur=root; if(cur->prev!=nullptr)cur=cur->prev; if(cur!=nullptr){listNode*tmp=cur;cur=cur->next;delete tmp;} delete cur;} void insert(int v) { if(root->next==nullptr&&root->prev==nullptr){root->val=v;return;} listNode* pre = root; while(pre->prev!=nullptr)pre=pre->prev; if(v<=pre->val){listNode*ins=new listNode();ins->val=v;ins->next=pre;return;} while (pre->next != nullptr && v > pre->val) { pre = pre->next; } listNode*a=pre->next; listNode*ins=new listNode(v, pre, pre->next); ins->prev=pre;ins->next=a; } }; class DescendOrderedList:public OrderedList { public: DescendOrderedList(){root=new listNode();} ~DescendOrderedList(){ listNode* cur=root; if(cur->prev!=nullptr)cur=cur->prev; if(cur!=nullptr){listNode*tmp=cur;cur=cur->next;delete tmp;} delete cur;} virtual void insert(int v){ if(root->next==nullptr&&root->prev==nullptr){root->val=v;return;} listNode*pre=root; while(pre->prev!=nullptr)pre=pre->prev; if(v>=pre->val){listNode*ins=new listNode();ins->val=v;ins->next=pre;return;} while(v<pre->val&&pre!=nullptr){pre=pre->next;} listNode*a=pre->next; listNode*ins=new listNode(v, pre, pre->next); ins->prev=pre;ins->next=a; } };输出错误
在代码中,构造函数和析构函数中的删除操作存在问题。在析构函数中,应该先删除链表中的节点,再删除root节点,否则会造成内存泄漏。同时,在构造函数中,应该将root节点的prev和next指针设为nullptr,否则会导致插入操作时出现问题。另外,在DescendOrderedList的insert函数中,应该先判断pre是否为nullptr,否则在pre为nullptr时会出现访问pre->val的错误。
#include <iostream> #include "listnode.h" using namespace std; struct listNode { listNode * prev, * next; int val; listNode(): val(0), prev(nullptr), next(nullptr){} listNode(int v, listNode *p, listNode *n): val(v), prev(p), next(n) { if (prev != nullptr) prev->next = this; if (next != nullptr) next->prev = this; } }; class OrderedList { protected: listNode * root; public: OrderedList(){root=new listNode();} ~OrderedList(){ listNode* cur=root; if(cur->prev!=nullptr)cur=cur->prev; if(cur!=nullptr){listNode*tmp=cur;cur=cur->next;delete tmp;}} virtual void insert(int val)=0; void printList() const { listNode* cur=root; if(cur->prev!=nullptr)cur=cur->prev; if(cur!=nullptr){cout<<cur->val<<' ';cur=cur->next;} } }; class AscendOrderedList:public OrderedList { public: AscendOrderedList(){root=new listNode();} ~AscendOrderedList(){ listNode* cur=root; if(cur->prev!=nullptr)cur=cur->prev; if(cur!=nullptr){listNode*tmp=cur;cur=cur->next;delete tmp;}} virtual void insert(int v){ listNode*pre=root; while(v>pre->val&&pre!=nullptr){pre=pre->next;} listNode*ins=new listNode(v, pre,pre->next); pre->next=ins; ins->next->prev=ins; } }; class DescendOrderedList:public OrderedList { public: DescendOrderedList(){root=new listNode();} ~DescendOrderedList(){ listNode* cur=root; if(cur->prev!=nullptr)cur=cur->prev; if(cur!=nullptr){listNode*tmp=cur;cur=cur->next;delete tmp;}} virtual void insert(int v){ listNode*pre=root; while(v<pre->val&&pre!=nullptr){pre=pre->next;} listNode*ins=new listNode(v, pre, pre->next); pre->next=ins; ins->next->prev=ins; } }为什么运行崩溃
这段代码中可能会有多个问题,具体原因需要根据您的输入、程序运行环境以及错误提示等来确定。以下是一些可能的问题和建议:
1. 在 `listNode` 的构造函数中,需要判断 `prev` 和 `next` 是否为空指针,才能为它们的 `prev` 和 `next` 成员变量赋值。修改代码如下:
```
listNode(int v, listNode* p, listNode* n) : val(v), prev(p), next(n) {
if (prev != nullptr) {
prev->next = this;
}
if (next != nullptr) {
next->prev = this;
}
}
```
2. 在 `OrderedList` 和其子类的析构函数中,需要循环删除链表中的所有节点。修改代码如下:
```
~OrderedList() {
listNode* cur = root;
while (cur->next != nullptr) {
cur = cur->next;
delete cur->prev;
}
delete cur;
}
~AscendOrderedList() {
listNode* cur = root;
while (cur->next != nullptr) {
cur = cur->next;
delete cur->prev;
}
delete cur;
}
~DescendOrderedList() {
listNode* cur = root;
while (cur->next != nullptr) {
cur = cur->next;
delete cur->prev;
}
delete cur;
}
```
3. 在 `AscendOrderedList` 和 `DescendOrderedList` 的 `insert()` 方法中,需要先判断当前节点是否为空指针,否则会出现访问空指针的错误。修改代码如下:
```
void insert(int v) {
listNode* pre = root;
while (pre->next != nullptr && v > pre->val) {
pre = pre->next;
}
listNode* ins = new listNode(v, pre, pre->next);
pre->next = ins;
if (ins->next != nullptr) {
ins->next->prev = ins;
}
}
```
```
void insert(int v) {
listNode* pre = root;
while (pre->next != nullptr && v < pre->val) {
pre = pre->next;
}
listNode* ins = new listNode(v, pre, pre->next);
pre->next = ins;
if (ins->next != nullptr) {
ins->next->prev = ins;
}
}
```
以上是可能的问题和建议,如果您还有其他问题或者需要更多帮助,请提供更多详细的信息和错误提示,以便我们更好地帮助您解决问题。
阅读全文