gpt4 book ai didi

c++ - 链表段错误。我猜我在这里用指针做错了什么

转载 作者:行者123 更新时间:2023-11-28 08:15:04 24 4
gpt4 key购买 nike

编辑:这是更新的插入代码。我会尝试断言它,但看看我是否仍然在犯错误

bool SortedList::insert(Student *s){    
bool inList = false;
// create an iterator for the list
ListNode *current = head;
// create a new ListNode for the student to be added
ListNode *addition = new ListNode();
// initialize addition to the student and the next to NULL
addition->student = s;
addition->next = NULL;
//while current is not at the end of the list
while(current != NULL){
// if the iteration's ID is equal to the given ID return
// false and delete the ListNode addition
if(current->student->getID() == addition->student->getID()){
delete addition;
return false;
// else if the next student ID in the list is greater than
// the given ID break the while loop
}else if(current->next != NULL && current->next->student->getID() > addition->student->getID()){
inList = true;
break;
}
// otherwise set current to the next student in the list
current = current->next;
}
// if current is at the end of the list and student wasn't found, set
// current next to addition
if(!inList){
current->next = addition;
// else set addition next to current next next and current next to addition
}else{
addition->next = current->next;
current->next = addition;
}
// return true regardless as the student has been added
return true;
}

我在处理这个基本的 linkedlist.cpp 文件时遇到了一些问题。我猜我可能使用了错误的指针或类似的东西。主文件作为目标文件给出,所以我无法查看它,出于某种原因,我只想测试插入方法(我的 sleep 和挣扎非常低)。不管怎样,我猜我在插入方法的某个地方错误地引用了 NULL,但我不知道在哪里。

错误:段错误(核心已转储)注意:运行插入后出现此错误

#include <iostream>
#include "SortedList.h"

using namespace std;

/**
* zero argument constructor - initializes an empty list
*/
SortedList::SortedList() : head(NULL){}

/**
* If a student with the same ID is not already in the list, inserts
* the given student into the list in the appropriate place and returns
* true. If there is already a student in the list with the same ID
* then the list is not changed and false is returned.
*
* @param *s a given pointer to a student
* @return boolean value based on whether the student was inserted or not
*/
bool SortedList::insert(Student *s){
// create an iterator for the list
ListNode *current = head;
// create a new ListNode for the student to be added
ListNode *addition = new ListNode();
// initialize addition to the student and the next to NULL
addition->student = s;
addition->next = NULL;
//while current is not at the end of the list
while(current != NULL){
// if the iteration's ID is equal to the given ID return
// false and delete the ListNode addition
if(current->student->getID() == addition->getID()){
return false;
delete addition;
// else if the next student ID in the list is greater than
// the given ID break the while loop
}else if(current->next->student->getID() > addition->getID()){
break;
}
// otherwise set current to the next student in the list
current = current->next;
}
// if current is at the end of the list and student wasn't found, set
// current next to addition
if(current == NULL){
current->next = addition;
// else set addition next to current next next and current next to addition
}else{
addition->next = current->next->next;
current->next = addition;
}
// return true regardless as the student has been added
return true;
}

/**
* Searches the list for a student with the given student ID. If the
* student is found, it is returned; if it is not found, NULL is returned.
*
* @param studentID the given studentID to find in the list
* @return a pointer to a the student found or NULL if the student isn't found
*/
Student * SortedList::find(int studentID){
// create iterator for the list
ListNode *current = head;
// while not at the end of the list iterate
while(current != NULL){
// if the current student ID equals the given student ID return
// the student
if(current->student->getID() == studentID){
return current->student;
}
// otherwise continue iterating
current = current->next;
}
// if not found then return NULL
return NULL;
}

/**
* Searches the list for a student with the given student ID. If the
* student is found, the student is removed from the list and returned;
* if no student is found with the given ID, NULL is returned.
*
* @param studentID the given student ID to be removed from the list
* @return a pointer to the student that was removed or NULL if the student
* wasn't found
*/
Student * SortedList::remove(int studentID){
// create iterator for the list
ListNode *current = head;
// create the to hold the value ahead of the iterator
ListNode *currentNext;
// create to hold the removed student
Student *remove;
// while current is not at the end of the list iterate
while(current != NULL){
// set currentNext to the value ahead of iterator
currentNext = current->next;
// if its ID equals the given ID
if(currentNext->student->getID() == studentID){
// set remove to the student removing
remove = currentNext->student;
// set current next to currentNext next
// (current next next)
current->next = currentNext->next;
//delete the removed ListNode
delete currentNext;
//return the removed student
return remove;
}
}
//if the student wasn't found return NULL
return NULL;
}

/**
* Prints out the list of students to standard output. The students are
* printed in order of student ID (from smallest to largest), one per line
*/
void SortedList::print() const{
//create iterator for list
ListNode *current = head;
//while current is not at the end of the list iterate
while(current != NULL){
// print each individual student and end line
current->student->print();
cout << endl;
//iterate the list
current = current->next;
}
}

最佳答案

   while(current != NULL){
// if the iteration's ID is equal to the given ID return
// false and delete the ListNode addition
if(current->student->getID() == addition->getID()){
2 return false;
delete addition;
// else if the next student ID in the list is greater than
// the given ID break the while loop
1 }else if(current->next->student->getID() > addition->getID()){
break;
}

在我标记为 1 的行中,您取消引用 current->next 而无需检查它是否为 NULL。此外,在我标记为 2 的行,您结束执行,然后然后删除指针。您应该在返回之前删除

if(current == NULL){
3 current->next = addition;
// else set addition next to current next next and current next to addition
}else{
4 addition->next = current->next->next;
current->next = addition;
}

在标记为 3 的行中,仅当它为 NULL 时才取消引用 current。坏枣。在标记为 4 的行中,您取消引用 current->next 而不先检查它是否为 NULL。我认为您打算将 addition->next 设置为 current->next 无论如何。

关于c++ - 链表段错误。我猜我在这里用指针做错了什么,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/7932693/

24 4 0
Copyright 2021 - 2024 cfsdn All Rights Reserved 蜀ICP备2022000587号
广告合作:1813099741@qq.com 6ren.com