無限循環鏈表C

Question

我正在研究一個練習問題以提高我的C。這是問題：

編寫一個函數來對整數的鏈表進行排序，如下所示：

a）在列表中找到最大值。

b）從其位置刪除它，並將其插入列表的開頭。

c）從現在的第二個元素開始，重復（a）和（b）。

d）從現在的第三個元素開始，重復（a）和（b）。

繼續，直到列表已排序。

我的排序功能：

NodePtr sortList(NodePtr np) {
  NodePtr makeNode(int), head, temp;

  head = temp = np;

  if (np != NULL) {
    // Determine linked list length
    int len = 0;
    while (temp != NULL) {
        len++;
        temp = temp -> next;
    }

    int i;
    for (i = 0; i < len; i++) {
        temp = head;
        int j;
        // Traverse to spot j in linked list
        for (j = 0; j < i; j++) {
            temp = temp -> next;
        }

        // Find largest
        NodePtr largest, prev, prevToLargest;
        largest = prev = prevToLargest = temp;
        while (temp != NULL) {
            if (temp -> num > largest -> num) {
                largest = temp;
                prevToLargest = prev;
            }

            if (temp != prev) prev = prev -> next;
            temp = temp -> next;
        }

        prevToLargest -> next = largest -> next;
        largest -> next = head;
        head = largest;
    }
  }

  return head;
}

使用調試器，我發現我的列表已排序，但它是無限的。 我相信我已經創建了一個循環鏈表，但不確定如何解決。

謝謝您的幫助！

Answer 1

有幾件事。 首先，當要求人們調試代碼嘗試包括行號時，它使我們更容易引用問題。 現在可以解決實際問題。

1. if (temp != prev) prev = prev -> next; 在那行代碼上，只需說prev = temp，而不是prev-> next，因為temp始終是下一個，所以這不是一個大問題，但是我認為這樣做更容易。

2. 好的，因此您可以通過使largest->next = head插入最大的largest->next = head 。 但是插入點之前仍指向頭部而不是最大的索引該怎么辦，因此鏈接列表中的鏈接已損壞。 ***我在談論prevToLargest -> next = largest -> next;之后的代碼prevToLargest -> next = largest -> next; 接近尾聲。

3. 當您返回頭部時，頭部被多次移動。 因此，在您遍歷整個列表時，您的代碼將head移到了列表中的最后一個元素，因為head = largest; 最后一個最大的元素將是列表末尾的最小元素。 因此，在返回head之前，請嘗試將其指向列表中的第一個元素。

查看是否可以解決您的問題。 可能會有更多的錯誤，但是這些錯誤對我很突出。