2011-11-07 10 views
1

私はプログラムを終えようとしていて、私はその場で捕まっています。私のdeleteCurrentNodeメソッドは部分的にしか動作しません。何らかの理由で、リンクされたリストをトラバースしてcurrentNodeを見つけようとすると、決してそれを見つけられません。誰かが私にそれを働かせるためのヒントを教えてくれますか?リストが空の場合単一リンクリストの現在のノードを削除するにはどうすればよいですか?

  1. 方法自体は4つの条件をチェックします。

  2. currentNodeがnullの場合。
  3. currentNodeがリストの最初のノードである場合。
  4. currentNodeがリストのどこかにある場合。

その他の条件は(私の知る限り)機能します。 4は問題がどこにあるかです。

public class LinkedList 
{ 
private Node currentNode; 
private Node firstNode; 
private int nodeCount; 

public static void main(String[] args) 
{ 
LinkedList test; 
String dataTest; 
test = new LinkedList(); 
dataTest = "abcdefghijklmnopqrstuvwxyz"; 
for(int i=0; i< dataTest.length(); i++) { test.insert(new String(new char[] { dataTest.charAt(i) })); } 
System.out.println("[1] "+ test); 

    for(int i=0; i< dataTest.length(); i++) { test.deleteCurrentNode(); } 
    System.out.println("[2] "+test); 

    for(int i=0; i< dataTest.length(); i++) 
    { 
    test.insertBeforeCurrentNode(new String(new char[] { dataTest.charAt(i) })); 
    if(i%2 == 0) { test.first(); } else { test.last(); } 
    } 

    System.out.println("[3] "+test); 

    for(int i=0; i< dataTest.length(); i++) { test.last(); test.deleteCurrentNode(); } 
     System.out.println("[4] "+test); 


     for(int i=0; i< dataTest.length(); i++) { test.insert(new String(new char[] { dataTest.charAt(i) })); test.last(); } 
     System.out.println("[5] "+test); 

    while(!test.isEmpty()) { test.deleteFirstNode(); } 
    System.out.println("[6] "+test); 

    for(int i=0; i< dataTest.length(); i++) { test.insert(new String(new char[] { dataTest.charAt(i) })); test.last(); } 
    System.out.println("[7] "+test); 

    while(!test.isEmpty()) { test.deleteFirstNode(false); } 
    System.out.println("[8] "+test); 

    for(int i=0; i< dataTest.length(); i++) { test.insertBeforeCurrentNode(new String(new char[] { dataTest.charAt(i) })); test.first(); } 
    System.out.println("[9] "+test); 
} 

public LinkedList() 
{ 
    setListPtr(null); 
    setCurrent(null); 
    nodeCount = 0; 
} 

public boolean atEnd() 
{ 
    //checkCurrent(); 
    return getCurrent().getNext() == null;  
} 

public boolean isEmpty() 
{ 
    return getListPtr() == null; 
} 

public void first() 
{ 
    setCurrent(getListPtr()); 
} 

public void next() 
{ 
    checkCurrent(); 
    if (atEnd()) {throw new InvalidPositionInListException("You are at the end of the list. There is no next node. next().");} 
    setCurrent(this.currentNode.getNext()); 
} 

public void last() 
{ 
    if (isEmpty()) {throw new ListEmptyException("The list is currently empty! last()");} 

    while (!atEnd()) 
    { 
     setCurrent(getCurrent().getNext()); 
    } 

} 

public Object getData() 
{ 
    return getCurrent().getData(); 
} 

public void insertBeforeCurrentNode(Object bcNode) //beforeCurrentNode 
{ 
    Node current; 
    Node hold; 
    boolean done; 
    hold = allocateNode(); 
    hold.setData(bcNode); 
    current = getListPtr(); 
    done = false; 
    if (isEmpty()) 
    { 
     setListPtr(hold); 
     setCurrent(hold);  
    } 

    else if (getCurrent() == getListPtr()) 
    { 
    // System.out.println("hi" + hold); 
     hold.setNext(getCurrent()); 
     setListPtr(hold); 
    } 

    else if (!isEmpty() && getCurrent() != getListPtr()) 
    { 
     while (!done && current.getNext() != null) 
     { 
      //System.out.println("in else if " + hold); 
      if (current.getNext() == getCurrent()) 
      { 
       //previous.setNext(hold); 
       //System.out.println("hi"+ "yo" + " " + getListPtr()); 
       hold.setNext(current.getNext()); 
       current.setNext(hold); 
       done = true; 
      } 

      //previous = current; 
      current = current.getNext(); 
     } 

    } 
    //System.out.println("current " + getCurrent()); 
    //System.out.println("pointer " + getListPtr()); 

} 

public void insertAfterCurrentNode(Object acNode) //afterCurrentNode 
{ 
    Node hold; 
    hold = allocateNode(); 
    hold.setData(acNode); 
    if (isEmpty()) 
    { 
     setListPtr(hold); 
     setCurrent(hold); 
     //System.out.println(hold + " hi"); 
    } 

    else 
    { 
     //System.out.println(hold + " hia"); 
     hold.setNext(getCurrent().getNext()); 
     getCurrent().setNext(hold); 
    } 
} 

public void insert(Object iNode) 
{ 
    insertAfterCurrentNode(iNode); 
} 

public Object deleteCurrentNode() 
{ 
    //System.out.println("in delete current"); 
    Object nData; 
    Node previous; 

    if (isEmpty()) {throw new ListEmptyException("The list is currently empty! last()");} //if list is empty throw exception 

    checkCurrent(); //check if currentNode is null, method throws exception if it is. 

    nData = getCurrent().getData(); 

    if (getCurrent() == getListPtr()) 
    { 
     setListPtr(getCurrent().getNext()); 
     setCurrent(getCurrent().getNext()); 
     nodeCount = nodeCount -1; 
    } 

    else 
    { 
     previous = getListPtr(); 
     while (previous.getNext() != getCurrent()) 
     { 
      previous = previous.getNext(); 
      //System.out.println("test"+ previous); 
     } 


     if (getCurrent().getNext() != null) 
     { 
      previous.setNext(null); 
     } 

     previous.setNext(getCurrent().getNext());  } 

    return nData; 
} 

public Object deleteFirstNode(boolean toDelete) 
{ 
    if (toDelete) 
    { 
     setListPtr(null); 
    } 
    return getListPtr(); 
} 

public Object deleteFirstNode() 
{ 
    Object deleteFirst; 
    deleteFirst = deleteFirstNode(true); 
    return deleteFirst; 
} 

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

public String toString() 
{ 
    String nodeString; 
    Node sNode; 
    sNode = getListPtr(); 
    //System.out.println(nodeCount); 
    nodeString = ("List contains " + nodeCount + " nodes"); 
    while (sNode != null) 
    { 
     nodeString = nodeString + " " +sNode.getData(); 
     sNode = sNode.getNext(); 
    } 
    return nodeString; 
} 

private Node allocateNode() 
{ 
    Node newNode; 
    newNode = new Node(); 
    nodeCount = nodeCount + 1; 
    return newNode; 
} 

private void deAllocateNode(Node dNode) 
{ 
    dNode.setData(null); 
} 

private Node getListPtr() 
{ 
    return this.firstNode; 
} 

private void setListPtr(Node pNode) 
{ 
    this.firstNode = pNode; 
} 

private Node getCurrent() 
{ 
    return this.currentNode; 
} 

private void setCurrent(Node cNode) 
{ 
    this.currentNode = cNode; 
} 

private void checkCurrent() 
{ 
    if (getCurrent() == null) {throw new InvalidPositionInListException("Current node is null and is set to an invalid position within the list! checkCurrent()");} 
} 

/**NODE CLASS ----------------------------------------------*/ 

