Question

我一直在查看关于黑客排名的Linkedlist问题，我目前正在解决这个问题，它会要求你在一个有序的双向链接列表中插入一个节点。

这是我用Java编写的逻辑

Node SortedInsert(Node head,int data) {

    Node newNode = new Node();
    Node temp = head;
    newNode.data = data;
    newNode.next = null;
    newNode.prev = null;

    if(head == null){
        head = newNode;
    }else{
        while(data > temp.data && temp.next != null){
            temp = temp.next;
        }
        if(temp == head){
            newNode.next = temp;
            temp.prev = newNode;
            head = newNode;
        }else if(data > temp.data){
            newNode.prev = temp;
            temp.next = newNode;
        }else{
            newNode.prev = temp.prev;
            newNode.next = temp;
            temp.prev.next = newNode;
            temp.prev = newNode;
        }

    }

  return head;
}

这是我得到的错误。

Your Output (stdout)
Wrong Answer!
Some possible errors:
1. You returned a NULL value from the function. 
2. There is a problem with your logic
Wrong Answer!
Some possible errors:
1. You returned a NULL value from the function. 
2. There is a problem with your logic

我不知道我做错了什么。我真的想知道我哪里出错了。我知道在网上找到答案很容易，但我想如果有人能纠正我的错误，我会学到最好的。

Answer 1

问题是您始终在第一个元素之前插入第二个元素。请考虑以下插图：

首先让链表清空。现在，您按照算法插入1。 head == null案例已触发，head现在指向newNode。

x<-1->x
   |
  HEAD

现在，您尝试在列表中插入2。您会看到while循环结束，temp现在指向head，触发后面的if条件（if(temp == head)）。

x<-1->x
   |
  HEAD, temp

这会在 2之前插入temp（错误！）。

x<-2<=>1->x | HEAD

交换条件的顺序应解决问题：

if(data > temp.data) { // First, check if you need to insert at the end. newNode.prev = temp; temp.next = newNode; } else if(temp == head) { // Then, check if you need to insert before head. newNode.next = temp; temp.prev = newNode; head = newNode; } else { // Otherwise, insert somewhere in the middle. newNode.prev = temp.prev; newNode.next = temp; temp.prev.next = newNode; temp.prev = newNode; }

在预排序的链接列表中插入节点

1 个答案: