这个Java代码有什么问题

Question

这是我在我的Josephus类中添加的方法，我应该使用一个循环链表。我在我的课的几个领域得到空指针异常，但都是因为这种方法。任何人都可以通过查看这段代码看到任何显而易见的错误吗？

/** Inserts the specified element in the list at the 
    last position 
    @param dataItem the element to add 
    */ 
// Complexity O(1) 
@SuppressWarnings({ "unchecked" }) 
public void add(E dataItem) { 
    Node <E> node = new Node <E> (dataItem,null,null); 
    if (count == 0){ // list is empty 
     head = node.previous= node; 
    } 

    else { 
     head.previous.next = node; 
     node.previous = head.previous; 
     head.previous = node; 
    } 
    count++; 
    } 

全码：http://pastebin.com/k4Hmbqmw

Answer 1

是的，你需要附上else节在大括号：否则

public void add(E dataItem) { 
    Node <E> node = new Node <E> (dataItem, null, null); 
    if (count == 0) { // list is empty 
     head = node.previous= node ; 
    } else { 
     (head.previous).next = node; 
     node.previous = head.previous; 
     head.previous = node; 
    } 
    count++; 
} 

，最后三行，每次执行，而不是刚刚过去的一。

Answer 2

您不必周围的if/else块大括号