2013-03-06 48 views
2

現在我的鏈接列表處於隊列形式。我的LList類包含兩個稱爲頭部和尾部的字段,它們是列表的頭部和尾部。頭部和尾部是LNode對象,LNode是包含int值的列表元素,它是之前的LNode和下一個LNode。需要關於鏈接列表刪除方法的幫助

這裏是我的葉子節點類:

class LNode{ 
    private int val; 
    private LNode next;//not recursive 
    private LNode prev; 
    public LNode(int v, LNode n, LNode p){ 
     next = n; 
     prev = p; 
     val = v; 
    } 
    public int getVal(){ 
     return val; 
    } 
    public LNode getNext(){ 
     return next; 
    } 
    public LNode getPrev(){ 
     return prev; 
    } 
    public void setVal(int v){ 
     val = v; 
    } 
    public void setNext(LNode n){ 
     next = n; 
    } 
    public void setPrev(LNode p){ 
     prev = p; 
    } 
} 

我試圖做一個刪除方法在我LLIST類,因此,它需要一個值,並刪除了該值的葉子節點。我的問題是,我不知道如何處理我試圖刪除的LNode是頭部還是尾部的情況。

public void delete(int v){ 

    if(head.getVal()==v){//delete head 
     head = head.getNext(); 
     head.setPrev(null); 
    } 
    else if(tail.getVal()==v){//delete tail 
     System.out.println("boiboi"); 
     tail = tail.getPrev(); 
     tail.setNext(null); 
    } 
    else{//delete other element 
     LNode tmp = head; 
     while(tmp.getVal()!=v){ 
      tmp = tmp.getNext(); 
     } 
     tmp.getPrev().setNext(tmp.getNext()); 
     tmp.getNext().setPrev(tmp.getPrev()); 
    } 
} 

我試過的是設置新的頭以前的LNode爲空,但Java不允許。所以我該怎麼做?

謝謝。

+0

它可以是一個循環列表嗎?在這種情況下,你可以'head.setPrev(tail)'。 – 2013-03-06 04:46:56

+2

「Java不允許」是什麼意思? – 2013-03-06 04:47:09

+0

我不這麼認爲......它必須是隊列 – 2013-03-06 04:47:40

回答

3

你的代碼看起來還好我,除了你要移除值是唯一值的情況下 - 在這種情況下,你要同時與頭部尾部爲空落得。我懷疑所有你需要做的是改變頭案例:

if (head.getVal() == v) { 
    head = head.getNext(); 
    if (head != null) { 
     head.setPrev(null); 
    } else { 
     // If head.getNext() returns null, then tail must have been equal to head. 
     tail = null; 
    } 
} 

您也應該檢查空列表情況第一:

if (head == null) { 
    return; 
} 

並在您的一般情況,處理情況,其中值未找到:

while (tmp != null && tmp.getVal() != v) { 
    tmp = tmp.getNext(); 
} 
if (tmp == null) { 
    return; 
} 
+0

是的,你說得對,我的代碼沒有問題,我剛剛在我的Testing類中發生錯誤,這就是造成NullPointerException的原因。 – 2013-03-06 05:10:35

+2

@cookcook:你用你的代碼以什麼方式「沒有問題」?我指出的一切都是你的代碼中的一個錯誤 - 而不是你的測試。在空列表中調用'delete'不應該拋出'NullPointerException',也不應該用不在列表中的值調用'delete'。 – 2013-03-06 05:11:34

+0

我確實意識到我的問題只有列表中的一個元素。我只是說我不知道​​這是原因,我的原始代碼正常工作,正常情況下刪除頭部和尾部元素。感謝您的幫助,並且我已經更改了我的代碼以適用於您提到的所有情況。 – 2013-03-06 05:18:53