c++ - C++中有序链接列表类的插入功能出现问题

我有一个模板类OList，它是一个有序列表（元素按升序排列）。它具有一个称为void insert(const T & val)的函数，该函数可将元素插入列表中的正确位置。例如，如果我有一个值为{ 1,3,5 }的整数OList并称为insert(4)，则将4插入3和5之间，从而使OList { 1,3,4,5 }。

现在，将元素插入EMPTY OLists时，我的工作正常。但是，当我使用以下代码时：

OList<char> list;
for (int i = 0; i < 3; i++) {
    list.insert('C');
    list.insert('A');
}
printInfo(list);

printList(list)应该输出：

List = { A,A,A,C,C,C }  Size = 6        Range = A...C

相反，它输出：

List = { A,C,C,C,

其次是运行时错误。

我已经把这个弄乱了大约5个小时，但是我似乎没有任何进展（除了得到不同的错误输出和错误之外）。

有三段相关的代码：OList的默认构造函数，operator <

// default constructor
OList() {
    size = 0;
    headNode = new Node<T>;
    lastNode = new Node<T>;
    headNode->next = lastNode;
    lastNode->next = NULL;
}


void insert(const T & val) {
    if ( isEmpty() ) {
        lastNode->data = val;
    }
    else {
        Node<T> * pre = headNode;
        Node<T> * insertPoint = findInsertPoint(pre, val);
        Node<T> * insertNode = new Node<T>;
        insertNode->data = val;
        insertNode->next = insertPoint;
        pre->next = insertNode;

        // why is pre equal to headNode?
        // I thought I changed that when using it
        // with findInsertPoint()
        cout << (pre == headNode) << endl;
    }

    size++;
}

// returns the node AFTER the insertion point
// pre is the node BEFORE the insertion point
Node<T> * findInsertPoint(Node<T> * pre, const T & val) {
    Node<T> * current = pre->next;

    for (int i = 0; (i < getSize()) && (val > current->data); i++) {
        pre = current;
        current = current->next;
    }

    return current;
}

lastNode只是列表中的最后一个节点。
headNode是一个“虚拟节点”，不包含任何数据，仅用作列表的起始位置。

先谢谢了。我真的很尴尬地在互联网上寻求家庭作业的帮助，尤其是因为我确定主要问题是我对指针缺乏透彻的了解。

最佳答案

您将按值传递的指针传递给findInsertPoint，以便将其复制，然后函数更改指针的副本，并且当函数返回时，它仍然是旧的pre，而不是函数内部的pre。

如果要更改指针，则必须将指针传递给该函数的指针（或对指针的引用）。