for循环中的Java冗余第二个if语句

Question

我是java新手，受这次讨论的启发，我在这里写了一个while循环版本和一个for循环版本（这个问题），这个程序在数组中搜索一个“项”（在main方法中称为int 20）并返回其值，这两个版本都有效。

然而，在这个for循环版本中，第二个if语句，尤其是“database[i]==database[database.length-1]”，看起来是多余的，因为for循环中的终止条件已经指定

class For {

static int [] database = {17,18,19,20,21};

public static int findItem(int item) {

    for ( int i=0; i<database.length; ++i ) {
        if ( database[i] == item ) {
            System.out.println("Item found at position: " + i);
            return i;
        }
        if ( database[i] == database[database.length-1] && database[i] != item )
            System.out.println("Item not found.");
    }
    return item;
}

public static void main(String [] args) {
    findItem(20);
}}

我尝试用else语句替换第二个if，但for循环计算了数组中20之前的每个值，并打印了三次“Item not found”。

这是对for循环和if语句的可接受使用，还是完全冗余？

Answer 1

您可以通过在循环之后移动“not found”消息来简化该方法，如下所示：

public static int findItem(int item) {

    for (int i = 0; i < database.length; i++) {
        if ( database[i] == item ) {
            System.out.println("Item found at position: " + i);
            return i;
        }   
    }
    System.out.println("Item not found.");
    return -1; // not found
}

因为你已经在循环中退出了return方法，如果找到了数字。

还请注意，我已经更改了返回-1的方法，以防找不到数字。这是一种常见的做法，表明搜索没有找到结果。

Answer 2

如果我理解正确，那么如果该项不在数组中，您希望打印一条消息。这不应该在for循环内完成，因为此时您还没有处理整个数组。应在for循环之后进行：

for (int i = 0; i < database.length; i++) {
    if (database[i] == item) {
        System.out.println("Item found at position: " + i);
        return i;
    }
}
System.out.println("Item not found.");
return item;

现在，如果没有找到物品，就把它退回是个糟糕的主意。调用者将假定在索引项中找到了该项，这是不正确的。如果找不到项，则应返回一个特殊值（或引发异常），不能将其与有效索引混淆。例如：-1（这是在这种情况下返回的传统值）：

for (int i = 0; i < database.length; i++) {
    if (database[i] == item) {
        System.out.println("Item found at position: " + i);
        return i;
    }
}
System.out.println("Item not found.");
return -1;

现在，这个方法不应该负责打印它返回的结果。打电话的人应该这样做：

public static int findItem(int item) {
    for (int i = 0; i < database.length; i++) {
        if (database[i] == item) {
            return i;
        }
    }
    return -1;
}

public static void main(String[] args) {
    int index = findItem(20);
    if (index >= 0) {
        System.out.println("Item found at position: " + index);
    }
    else {
        System.out.println("Item not found");
    }
}