时间戳

首页 > 解决方案 > Java 8 将对象添加到列表的有效方法

java - Java 8 将对象添加到列表的有效方法

问题描述

是否有一种简短有效的方法可以将对象添加到具有空值检查的列表中？

Book book = new Book();
if (CollectionUtils.isNotEmpty(library.getBooks())) {
        library.getBooks().add(book);
} else {
        library.setBooks(Collections.singletonList(book));
}

标签： javalistperformancejava-8

解决方案

List我绝对不喜欢你可以包含的事实，那有null什么意义呢？你为什么不重构并返回一个空的。假设您无法更改它（如果可以，请仔细检查），不要仅仅因为它很花哨而在一行中执行它 - 在这种特殊情况下它将非常难以理解。此外，我强烈反对CollectionsUtilsorStringUtils等​​这些辅助方法——它们只会让阅读代码变得更糟；并不是说所有的人（guava有一些我真的很喜欢和使用）

相反，将其重构为一种更简单、更易于查看的方法：

List<Book> books = library.getBooks();
if(books != null && !books.isEmpty()) {
     books.add(book);   
} else {
     library.setBooks(Collections.singletonList(book));    
}

你用了多少秒来理解这个逻辑？例如，与此处的另一个答案相反（不是个人的，您的观点很好）。

但这仍然对我个人提出了一些问题。为什么不把整个逻辑移到Library课堂上呢？如果您想稍后将另一本书添加到图书馆怎么办？因为你使用Collections.singletonList了不可变的，你将无法做到。

推荐阅读