    private class Node 
    { 
     private Node next; //serves as a reference to the next node 
     private Object data; 

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


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

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

     public Node getNext() 
     { 
      return this.next; 
     } 

     public void setNext(Node nextNode) 
     { 
      this.next = nextNode; 
     } 

     public String toString() 
     { 
      String nodeString; 
      Node sNode; 
      sNode = getListPtr(); 
      //System.out.println(nodeCount); 
      nodeString = ("List contains " + nodeCount + " nodes"); 
      while (sNode != null) 
      { 
       nodeString = nodeString + " " +sNode.getData(); 
       sNode = sNode.getNext(); 
      } 
      return nodeString; 
     } 
    } 


} 

[4] [2](一覧が0のノードが含まれています。)

ノートと同じように読んでください。私は既に持っている任意の複数の変数/メソッドの/ etcを使用することはできません。ヘッドノードは使用できません。

+0

あなたは実際にここで何もしていない、私はあなたの問題が何かを理解していない。 – leo

+0

あなたはノードを削除していません。これは "前"にあるはずです – leo

+0

私のプログラムの間違ったバージョンをコピーしました申し訳ありません。更新されました。メソッド自体は、削除されている現在のノードのデータ部分を返すと仮定されていますが、今は、前の変数がcurrentNodeにあるようにリストをトラバースしようとしています。 – solllodolllo

