在 java 中删除双向链表中的两个节点时出错

Error while removing two nodes in doubly linked list in java

我在双重 LinkedList 中遇到问题,我没有使用尾巴而是使用当前指针。当我使用尾巴时,我没有发现问题,但是当我使用当前指针时,我得到一个错误,我无法解决它。它发生在我删除节点时。我的程序正在运行,但我无法删除下一个节点。 这是错误消息:

这个class DNode

public class DNode<T> { 
 T data;
 DNode<T> next;
DNode<T> prev;
public DNode(T e){
 data = e;
    next = prev = null;
}

这个classDoubleLinkedList

public class DoubleLinkedList<T> {

DNode<T> head;
DNode<T> current;
int size = 0;
public DoubleLinkedList() {
    head = current = null;
}
public void Insert(T e) {
    DNode<T> tmp = new DNode(e);
    if (size == 0) {
        head = current = tmp;
    } else {
        tmp.next = current.next;
        tmp.prev = current;
        current.next = tmp;
        current = tmp;

    }
    size++;
}

public void remove() {
    if (head == current) {
        if (current.next == null) {
            head = current = null;
        } else {
            current.next.prev = null;
            head = current.next;
            current.next = null;
            current = head;
        }
    } else {
        DNode<T> tmp = current.next;

        current.prev.next = tmp;
        if (tmp != null) {
            tmp.prev = current;
        }
        current.next = current.prev = null;
        current = tmp;
    }

    size--;

}

主要课程

public static void main(String[] args) {
DoubleLinkedList<String> d = new DoubleLinkedList();


  d.Insert("jon");
    d.Insert("jack");
    d.Insert("mohammed");
    d.remove();
    d.remove();// here my problem
 }

注释行是我出错的地方。

在 remove 函数的 else 部分,您将 current 分配给 tmp。当您说 current = tmp; tmp 可能为空时。因此,下次调用 remove 时,如果它进入 else 子句(在您的情况下),current 将为空,并且 current.next 给出空指针异常。

编辑: 这将是一个可能的建议:

/*Deletes the last node in the list*/
public void remove() {
  if(head == null || size == 0) return; // Empty list
  if(head == current || size ==1){    // List with only one node
    head = null;
    current = null;
    size--;
    return;
  }
  DNode<T> previous = current.prev;
  previous.next = current.next;
  current.prev = null;
  current = previous;
    size--;
}