Java双向链表循环

Question

我的目标是通过我创建的列表向后循环，如果一个元素小于我选择的数字t 一个具有t 值的节点将被插入到更小的元素前面。此外，如果列表中的每个元素都大于t，则值为t 的节点将被放置在列表的开头。示例：

DoublyLinkedList<Integer> list = new DoublyLinkedList<>();

list.add(1);
list.add(2);
list.add(3);
list.add(4);
list.add(5);

list.addAtFirstSmaller(4);
System.out.println("Nodelist: " + list.toString());

我应该期待结果： [1, 2, 4, 3, 4, 5] 但相反，我得到： [1, 2, 3, 4, 4, 5]

这是我目前的代码：

public void addAtFirstSmaller(T t)
{
    ListNode<T> node = tail;
    ListNode<T> newNode = new ListNode<T>(t);
    while(node.previous != null)
    {
        int compared = node.previous.element.compareTo(t);
        node = node.previous;
        if(compared < 0)
        {
            ListNode<T> temp = node.next;
            node.next = newNode;
            newNode.next = temp;
            newNode.next.previous = newNode;
            node.previous = null;
            size++;
        }
        else if(compared > 0 && node.previous == null)
        {
            addFirst(t);
            size++;
        }
    }
}

对我来说，似乎一切都被推到了右边。有什么想法吗？

Answer 1

正如您已经验证的那样：示例输出是正确的，因为其想法是保持列表排序。这种误解是可以理解的，因为 “在前面” 有点模棱两可。

但是，您的代码仍然存在一些问题：

• 虽然分配node.previous = null 将使循环在您插入新节点后结束，但这会破坏真正应该保持不变的previous 链接。只有头节点应该有previous 等于null，而一般node 可以有一个前任，它必须保持它的前任。

• newNode.previous 从不接收值。应该设置为node。

• 当要插入的值大于尾节点中的值时，该节点将被插入错误的位置——代码中永远不会出现新节点成为新尾节点的情况，但这是应该预见的可能性。

• 我会假设addFirst 已经增加了size，因此在调用此函数后再次增加它是错误的。这确实应该是addFirst的责任。

假设你已经实现了addLast，修正后的代码可能是这样的：

public void addAtFirstSmaller(T t) {
    ListNode<T> node = tail;
    ListNode<T> newNode = new ListNode<T>(t);
    while (node != null && node.element.compareTo(t) >= 0) {
        node = node.previous;
    }
    if (node == null) {
        addFirst(t);
    } else if (node == tail) {
        addLast(t);
    } else {
        ListNode<T> temp = node.next;
        node.next = newNode;
        newNode.next = temp;
        newNode.previous = node;
        temp.previous = newNode;
        size++;
    }
}

如果您还没有，addLast 可能如下所示：

public void addLast(T t) {
    ListNode<T> newNode = new ListNode<T>(t);
    tail.next = t;
    t.previous = tail;
    tail = t;
    size++;
}

Answer 2

我刚收到导师的答复，我的输出似乎是正确的（即 [1, 2, 3, 4, 4, 5]）。我会将这个线程标记为关闭，因为代码正在运行。