Java: How to search an arraylist for a instance variable of an object in the array

Question

The following piece of code is giving me a massive headache. It only kind of does what I want it to do, which is take an inputted string, and then search the arraylist of book objects by the object's classification (a string) and then return that book object. Currently the code below works as long as the input classification is an actual classification of a book in the list. Eg TD999 is found so it is returned. But if I type a bunch of nonsense eg yhgfsdsfbv for the class, it still returns a book eg TD345 (seem it turns a book that was originall classed as null and then classed using the setClassification method when the programme runs.

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;
}

My attempt to fix it was this:

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;
}

However this will throw the exception every single time, even if I correctly enter a classification that is in the list.

I have no idea how to fix this to do what I want, I have tired doing it numerous other ways, never works properly. I have read a through all my lecture notes, several textbooks and oracle webpages and have no idea what is causing the problem, hence I am unable to fix it.

John Kugelman · Accepted Answer · 2016-04-14 12:28:09Z

Let's take your first attempt. If you find a book you return it immediately. Therefore, if you get through the entire loop without finding anything, you should signal failure: return null, or throw an exception. There's no point in returning foundBook because you know you didn't find anything.

That means that there's no need for a foundBook variable at all.

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");
}

Another way to write this is to use Java 8 streams. Instead of an explicit loop you can let a stream take care of looping.

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"));
}

dryairship · Accepted Answer · 2016-04-14 12:28:39Z

1

The problem is that, here:

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

If the if condition is true, then it returns the book, else, if it is not true, then it sets foundbook to findclass.

Then, in the end, you return foundbook, which will always be returned, even if no book matched.

You must do:

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.

answered Apr 14, 2016 at 12:28

dryairship

6,0574 gold badges32 silver badges57 bronze badges

Comments

Yusuf K. · Accepted Answer · 2016-04-14 12:28:58Z

I will try to explain on code please read comment and fix your own so it will be useful to make you learn, I hope.

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")


}

thiagosteps227 · Accepted Answer · 2021-01-25 10:59:56Z

0

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;
}

answered Jan 25, 2021 at 10:59

thiagosteps227

462 bronze badges

2 Comments

thiagosteps227 Over a year ago

You can use this method that returns the index of the person in a ArrayList of people. If the person that you want is not found, the method returns -1. Useful and simple. I hope I could help! cheers

sɐunıɔןɐqɐp Over a year ago

From Review: Would you please also edit the answer and add and explanation? Thanks!

Collectives™ on Stack Overflow

Java: How to search an arraylist for a instance variable of an object in the array

4 Answers 4

Comments

Comments

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related