Question

我的任务是编写一个包装器和递归方法来搜索给定项，并返回该项在链表中的索引。这是我的代码，它适用于列表中的项目，但是当给出的项目不在列表中时，它仅返回尾部的索引。知道我在这里做错了吗？

public int searchIndex(E item) {
    return searchIndex(item, head, 0);
}

private int searchIndex(E item, Node<E> node, int index) {

    if (node == null) {     
        return -1;      
    }
    else if (item.equals(node.data)) {
        return 0;
    }
    else {
        return 1 + searchIndex(item, node.next, index);         
    }   



}

Answer 1

您的条件不正确。让我们分解一下：

private int searchIndex(E item, Node<E> node, int index) {
    if (node == null) {     
        // ok you say that if list ended found index is -1, i.e. not found
        return -1;      
    }
    else if (item.equals(node.data)) {
        // item is found, but your result is 0?
        // your result should be index
        return 0;
    }
    else {
        // your return is an index, so you can't make math on result
        // also passing same index over and over again 
        return 1 + searchIndex(item, node.next, index);         
    }   
}

对于递归，您必须陈述适当的条件。通常，您的回报是结果和参数的变化。

private int searchIndex(E item, Node<E> node, int index) {
  // break condition: not found at all
  if (node == null) return -1;
  // break condition: found at index
  if (item.equals(node.data)) return index;
  // continue condition: proceed to next node and index
  return searchIndex(item, node.next, index + 1);
}

Answer 2

当您返回-1时，您的递归语句会向其添加一个，并使其与由于匹配而导致尾节点返回零的情况无法区分。

在Java的递归链表中搜索项目的索引

2 个答案: