LinkedList 删除测试失败

Question

public void delete(int index) {
    if (index < 0 || index > count-1) {
        throw new ArrayIndexOutOfBoundsException();
    }

    Node<X> current = head;
    if (index == 0) {
        current = current.getLink();
        count--;
        return;
    }
    for (int i = 0; i < index-1; i++) {
        current = current.getLink();
    }

    current.setLink(current.getLink().getLink());
    count--;

}

.

@Test
public void testDeleteSeveral()
{
    LList<String> b = new LList<String>();
    b.add("hello");
    b.add("bye");
    b.add("adios");
    b.add("ciao");
    b.add("see ya");
    b.delete(0);
    assertEquals(4, b.size());
    assertEquals("ciao", b.get(0));
    assertEquals("adios", b.get(1));
    assertEquals("bye", b.get(2));
    assertEquals("hello", b.get(3));

.

public void add(X v)
{
    Node<X> a = new Node<X>();
    a.setValue(v);
    a.setLink(head);
    head = a;
    count++;
}

在上面写着assertEquals("ciao", b.get(0)); 的地方，我说它应该是“ciao”，因为它删除了第一个节点，但是当我认为“see ya”被删除时，错误出现了，因为它期待“see ya”我做了delete(0); 我错过了什么吗？

编辑：我添加了 add 方法

Answer 1

您的删除方法不正确。这是一个更正的版本，解释了你做错了什么。

    if (index == 0)
    {
        //we're deleting the head here. You need to reassign head variable.
        //current = current.getLink(); <-- no. this doesn't change the structure.
        head = head.getLink();//here we set head to head's next. Now structure is changed.
        count--;
        return;
    }

另外，你的 add 方法有点奇怪。它将项目附加到队列的前面。也许这就是你想要做的，但请注意，正如它所写的那样，项目将与你添加的顺序相反。所以...

list.add(1);
list.add(2);
list.add(3);

实际上将按顺序包含项目[3, 2, 1] 而不是[1, 2, 3]。

我假设你不想要这个，然后给你写一个新的 add 方法。

public void add(X v)
{
    Node<X> a = new Node<X>();
    a.setValue(v);
    if(head==null)
        head = a;
    else
    {
        Node<X> tail = getTail();
        tail.setLink(a);
    }
    count++;
}
private Node<X> getTail(){
    if(head==null)
        return null;
    Node<X> current = head;
    while(current.getLink()!=null)
        current = current.getLink();
    return current;
}

这会将项目按顺序添加到您的列表中。然而，结果是现在添加一个项目是 O(n)。这在单链表中是不可避免的。

编辑：

改写你的日常。我很无聊，请原谅我。

public void delete(int index) {
    if (index < 0 || index > count-1) {
        throw new ArrayIndexOutOfBoundsException();
    }

    if (index == 0) {
        head = head.getLink();
    }else{
        Node<X> beforeDeletedNode = head;
        for (int i = 0; i < index-1; i++) {
            beforeDeletedNode = beforeDeletedNode.getLink();
        }
        Node<X> toDelete = beforeDeletedNode.getLink();
        beforeDeletedNode.setLink(toDelete.getLink());
    }
    count--;

}

Answer 2

这里你的删除方法有问题。

if (index == 0)
{
    current = current.getLink();
    count--;
    return;
}

它实际上并没有删除，因为“头”仍然指的是同一个节点。试试

if (index == 0)
{
    head = current.getLink();
    count--;
    return;
}