如何在lambda流中查找搜索条件中的最后一个对象并将其移除

Question

我有一张书单。其中一些书是相同的，但它们不是重复的，因为它们的不同是因为一个名为CopyNumber的变量。例子。<代码>第1册（标题：蜘蛛侠，ISBN:1111，版权编号:1) ，<代码>第2册（标题：蜘蛛侠，ISBN:1111，版权编号:2) ，<代码>第3册（标题：蜘蛛侠，ISBN:1111，版权编号:3) ，<代码>第4册（标题：爱丽丝梦游仙境，ISBN:2222，版权编号:1) ，<代码>第5册（标题：爱丽丝梦游仙境，ISBN:2222，版权编号:2) 。所以我的目标是把所有这些书按拷贝号顺序列出，通过ISBN号找到这本书（和它的拷贝），然后删除最后一个拷贝。因此，对于上面的示例，如果调用deletebook（“1111”）；，我将删除Spiderman的第三个副本。如果调用deletebook（“2222”）；，我会删除爱丽丝梦游仙境的第二个副本

public void deleteBook(String isbn){
    Collections.sort(books, new OrderBooksByCopyNumber());
    Book book = books.stream().filter((b) -> (b.getISBNNumber().equals(isbn))).findFirst().get();
    books.remove(book);

 }

上面的代码很接近我想要的，我不想要findfirst()，我想要执行与findlast()等价的操作。下面是我的比较，如果它有任何相关性。感谢任何能帮忙的人！

private class OrderBooksByCopyNumber implements Comparator<Book>{

    @Override
    public int compare(Book o1, Book o2) {
        return o1.getCopyNumber() <  o2.getCopyNumber() ? -1 : o1.getCopyNumber() == o2.getCopyNumber() ? 0 : 1;
    }

}

Answer 1

您的代码有几个问题。

首先，当您只想比较拷贝数时，没有理由编写自己的比较器。您编写的比较器等效于comparator.comparingInt(Book::GetCopyNumber)。

其次，如果您使用的是可选的get方法，那么您可能做错了什么。在本例中，您不考虑可选为空的情况--没有任何书籍与给定的ISDN匹配。相反，你应该做一些类似的事情

...findFirst().ifPresent( book -> books.remove(book) );

public void deleteBook(String isbn){
    books.stream()
         .filter((b) -> (b.getISBNNumber().equals(isbn)))
         .max(Comparator.comparingInt(Book::getCopyNumber))
         .ifPresent(book->books.remove(book));
}

int maxCopyNumber = -1;
int bookIndex = -1;

ListIterator iter = books.listIterator();
while ( iter.hasNext() ) {
   Book book = iter.next();
   if ( book.getISBNNumber().equals(isbn) && book.getCopyNumber() > maxCopyNumber ) {
       maxCopyNumber = book.getCopyNumber();
       bookIndex = iter.previousIndex();
   }
}
if ( bookIndex >= 0 ) {
   books.remove(bookIndex);
}

如果您的列表是ArrayList或类似的列表，这将防止遍历它两次，因为删除是在O(1)中执行的。

最后，考虑完全使用不同的数据结构！例如，映射 list > ，其中键是ISDN，每个条目是图书列表，按拷贝数排序。这将允许您直接到达列表，只需移除最后一个元素。（如果列表为空，则完全删除该条目，或者从一开始就检查它不是空的）。