Question

我在Java中实现双链接列表时遇到了问题。特别是交换2个跟随节点（在我的情况下，一个节点包含一个政治候选人）假设以下DLL：
head - ＆gt; 1 - ＆gt; 2 - ＆gt; 3 - ＆gt; 4 - ＆gt; tail

public static void reverseTwoNode(Node<Candidate> N1, Node<Candidate> N2){
     N1.setNextNode(N2.getNextNode()); 
        N2.setPreviousNode(N1.getPreviousNode());

        if (N1.getNextNode() != null)
            N1.getNextNode().setPreviousNode(N1);

        if (N2.getPreviousNode() != null)
            N2.getPreviousNode().setNextNode(N2);


        N2.setNextNode(N1); 
        N1.setPreviousNode(N2);
}

作为输出，我从头到尾没有正确的DLL，但从头到尾都很好：

List of candidate 
head-->Jacques-->Joseph-->Francis-->Gilbert-->tail
Reverse
tail-->Gilbert-->Francis-->Joseph-->Jacques-->head

Reverse nodes : Francis , Joseph
List of candidate 
head-->Jacques-->Joseph-->Gilbert-->tail
Reverse
tail-->Gilbert-->Joseph-->Francis-->Jacques-->head

我已经写了这个方法reverseTwoNode的几个版本。我甚至试图在节点内交换数据而不是交换节点，我有同样的问题。你能帮助我很好，我用这个简单的功能花了这么多时间，我看不出什么打扰...... 提前谢谢你，

可能是显示方法的结果??

 /* Display DLL from head to tail
 * @see java.lang.Object#toString()
 * @return str
 */
public String toString(){
    String str = "List of candidate \n";
    str += "head-->";

    Node<Candidate> iterator = this.getHead();

    while (iterator != null) {
        str += iterator.getCandidate().getName();
        str += "-->";
        iterator = iterator.getNextNode();
        }
    return str + "tail";    
}


/**
 * Return string that display DLL from tail to head
 * @return str
 */
public String reverseToString(){
    String str = "Reverse\n";
    str += "tail-->";

    Node<Candidate> iterator = this.getTail();

    while (iterator != null) {
        str += iterator.getCandidate().getName();
        str += "-->" ;
        iterator = iterator.getPreviousNode();
        }

    return (str + "head");
}

解决方案： 我的方法addNode为false，这是在尾部添加节点的正确方法：

public void addNode(Node<Candidate> C){
    if(tail == null){
        this.head = C;
        this.tail = C;
    }
    else{
        this.tail.setNextNode(C);
        this.tail.getNextNode().setPreviousNode(this.tail);
        this.tail = this.tail.getNextNode();
        this.tail.setNextNode(null);
    }   
    this.size ++;
}

Answer 1

如果满足以下假设，则已发布代码有效：

N1和N2是相邻节点
N1在N2之前（N1→N2和N1 <--N2）。
没有N1和N2是头部或尾部元素（因为reverseTwoNode没有更新列表）

检查：http://ideone.com/3AKsBx

对于更通用的解决方案，您需要一些辅助变量来存储N1 / N2下一个和之前的引用。因为当你这样做：

N1.setNextNode(N2.getNextNode()); 
N2.setPreviousNode(N1.getPreviousNode());

然后在：

if (N1.getNextNode() != null)
    N1.getNextNode().setPreviousNode(N1);

N1.getNextNode()不是原始值。由于N2.getNextNode()，它是N1.setNextNode(N2.getNextNode())。

以下是一个示例：http://ideone.com/voaaTO

Answer 2

删除if语句，因为它们会阻止你移动头部和尾部节点（当一个节点实际指向null时可能会导致奇怪的问题）。我还假设您想要交换两个节点而不管它们的位置（不必是两个相邻节点）。

这应该只是交换节点并保持列表的其余部分不变。

public static void reverseTwoNode(Node<Candidate> N1, Node<Candidate> N2){
    Node<Candidate> prevNode1 = N1.getPreviousNode();
    Node<Candidate> nextNode1 = N1.getNextNode();
    Node<Candidate> prevNode2 = N2.getPreviousNode();
    Node<Candidate> nextNode2 = N2.getNextNode();

    if(N2.getPreviousNode().equals(N1)) { // test for adjacent nodes
        N2.setPreviousNode(prevNode1);
        N2.setNextNode(N1);
        N1.setNextNode(nextNode2);
        N1.setPrevNode(N2);
    } else {
        N1.setPreviousNode(prevNode2);
        N1.setNextNode(nextNode2);
        N2.setPreviousNode(prevNode1);
        N2.setNextNode(nextNode1);
    }
}

Java双链表，交换节点

2 个答案: