在我用来保存我正在制作的基于文本的rpg的清单的用户创建的链接列表中,我的链接列表中的delete函数遇到错误。我想知道是否有人可以指出为什么我会收到此错误。
void InventoryList::deleteNode(int num)
{
ListNode *previousNode; //To point to the previous node
ListNode *nodePtr; //to traverse the list
int number = 1;
//if the head is empty do nothing
if (!head)
{
return;
}
//Determine if the first node is the value
if (1 == num)
{
nodePtr = head->next;
delete head;
head = nodePtr;
}
else
{
//intialize the node as head.
nodePtr = head;
//Skip nodes whose value is not equal to num.
while (nodePtr != nullptr && number != num)
{
previousNode = nodePtr;
nodePtr = nodePtr->next;
number++;
}
if (nodePtr)
{
previousNode->next = nodePtr->next;
delete nodePtr;
}
}
}
我在代码的最终if语句中遇到此错误,错误是:
错误C4703:使用了可能未初始化的本地指针变量'previousNode'
最佳答案
R Nar的评论是对的。初始化PreviousNode的while循环不能保证运行,因此指针可能未初始化-因此,您的错误消息。
要修复它,您需要在一个保证运行的块中初始化PreviousNode-否if / while / etc。
这可以在任何地方进行,但是我建议您在声明它时将其设置为NULL,并检查以确保在使用它时不为null。声明和检查指针时,总是有初始化的习惯。