答えて

1
public class LinkedList 
{ 
private Node currentNode; 
private Node firstNode; 
private int nodeCount; 


public static void main(String[] args) 
{ 
    String  data; 
    Object  hold; 
    LinkedList list; 

    data = "abcdefghijklmnopqrstuvwxyz"; 
    hold = null; 
    list = new LinkedList(); 

    for(int i=0; i< data.length(); i++) { list.insert(new String(new char[] { data.charAt(i) })); } 
    System.out.println("[1] "+list); 

    for(int i=0; i< data.length(); i++) { list.deleteCurrentNode(); } 
    System.out.println("[2] "+list); 

    for(int i=0; i< data.length(); i++) 
    { 
    list.insertBeforeCurrentNode(new String(new char[] { data.charAt(i) })); 
    if(i%2 == 0) { list.first(); } else { list.last(); } 
    } 

    System.out.println("[3] "+list); 

    for(int i=0; i< data.length(); i++) { list.last(); list.deleteCurrentNode(); } 
    System.out.println("[4] "+list); 


    for(int i=0; i< data.length(); i++) { list.insert(new String(new char[] { data.charAt(i) })); list.last(); } 
    System.out.println("[5] "+list); 

    while(!list.isEmpty()) { list.deleteFirstNode(); } 
    System.out.println("[6] "+list); 

    for(int i=0; i< data.length(); i++) { list.insert(new String(new char[] { data.charAt(i) })); list.last(); } 
    System.out.println("[7] "+list); 

    while(!list.isEmpty()) { list.deleteFirstNode(false); } 
    System.out.println("[8] "+list); 

    for(int i=0; i< data.length(); i++) { list.insertBeforeCurrentNode(new String(new char[] { data.charAt(i) })); list.first(); } 
    System.out.println("[9] "+list); 

    list.first(); 
    list.next(); 
    list.deleteFirstNode(true); 
    list.deleteCurrentNode(); 
    for(int i=0; i< 5; i++) { hold = list.getData(); list.next(); } 
    for(int i=0; i< 10; i++) { list.next(); } 
    list.insertAfterCurrentNode(hold); 
    list.first(); 
    list.next(); 
    hold = list.getData(); 
    list.deleteCurrentNode(); 
    for(int i=0; i<9; i++) {list.deleteCurrentNode(); list.last(); } 
    list.insert(hold); 
    list.first(); 
    list.next(); 
    list.next(); 
    list.next(); 
    list.deleteFirstNode(false); 
    hold = list.getData(); 
    list.deleteCurrentNode(); 
    list.last(); 
    list.insertAfterCurrentNode(hold); 
    list.deleteFirstNode(); 
    list.deleteFirstNode(); 
    hold = list.getData(); 
    list.deleteFirstNode(); 
    list.last(); 
    list.insertBeforeCurrentNode(hold); 
    list.first(); 
    for(int i=0; i<6; i++) { list.next(); } 
    hold = list.getData(); 
    list.deleteCurrentNode(); 
    list.last(); 
    list.insertBeforeCurrentNode(hold); 
    list.first(); 
    list.deleteCurrentNode(); 
    list.deleteCurrentNode(); 
    hold = list.getData(); 
    list.deleteCurrentNode(); 
    for (int i=0; i< 7; i++) { list.next(); } 
    list.insertBeforeCurrentNode(hold); 
    for (int i=0; i< 4; i++) { list.first(); list.deleteCurrentNode(); } 
    System.out.println("\n\n"+list); 


} 

public LinkedList() 
{ 
    setListPtr(null); 
    setCurrent(null); 
    nodeCount = 0; 
} 

public boolean atEnd() 
{ 
    if (getCurrent() != null) 
     { 
     return getCurrent().getNext() == null; 
     } 
     return true; 

} 

public boolean isEmpty() 
{ 
    return getListPtr() == null; 
} 

public void first() 
{ 
    setCurrent(getListPtr()); 
} 

public void next() 
{ 
    checkCurrent(); 
    if (atEnd()) {throw new InvalidPositionInListException("You are at the end of the list. There is no next node. next().");} 
    setCurrent(this.currentNode.getNext()); 
} 

public void last() 
{ 
    if (isEmpty()) {throw new ListEmptyException("The list is currently empty! last()");} 

    while (!atEnd()) 
    { 
     setCurrent(getCurrent().getNext()); 
    } 

} 

public Object getData() 
{ 
    return getCurrent().getData(); 
} 

public void insertBeforeCurrentNode(Object bcNode) //beforeCurrentNode 
{ 
    Node current; 
    Node hold; 
    boolean done; 
    hold = allocateNode(); 
    hold.setData(bcNode); 
    current = getListPtr(); 
    done = false; 
    if (isEmpty()) 
    { 
     setListPtr(hold); 
     setCurrent(hold);  
    } 

    else if (getCurrent() == getListPtr()) 
    { 
    // System.out.println("hi" + hold); 
     hold.setNext(getCurrent()); 
     setListPtr(hold); 
    } 

    else if (!isEmpty() && getCurrent() != getListPtr()) 
    { 
     while (!done && current.getNext() != null) 
     { 
      //System.out.println("in else if " + hold); 
      if (current.getNext() == getCurrent()) 
      { 
       //previous.setNext(hold); 
       //System.out.println("hi"+ "yo" + " " + getListPtr()); 
       hold.setNext(current.getNext()); 
       current.setNext(hold); 
       done = true; 
      } 

      //previous = current; 
      current = current.getNext(); 
     } 

    } 
    //System.out.println("current " + getCurrent()); 
    //System.out.println("pointer " + getListPtr()); 

} 

public void insertAfterCurrentNode(Object acNode) //afterCurrentNode 
{ 
    Node hold; 
    hold = allocateNode(); 
    hold.setData(acNode); 
    if (isEmpty()) 
    { 
     setListPtr(hold); 
     setCurrent(hold); 
     //System.out.println(hold + " hi"); 
    } 

    else 
    { 
     //System.out.println(hold + " hia"); 
     hold.setNext(getCurrent().getNext()); 
     getCurrent().setNext(hold); 
    } 
} 

public void insert(Object iNode) 
{ 
    insertAfterCurrentNode(iNode); 
} 

public Object deleteCurrentNode() 
{ 
    //System.out.println("in delete current"); 
    Object nData; 
    Node previous; 

    if (isEmpty()) {throw new ListEmptyException("The list is currently empty! last()");} //if list is empty throw exception 

    checkCurrent(); //check if currentNode is null, method throws exception if it is. 

    nData = getCurrent().getData(); 

    if (getCurrent() == getListPtr()) 
    { 
     setListPtr(getCurrent().getNext()); 
     setCurrent(getCurrent().getNext()); 
     nodeCount = nodeCount -1; 
    } 

    else 
    { 
     previous = getListPtr(); 
     //System.out.println(getCurrent()); 
     //System.out.println(previous + "ptrb "); 
     while (previous.getNext() != getCurrent()) 
     { 
      previous = previous.getNext(); 
      //System.out.println("test"+ previous); 
     } 

     //System.out.println(previous.getNext() == getCurrent()); 

     if (previous.getNext() == getCurrent()) 
     { 
      //System.out.println("say hi"); 
      previous.setNext(getCurrent().getNext()); 
      deAllocateNode(getCurrent()); 
      setCurrent(previous); 
      nodeCount = nodeCount - 1; 
     } 

     previous.setNext(getCurrent().getNext()); 

    } 

    return nData; 
} 

public Object deleteFirstNode(boolean toDelete) 
{ 
    if (toDelete) 
    { 
     setCurrent(getListPtr().getNext()); 
    } 
    deAllocateNode(getListPtr()); 
    setListPtr(getListPtr().getNext()); 

    nodeCount = nodeCount - 1; 
    return getListPtr(); 
} 

public Object deleteFirstNode() 
{ 
    Object deleteFirst; 
    deleteFirst = deleteFirstNode(true); 
    //System.out.println("called"); 
    return deleteFirst; 
} 

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

public String toString() 
{ 
    String nodeString; 
    Node sNode; 
    sNode = getListPtr(); 
    //System.out.println(nodeCount); 
    nodeString = ("List contains " + nodeCount + " nodes"); 
    while (sNode != null) 
    { 
     nodeString = nodeString + " " +sNode.getData(); 
     sNode = sNode.getNext(); 
    } 
    return nodeString; 
} 

private Node allocateNode() 
{ 
    Node newNode; 
    newNode = new Node(); 
    nodeCount = nodeCount + 1; 
    return newNode; 
} 

private void deAllocateNode(Node dNode) 
{ 
    dNode.setData(null); 
} 

private Node getListPtr() 
{ 
    return this.firstNode; 
} 

private void setListPtr(Node pNode) 
{ 
    this.firstNode = pNode; 
} 

private Node getCurrent() 
{ 
    return this.currentNode; 
} 

private void setCurrent(Node cNode) 
{ 
    this.currentNode = cNode; 
} 

private void checkCurrent() 
{ 
    if (getCurrent() == null) {throw new InvalidPositionInListException("Current node is null and is set to an invalid position within the list! checkCurrent()");} 
} 

/**NODE CLASS ----------------------------------------------*/ 

