插入排序算法java双链表

Question

我有一个带有sentinel节点的双链表，我需要使用复杂度为O（n^2）的Insertion sort对其进行排序。我已经写了这个代码，但它没有按预期工作。

有没有关于插入排序的一般帮助，特别是关于代码的帮助？

    public void insertionSort()
    {
        int key,i;
        int size = getSize();
        this.restart();
        for(i = 2; i < size; i++)
        {
            this.next();  // Go to next node
            key = this.getVar(); // get the integer a node holds
            while(this.hasNext() == 1 && position.prev.var < key && position.var != -1)
            {
                position.prev.setNext(position.next);
                position.next.setPrev(position.prev);
                position.setNext(position.prev);
                position.setPrev(position.prev.prev);
                position.prev.setNext(position);
                position.next.setPrev(position);    
                this.goBack(); // go to the previous node
            }                       
        }
    }

大小是我的列表中有多少元素。我的sentinel有var=-1，所以我想在我领先的时候停下来，这就是我使用这个的原因。

position.var != -1

并且CCD_ 1是真的，只要我们处于一个位置！=哨兵节点。

在具体的例子中，我的列表中有35个整数：

5 9 1 1 1 1 11 11 11 11 1 11 11 1 1 11 1 1 1

我想这样分类：

9 5 2 1 1 1 1 11 11 11 11 1 1 1

更新：我在插入排序中使用的代码如下：

public int getSize()
        {
            int size = 0;
            this.restart();
            while(this.hasNext() == 1)
            {
                size++;
                this.next();
            }
            return size;
        }
public void restart()
        {
            position = header;
            previous = Sentinel;
        }
public void next()
        {
            previous = position;
            position = position.next;
        }
public int getVar()
        {
            return position.var;
        }
public void goBack()
        {
            position = previous;
            previous = previous.prev;
        }
    public int hasNext()
        {
            if(position.var != -1)
                return 1;
            else 
                return 0;
        }
    public void setNext(Node next)
        {
            this.next = next;
        }

public void setPrevi（节点prev）{this.prev=prev；}

此外，我使用了一个列表迭代器。

Answer 1

这是我对内部循环分析的注释，你是对的，肯定有问题。

position.unlink（）：越界，邻居变成直接邻居

position.prev.next = position.next;  // TODO 1: position.next.prev = position.prev
position.next.prev = position.prev;  // TODO 2: position.prev.next = position.next
                  ^  Breaks TODO 1, but we can: position.prev.next.prev = position.prev

因此，我们仍然需要TODO 1，然后是2，按顺序：

     -- TODO 1: position.prev.next.prev = position.prev
     -- TODO 2: position.prev.next      = position.next

position.insertBefore（position.prev）：在行的后面再排一位

position.next = position.prev;          // TODO 3: position.prev.prev = position
position.prev = a = position.prev.prev; // TODO 4: a.next = position
//           ^                          //   same: position.prev.next = position   
//           |                          // *Error 1!*
//           + breaks TODO's 1 and 2, *Error 2!*
//           + breaks TODO 3, but we can instead: position.next.prev = position

关于错误1，TODO 3和TODO 4都涉及position.prev，将其next和prev都设置为position；这有效地包围了position.prev用CCD_ 7。无论它前进还是后退，它的直接邻居都将是position。不过，走了一步之后，一切都是一样的。有趣的结构——就像你房子的前门和后门都通向同一个地方。

错误2导致无法更新position.prev，这是TODO的1、2和3所需的。通过使用通过this.hasNext() == 11的next字段访问a.next的备用表达式，我们仍然可以满足TODO3，但不能再满足TODO1和2。

position.insertBefore（position.prev），continued:

position.prev.next = position;        // DONE: 4 
position.next.prev = position;        // DONE: 3 

这两行完成了TODO 3和4，所以剩下的就是：

     -- TODO 1: position.prev.next.prev = position.prev
     -- TODO 2: position.prev.next      = position.next
     -- TODO 5: fix Error 1
     -- TODO 6: fix Error 2

修复错误1涉及确保在TODO 4之前完成TODO 1和2，修复错误2涉及确保在分配a之前完成TOD奥3。

你最终完成了8项任务；事后看来，对于双链表和涉及4个节点的移动来说，这并不奇怪。

Answer 2

这应该可以解决问题：

        int j;
        // ...
        for(i = 1; i < size; i++)
        {
            this.restart();          // start at ith node
            for(j = 0; j < i; j++)
                this.next();
            key = this.getVar();     // same code as before

或者对于每个外循环使用每次前进一个节点的另一个变量。

另外，this.hasNext（）不应该重命名为this.hasPrevi（）吗？

代码的主要部分似乎是正确的，示例图：

                // goal: swap B and C
                // initial state
          p        
    -> -> -> ->
    A  B  C  D
   <- <- <- <-
                // remove C from list
                position.prev.setNext(position.next);
                position.next.setPrev(position.prev);
          p
       ----->
    ->    -> ->
    A  B  C  D
   <- <- <-
         <----
                // update C next and previous                    
                position.setNext(position.prev);
                position.setPrev(position.prev.prev);
       p
    ----->
       -> -> ->
    A  C  B  D
   <- <-    <-
      <----
                // insert C into list
                position.prev.setNext(position);
                position.next.setPrev(position);
       p
    -> -> -> ->
    A  C  B  D
   <- <- <- <-