对字符串的链接列表进行排序

Question

public class LinkedList {
    private Node top;

    public LinkedList() {
        top = null;
    }

    public void add(String data) {
        Node temp = new Node(data, top);
        top = temp;
    }

    public void sort() {
        LinkedList sortedList = new LinkedList();
        Node i;
        Node j;
        String temp;
        String temp2;
        String temp3;
        for (i = top; i != null; i.getNext()) {
            for (j = i.getNext(); j != null; j.getNext()) {
                if (i.getData().compareTo(j.getData()) < 0) {
                    temp = i.getData();
                    i.setData(temp);
                    temp2 = j.getData();
                    j.setData(temp2);
                    temp3 = temp;
                    temp = temp2;
                    temp2 = temp3;
                    sortedList.add(temp3);
                } else if (i.getData().compareTo(j.getData()) > 0) {
                    temp3 = i.getData();
                    sortedList.add(temp3);
                }
            }
        }
    }
}

谁能检查我的代码并告诉我为什么我的temp 和temp2 从未被分配和使用？为什么这不起作用？

当我运行 main 时，我只得到原始链表而不是排序的链表。

我的逻辑是否正确？我正在尝试按升序对字符串进行排序。

Answer 1

自从 OP 更新了他的代码 sn-p 后，下面的代码就不能工作了，因为提到的 LinkedList 是自定义 List 而不是 java.util.LinkedList。

由于缺少“top”变量，无法运行您的代码。 顺便说一句，如果您需要将 i 和 j 的所有值都保存到单个排序的 LinkedList 中，请尝试以下操作：

    LinkedList sortedList = new LinkedList();

    sortedList.addAll( ALL_YOUR_VALUES FROM i )
    sortedList.addAll( ALL_YOUR_VALUES FROM j )

    Collections.sort(sortedList);
    // now your LinkedList should be sorted

Answer 2

说明

您正在将（可能）排序的值分配给新列表sortedList。此列表与您的原始列表 (this)无关。因此您的原始列表不会改变。

最后你需要将sortedList的结构复制到你当前的实例中，或者直接就地排序。

---

解决方案

因此，假设您的排序确实有效，请将类似这样的内容附加到 sort() 的末尾：

// Copy data of sorted list over to own instance
Node currentSortedElement = sortedList.top;
Node currentNonSortedElement = top;

while (currentSortedElement != null) {
    // Extract data and copy over
    currentNonSortedElement.setData(currentSortedElement.getData());

    // Prepare next iteration
    currentSortedElement = currentSortedElement.next;
    currentNonSortedElement = currentNonSortedElement.next;
}

因此请查看以下列表

index      | 0  1  2  3  4  5
-----------|-----------------
sorted     | 1  4  4  6  8  9
non-sorted | 4  1  6  9  8  4

算法只会将排序列表中的每一项的数据复制到非排序列表中相同索引的项目中，所以

index      | 0  1  2  3  4  5
-----------|-----------------
sorted     | 1  4  4  6  8  9
           | |  |  |  |  |  |
           | v  v  v  v  v  v
non-sorted | 1  4  4  6  8  9

---

注意事项

根据您的评论，您尝试实现Bubblesort。考虑查看相关问题Bubble Sort Manually a Linked List in Java，它显示了一个工作代码示例，并附有解释。