在我用来保存我正在制作的基于文本的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。声明和检查指针时,总是有初始化的习惯。

10-08 00:12