我正在尝试使用插入排序在列表中以排序方式插入节点

Question

我正在尝试在 Linked List 上的 C# 中实现桶排序算法，但我得到了正确的结果。此函数应将 List 和 Node 作为参数，然后以正确的升序插入该节点。 我有一个列表 [4, 7, 12, 15]，我希望以升序插入数据为 20 的新节点，例如此列表变为 [4,7,12,15,20]。但结果我得到此列表[4,7,12,20,15] 请帮助我，其中有什么问题。我猜while循环里面有一些问题。此函数调用的所有其他函数都完美无缺

    static public void InsertionSort(LinkedList list, Node s)
            {
                int data = s.getData();
                if (list.start.getNext() == null)
                    list.InsertAtEnd(data);
                else
                {
                    int key = 0; 
                    Node temp = list.start;
                    while (temp.getNext() != null)
                    {
                        temp = temp.getNext();
                        if (data >= temp.getData())
                            key++;

                    }
                    list.InsertAt(key, data);
                }
            }

派生函数（主函数）：

LinkedList list = new LinkedList();
            int[] array = { 4, 7, 12, 15 };
            for (int i = 0; i < array.Length; i++)
                list.InsertAtEnd(array[i]);
            System.Console.WriteLine("Before Insertion");
            list.Display();
            Node n = new Node(20);
            InsertionSort(list, n);
            System.Console.WriteLine("After Insertion");
            list.Display();

输出：

1:Output of The Code

Answer 1

一开始的设置是错误的。

int[] array = { 4, 7, 12, 15 };
for (int i = 0; i < array.Length; i++)
    list.InsertAtBegin(array[i]);

因为您遍历了初始数组并在开头添加了所有项目，所以结果列表的排序顺序错误。

要从正确的排序开始，请在末尾添加：

int[] array = { 4, 7, 12, 15 };
for (int i = 0; i < array.Length; i++)
    list.InsertAtEnd(array[i]);

或者从另一个方向遍历初始数组：

int[] array = { 4, 7, 12, 15 };
for (int i = array.length - 1; i >= 0; i--)
    list.InsertAtBegin(array[i]);

---

另外，您的插入算法不是最优的。

对于链表，要到达列表中的某个元素是昂贵的，因为您总是必须从开始到正确的位置。

您的代码首先尝试找到插入新元素的正确位置，然后调用另一个方法InsertAt，这将不得不从头开始遍历列表以再次找到相同的位置。

改为更改代码，以便在找到具有更高值的元素后立即插入新节点。

Answer 2

感谢所有试图帮助我的人。我有正确的代码。现在它正在产生 100% 准确的结果。

static public void InsertionSort(LinkedList list, int data)
        {
            Node n = new Node(data);
            Node temp = list.start;
            Node sNode = null;
            while(temp.getNext() !=null)
            {
                if (data >= temp.getNext().getData())
                    sNode = temp.getNext();
                temp = temp.getNext();
            }
            if (sNode == null)
                list.InsertAtBegin(data);
            else
                list.InsertAfter(sNode,n);
        }