    private class Node 
    { 
     private Node next; //serves as a reference to the next node 
     private Object data; 

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


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

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

     public Node getNext() 
     { 
      return this.next; 
     } 

     public void setNext(Node nextNode) 
     { 
      this.next = nextNode; 
     } 

     public String toString() 
     { 
      String nodeString; 
      Node sNode; 
      sNode = getListPtr(); 
      //System.out.println(nodeCount); 
      nodeString = ("List contains " + nodeCount + " nodes"); 
      while (sNode != null) 
      { 
       nodeString = nodeString + " " +sNode.getData(); 
       sNode = sNode.getNext(); 
      } 
      return nodeString; 
     } 
    } 


    } 

あなたの貢献に感謝します。彼らは私の問題を解決するのを手伝ってくれました。(あまりにも曖昧ですが)。

他の誰かが同じような問題を抱えても、私は自分の質問に答えています。

もう一度、ありがとう。

1

あなたのようなリストの1つのノードを表す、Nodeクラスを持っている場合:

:(さんはマイリストを言わせて)あなたのリストの実装であなたよりも

public class Node{ 
public Object value; 
public Node next; 

public Node(){ 

} 

public Node(Object p){ 
    value= p; 
} 

public String toString(){ 
    return "" + this.value.toString(); 
} 
} 

