以有序方式在链接列表中间插入元素吗？

Question

我得到了我的链表：

typedef struct t_node {
    ELEMLIST data;
    struct t_node *next;
} NODE;

typedef NODE *LIST;

而且我尝试以有序方式（从较小的数字到较大的数字）插入整数，但似乎有些事情不起作用：

STATUS insertInOrderList(LIST *list, const ELEMLIST *pElem) {

    NODE *newNode, *nodeIns;

    newNode = getNode();//allocate memory
    nodeIns = getNode();

   //checkout getnode return

    newNode->data = *pElem;

    for (nodeIns = *list; nodeIns != NULL; nodeIns = nodeIns->next) {
        if (cmpEleList(&newNode->data, &nodeIns->data) != 1) { 
             //if arg1 is not > arg2 it breaks the loop
            break;
        }
    }

    newNode->next = nodeIns;
    nodeIns->next = newNode;

    return OK;
}

当我运行它时，它只是告诉我我的列表为空...

我敢肯定，这只是我错过的一些细节，但是我无法意识到

Answer 1

您的代码错误地完成了几件事：

• 您正在分配两个节点。 不是一个（这不是Java或C＃）。
• 您无需考虑列表中的第一个节点可能已经比传入节点“更大”的可能性。
• 您没有正确连接新节点。

我完全不知道您的比较功能如何工作。 该代码似乎表明只要列表第一个值“小于”第二个就返回1（这与大多数比较器的工作方式正好相反。大多数这样做：

• lhs <rhs：返回<0
• lhs == rhs：返回0
• lhs> rhs：返回> 0

尽管如此，我还是对您的算法进行了改进。

STATUS insertInOrderList(LIST *list, const ELEMLIST *pElem) 
{

    NODE *newNode = getNode();
    newNode->data = *pElem;

    while (*list && cmpEleList(&(*list)->data, pElem) != 1)
        list = &(*list)->next;

    newNode->next = *list;
    *list = newNode; 

    return OK;
}

这假设列表为空， *list将为NULL，并且分配成功。 量身定制，随心所欲。 祝你好运。