Java：如何在数组列表中搜索数组中对象的实例变量

Question

下面的一段代码让我非常头疼。 它只做我想要它做的事情，即获取输入的字符串，然后通过对象的分类（字符串）搜索书对象的数组列表，然后返回该书对象。 目前，只要输入分类是列表中书籍的实际分类，下面的代码就可以工作。 例如，找到了 TD999，因此将其返回。 但是，如果我为该类键入一堆废话，例如 yhgfsdsfbv，它仍然返回一本书，例如 TD345（似乎它变成了一本最初归类为 null 然后在程序运行时使用 setClassification 方法归类的书。

public Book searchClass(String inputClass){
    Book foundBook = null;
    for(Book findClass : bookStore){
        if(findClass.getClassification()!=null &&
 findClass.getClassification().equalsIgnoreCase(givenClass)) 
        return findClass;
            foundBook = findClass;
    }
    return foundBook;
}

我试图修复它是这样的：

public Book searchClass(String inputClass) throws IllegalArgumentException{

    Book foundBook = null;
    for(Book findClass : bookStore){
        if(!(findClass.getClassification().equalsIgnoreCase(inputClass))){
                   throws new IllegalArgumentException("book not found")
        }
        else if(findClass.getClassification().equalsIgnoreCase(inputClass)){
            foundBook = findClass;
        }
    }
    return foundBook;
}

但是，即使我正确输入了列表中的分类，这也会每次都抛出异常。

我不知道如何解决这个问题来做我想做的事，我已经厌倦了很多其他的方式，从来没有正常工作过。 我已经通读了我所有的讲义、几本教科书和 oracle 网页，但不知道是什么导致了问题，因此我无法修复它。

Answer 1

让我们开始你的第一次尝试。 如果您找到一本书，请立即归还。 因此，如果你在整个循环中没有找到任何东西，你应该发出失败信号：返回空值，或者抛出异常。 返回foundBook是没有意义的，因为你知道你没有找到任何东西。

这意味着根本不需要foundBook变量。

public Book searchClass(String inputClass) {
    for (Book book: bookStore) {
        if (book.getClassification() != null &&
            book.getClassification().equalsIgnoreCase(inputClass))
        {
            return book;
        }
    }

    throw new IllegalArgumentException("book not found");
}

另一种编写它的方法是使用 Java 8 流。 您可以让流处理循环，而不是显式循环。

public Book searchClass(String inputClass) {
    return bookStore.stream()
        .filter(book -> book.getClassification() != null)
        .filter(book -> book.getClassification().equalsIgnoreCase(inputClass))
        .findAny()
        .orElseThrow(() -> new IllegalArgumentException("book not found"));
}

Answer 2

问题是，在这里：

public Book searchClass(String inputClass){
    if(findClass.getClassification()!=null && findClass.getClassification().equalsIgnoreCase(givenClass)) 
    return findClass;
        foundBook = findClass;
}

如果 if 条件为真，则返回书，否则，如果条件不为真，则将foundbook设置为findclass 。

然后，最后，您返回foundbook ，即使没有匹配的书，它也将始终返回。

你必须这样做：

for(Book findClass : bookStore){
    if(findClass.getClassification()!=null && findClass.getClassification().equalsIgnoreCase(givenClass)) 
    return findClass; //return the book
}
return null; // if no book was found, then return null.

Answer 3

我将尝试对代码进行解释，请阅读评论并修复您自己的评论，以便让您学习有用，我希望。

public Book searchClass(String inputClass) throws IllegalArgumentException{

    Book foundBook = null;
    for(Book findClass : bookStore){
       // if(!(findClass.getClassification().equalsIgnoreCase(inputClass))){
         //          throws new IllegalArgumentException("book not found")
        //}//remove these because each loop checks that 
        if(findClass.getClassification().equalsIgnoreCase(inputClass)){
            foundBook = findClass; // return findClass; // you do not need any more iteration you found it.
        }
    }
    return foundBook; //if you are here that means no match throw exception //throws new IllegalArgumentException("book not found")


}

Answer 4

public static int personThereAlready(String name, ArrayList<Person> people) {
    int index = -1;
    for(int i=0; i < people.size();i++) {
        Person person = people.get(i);
            
        if (person.getName().equalsIgnoreCase(name)) {
            index = i;
            break;
        }
    }
    return index;
}