删除列表的一部分并成功返回true的方法的正确语法是什么？

Question

I m trying to create a method that searches the content of a list called authors looking for the element (the author's name) given as arg, and then deletes the specific field. If successful it returns true. But it won't work for some reason. I believe the error lies in the authors.remove(authorName); because the main class will not erase anything when given the order Book.removeAuthorByName("White"); .

public boolean removeAuthorByName(String authorName){
    boolean val = authors.contains(authorName);
    for (int i = 0; i <= numAuthors; i++){
        if(val = true){
            authors.remove(authorName);
            authors.trimToSize();
        }
    }
    return val;
}

public int listSize(){
    return authors.size();
}

Answer 1

Little more concise and better will be

public boolean removeAuthorByName(String authorName){

     int index = authors.indexOf(authorName);
     if(index > -1){
         authors.remove(index);
         return true;
     }
     return false;
 }

Answer 2

First of all, you should check whether your list supports the remove(Object) method; it does because otherwise you would have received an UnsupportedOperationException.

You can simply have to call authors.remove(name); the method will return true if the passed name was found and deleted successfully - see the documentation .

Answer 3

Remove the while loop check using if clause. Also check case sensitivity

if(list.contains(authorname)){

    list.remove(authorname);

    return true;
 }
 else
 {
    return false;
 }