2016-11-15 37 views
-1

这里是我的Node对象:为什么在插入自定义链表的末尾时出现NullPointerException?

public class Node<Any> { 

protected Any data; 
protected Node<Any> link; 

public Node() { 
    this.data=null; 
    this.link=null; 
} 

public Node(Any data, Node<Any> link) { 
    this.data=data; 
    this.link=link; 
} 

public void setData(Any data) { 
    this.data=data; 
} 

public void setLink(Node<Any> link) { 
    this.link=link; 
} 

public Any getData() { 
    return this.data; 
} 

public Node<Any> getLink() { 
    return this.link; 
} 

public String toString() { 
    return "Data: "+this.data+" | Link: "+this.link; 
} 
} 

这是我的SinglyLinkedList对象:

public class SinglyLinkedList<Any> { 
private Node<Any> head; 
private Node<Any> tail; 
private int size; 

public SinglyLinkedList() { 
    this.head=null; 
    this.tail=null; 
    this.size=0; 
} 

//overloaded constructor for array to be added here 

public void insertAsHead(Any thing) { 
    Node<Any> tmp=new Node<Any>(); //Create new node 
    tmp.data=thing; //new_node->node.data=new_value 
    tmp.link=this.head; //new_node->node.link=head 
    this.head=tmp; //head=new_node 
    size++; 
} 

public void insertAsTail(Any thing) { 
    if(head==null) 
     insertAsHead(thing); 
    else { 
     Node<Any> tmp = new Node<Any>(); //Create new node 
     tmp.data=thing; //new_node->node.data=new_value 
     tmp.link=null; //new_node->node.link=null 
     this.tail.link=tmp; //tail->node.link=new_node 
     this.tail=tmp; //tail=new_node; 
    } 
    size++; 
} 

public void insertBefore(int i, Any thing) { 

} 

public void insertAfter(int i, Any thing) { 

} 

public void insertAt(int i, Any thing) { 

} 

public void deleteHead() { 
    size--; 
} 

public void deleteTail() { 
    size--; 
} 

public void deleteAt(int i) { 
    size--; 
} 

public Any retrieve(int i) { 
    return null; 
} 

public void set(int i, Any thing) { 

} 

public boolean isEmpty() { 
    if(this.size==0) 
     return true; 
    else 
     return false; 
} 

public int size() { 
    return this.size; 
} 

public String toString() { 
    String s="["+this.head.data; 
    Node<Any> next = this.head.link; 
    while(next!=null) { 
     s+=", "+next.data; 
     next=next.link; 
    } 
    return s+"]"; 
} 
} 

每当我使用的方法“insertAsTail(X)”当链表中包含的值,程序执行暂停,并说在行this.tail.link=tmp; //tail->node.link=new_node处有NullPointerException。我尝试去this.tail.setLink(tmp)路线,但同样的异常发生。

+0

我忘了提及它尚未完成,所以请原谅代码中几个部分的空白。 – ArcIX

+3

可能重复[什么是NullPointerException,以及如何解决它?](http://stackoverflow.com/questions/218384/what-is-a-nullpointerexception-and-how-do-i-fix-it ) – khelwood

+0

这意味着''this.tail''为空。当列表为空时,为什么它不是“空”? – f1sh

回答

0

你真的需要学习如何使用调试器。但是,您的问题是:

this.tail.link 

this.tail在这里为null。

0

你的代码非常不理解。 为了在最后插入,你只需要检查头是否存在。如果不创建新节点并指定为头部。

如果头部已经存在,则需要遍历至列表的末尾。然后你需要添加节点。

我看不到你遍历列表直到结束。

伪代码:

public void addAtEnd(int new_data) 
{ 
    Node new_node = new Node(new_data); 
    if (head == null){ 
     head = new Node(new_data); 
     return; 
    } 
    new_node.next = null; 
    Node last = head; 
    while (last.next != null) 
     last = last.next; 
    last.next = new_node; 
    return; 
} 
+0

对不起。我意识到这不是完全可以理解的,但我试图保持逻辑结构与我的数据结构教授教给我的结果接近。我开始考虑遍历整个列表方法。谢谢,先生/女士。 – ArcIX

0

看起来你(链表)尾从未被初始化。对于第一个元素作为头部,你需要表示与尾部相同的元素。

public void insertAsHead(Any thing) { 
    Node<Any> tmp=new Node<Any>(); //Create new node 
    tmp.data=thing; //new_node->node.data=new_value 
    tmp.link=this.head; //new_node->node.link=head 
    this.head=tmp; //head=new_node 
    if(this.size == 0){ 
    this.tail=this.head; 
    } 
    size++; 
} 

基本上当你添加第一个元素时,它的头部和尾部都是因为它是唯一的元素。