0

所以我一直在尝试使用合并排序对链表进行排序，我找到了这段代码并尝试处理它，但它并没有真正起作用？

它可能有什么问题？我不太确定 getMiddle 方法，尽管我知道它应该获取列表的中间值以便处理列表本身的 2 个列表

这是代码；

public Node mergeSort(Node head) {

    if (head == null || head.link == null) {
        return head;       
    }

    Node middle = getMiddle(head);

    Node sHalf = middle.link;
    middle.link = null;       

    return merge(mergeSort(head), mergeSort(sHalf));
}

public Node merge(Node a, Node b) {
    Node dummyHead;
    Node current;

    dummyHead = new Node();
    current = dummyHead;

    while (a != null && b != null) {
        if ((int) a.getData() <= (int) b.getData()) {
            current.link = a;
            a.link = a;
        }
        else {
            current.link = b;
            b.link = a;
        }
        current = current.link;
    }
    current.link = (a == null) ? b : a;
    return dummyHead;
}

public Node getMiddle(Node head) {
    if (head == null) {
        return head;
    }
    Node slow, fast;
    slow = fast = head;
    while (fast.link != null && fast.link.link != null) {
        slow = slow.link;
        fast = fast.link.link;
    }
    return slow;
}

在主要方法中：

  Object data;
    MyLinkedList list = new MyLinkedList();         //empty list.


    for (int i = 0; i < 3; i++) {           //filling the list
        data = console.nextInt();
        list.insertAtFront(data);
    }


    System.out.print("Print(1): ");
    list.printList();

   list.mergeSort(list.getHead());
    System.out.print("List after sorting: ");
    list.printList();
4

1 回答 1

1

一个问题是该getMiddle方法没有正确返回 middle Node。

考虑一个有 5 个节点 (a, b, c, d, e) 的链表

head, slow, 并且fast都从索引 0 (a) 开始。

在while循环的第一次迭代之后，slow位于 1 (b) 和fast位于 2 (c)；在第二次迭代之后，slow在 2 (c) 和 fast 在 4 (e)。这些都不是空的，所以会发生另一次迭代，在 3 (d) 处放慢，在 3 处放快null。由于fast为null，因此while退出循环并slow返回；但是slow有节点 3 (d) 而不是中间节点 2 (c)。

获取中间节点的另一种方法是简单地使用节点数：

public Node getMiddle(Node head) {
    Node counter = head;
    int numNodes = 0;
    while(counter != null) {
        counter = counter.link;
        numNodes++;
    }
    if(numNodes == 0)
        return null;
    Node middle = head;
    for(int i=0; i<numNodes/2; i++)
        middle = middle.link;
    return middle;
}

我你的mergeSort方法很好，但从技术上讲，它只需要返回headif head.linkis null，而不是 ifhead本身是null（因为无论如何这永远不会发生）：

public Node mergeSort(Node head) {
    if (head.link == null) {
        return head;       
    }
    // same
}

最重要的是，你的merge方法。您可以在您的类中编写一个public void setData(Object)方法Node来简化此操作。以下代码应该可以工作，尽管我不能声称它是完成这项工作的最佳/最有效的方法

public Node merge(Node a, Node b) {
    Node combined = new Node();
    Node current = combined;
    while(a != null || b != null) {
        if(a == null)
            addNode(current, b);
        if(b == null)
            addNode(current, a);
        if((int)a.getData()<(int)b.getData())
            addNode(current, a);
        else
            addNode(current, b);
    }
    return combined;
}

使用以下辅助方法：

public void addNode(Node n1, Node n2) {
    n1.setData((int)n2.getData());
    n1.link = new Node();
    n1 = n1.link;
    n2 = n2.link
}
于 2013-10-31T15:47:52.547 回答