あなたは(だけ除去する方法を)持っています

public class MyList{ 
    private Node head; 

public Object removeFirst(){ 
    if(head == null){ 
     return null; 
    } 

    Object o= head.value; 
    head= head.next; 
    return o; 
} 

    // remove by index 
public Object remove(int i){ 
    int n= this.size(); 
    if(i<0 || i>=n){ 
     return null; 
    } 

    if(i==0){ 
     return this.removeFirst(); 
    } 

    int k=0; 
    Node t= head; 

    while(k < i-1){ 
     t= t.next; 
     k= k+1; 
    } 

    Object o= t.next.value; 
    t.next= t.next.next; 

    return o; 
} 

    //remove by object 
    public boolean remove(Object o){ 
    int k= this.indexOf(o); 
    if(k<0){ 
     return false; 
    } 
    this.remove(k); 

    return true; 
} 

    //not necessary, but you may study the logic 
    public Object removeLast(){ 
    Object o= null; 
    if(head!=null){ 
     if(head.next==null){ 
      o= head.value; 
      head= null; 
     }else{ 
      Node t= head.next; 
      while(t.next.next != null){ 
       t= t.next; 
      } 
      o= t.next.value; 
      t.next= null; 
     } 
    } 
    return o; 
} 
} 

編集:配列に1つの要素しか含まれていない場合、いくつか問題があります。あなたの最後の()メソッドでは例えば

- >

while (!atEnd()) { 
     setCurrent(getCurrent().getNext()); 
    } 
+0

私の実装では、currentNodeとfirstNodeの2つの異なるノードが必要です。私は問題を抱えているメソッドのために、私はリスト内のcurrentNodeを見つけて削除する必要がありますが、私のメソッドが正しく動作するようにすることはできません。 – solllodolllo

+0

deleteCurrent()は== getIndexOfCurrnet() - > deleteByIndex(indexOfcurrent)と等しくなければならないことを暗示するこのコードを投稿しました – hovanessyan

+0

私のプログラムを反復することによってcurrentNode(またはそのインデックス)を見つけることができません。 私のプログラムでは、自分のローカルノードをリストの先頭に「前」に設定しました。それから、previous.nextはcurrentNodeと等しくないのに対し、previous(previous = previous.next。) はwhile文の前に比較してprint文を前に比較し、getCurrent()は真になるはずですが、そうではありませんそれは私の問題がある場所です。 – solllodolllo

1

あなたは、あなたのコード内でnullに対する保護を検討する必要があります。前提をしないのが最善です。

例えば、

public boolean atEnd() 
{ 
    return getCurrent().getNext() == null; 

} 

が良い

public boolean atEnd() 
{ 
    if (getCurrent() != null) 
    { 
    return getCurrent().getNext() == null; 
    } 
    return true; 
} 

として書き込まれることがありますこれは、必ずしもそれを行うための最善の方法ではありません。あなたはNoCurrentNodeExceptionか何かを投げたいかもしれません。それはあなたが探しているセマンティクスに依存します。

いずれにしても、NullPointerExceptionsには埋め込まれません。

関連する問題