Question

我已经用Java做了一些练习，现在我遇到了这样的问题 - 我的列表工作不正确。我确信remove工作不正确，也许您可以帮助我（使用建议或代码）以正确的方式实现循环单个链接列表。我不确定其他功能是否正常，但我已尽力做到最好。

这是我的代码：

import java.util.*;


public class Node {
    private Object value;
    private Object nextValue;

    private Node next;


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

    public Object getValue() {
        return this.value;
    }

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

    public void setNextItem(Node nextItem) {
        this.next = (Node) nextItem;
        this.next.setValue(nextItem.getValue());
    }


    public void setValue(Object arg0) {
        this.value = arg0;
    }

}

    -------------------------------------------------------------------

   import java.util.*;



public class CircularList  {
    private Object[] array;
    private int arrSize;
    private int index;
    private Node head;
    private Node tail;
    public CircularList() {
        head = null;
        tail = null;
    }



    public boolean add(Node item) {

        if (item == null) {
            throw new NullPointerException("the item is null!!!");
        }


        if (head == null) {
            head = item;
            head.setNextItem(head);
            arrSize++;

            return true;
        } 

        Node cur = head;
        while(cur.nextItem() != head) {
            if(cur.getValue() == item.getValue()) {
                throw new IllegalArgumentException("the element already " +
                        "exists!");
            }
            cur = cur.nextItem();
        }

            head.setNextItem(item);
            item.setNextItem(head);
            arrSize++;

            return true;

    }


    public Node getFirst() {
        return head;
    }


    public void insertAfter(Node item, Node nextItem) {

        if ((item == null) || (nextItem == null)) {
            throw new NullPointerException("the item is nul!!!");
        } else if (this.contains(nextItem) == true) {
            throw new IllegalArgumentException("the item already exists!");
        } 

        Node cur = head;
        while(cur.nextItem() != head) {
            if(cur.getValue() == item.getValue()) {
                nextItem.setNextItem(item.nextItem());
                item.setNextItem(nextItem);
            } else {
                cur = cur.nextItem();
            }
        }

    }


    public boolean remove(Node item) {

        if(item == head) {
            Node cur = head;
            for(int i = 0; i < arrSize-1; i++) {
                cur = cur.nextItem();
            }

            head = head.nextItem();

            for(int i = 0; i < arrSize; i++) {
                cur = cur.nextItem();
            }
            arrSize--;
            return true;
        }

        Node cur = head;
        int counter = 0;
        while(cur.nextItem() != head) {
            if(cur == item) {
                item = null;
                cur = cur.nextItem();
                while(cur.nextItem() != head) {
                    cur.setNextItem(cur.nextItem().nextItem());
                }
            return true;    
            }
            cur = cur.nextItem();
        }



        return false;
    }

    public int size() {

        return arrSize;
    }

    public boolean contains(Object o) {
        if ((o == null) && (arrSize == 0)) {
            return false;
        }
        Node cur = head;
        while(cur.nextItem() != head) {
            if(cur.getValue() == o) {
                return true;
            }
            cur = cur.nextItem();
    }


        return false;
    }



}

Answer 1

其中许多算法可能更简单。

示例：

  public boolean remove(Node item) {
     Node current = head;
     for(int i = 0; i < size; i++) {
       if (current.getNext() == item) {
          current.next = current.getNext().getNext();
          size --;
          return true;
       }
       current = current.getNext()
     }
     return false;
  }

Answer 2

除此之外，此处还有各种各样的问题。您似乎将节点与==进行比较。此代码将输出“不匹配”。

Node n1 = new Node(5);
Node n2 = new Node(5);
if (n1 == n2)
  System.out.println("objects match");
else
  System.out.println("no match");

在add（）中，看起来你在列表中只能有两个项目，所以

head.setNextItem(item);
item.setNextItem(head);

应该是这样的：

cur.setNextItem(item);
item.setNextItem(head);

Answer 3

你的代码中有很多内容，这里有一些建议：

在你的Node类中：Java命名约定：与setter前缀为“set”的方式相同，getter的前缀应为“get：”nextItem()应该是getNextItem()
同样在您的Node类中：据我所知，链表的节点的“下一个值”字段通常是对列表中下一个节点的引用，因此应该是{ {1}}，而不仅仅是任何对象。它应该按照你的方式工作，但使用显式键入更安全。（如果使用“对象”确实是构建链表的下一个节点的常用方法，请纠正我。）
在Node的第一种情况下，当移除头部时：您正在遍历列表以达到最后一个值，可能是将其“下一个值”重置为新头，但是实际上并没有这样做。你想要这样的东西：
```
remove()
```
我不确定你希望用第二个循环完成什么。
在if (item == head) { head = head.nextItem(); for(int i = 0; i < arrSize-1; i++){ cur = cur.nextItem(); } } cur.setNextItem(head);的第二种情况中：我不确定您要对第二个remove()循环尝试做什么 - 重置整个列表中的所有链接？链表的重点是使其不必要。删除链接列表中的节点实际上并没有删除该对象（因此您不必将while设置为item）。相反，你只需“绕过”不需要的对象并“忽略”它，有效地将其从列表中删除，如：

原始列表：

null

删除节点B后：

[ Value: A; Next: B ] --> [ Value: B; Next: C ] --> [ Value C; Next: D ] ...

[ Value: A; Next: C ] --> [Value C; Next: D ] ...仍然存在于内存中，但没有任何内容指向它，因此它将在下一个垃圾收集周期中被删除。

要实现：当您遍历列表时，请保留对您访问过的上一个节点的引用。然后，一旦找到您正在寻找的项目（使用正确的比较，如Thomas所说），您只需设置[ Value: B; Next: C ]（警告：未经测试的代码）：

prev.setNextItem(cur.nextItem());

我希望这些技巧可以帮助您走上正确的道路。

圆形单链表

3 个答案: