Java 8 Stream循环使用问题：结果为Null求解决方案

核心问题 #

1. 变量覆盖：循环中每次通过allBooks = ...重新赋值，最终仅保留最后一次循环的结果，前两次查询的书籍数据全部丢失。
2. 笔误导致返回错误对象：map操作中创建了bookInfo但返回了未定义的pullRequestInfo（若该变量为null，会导致最终列表全是null；若未声明则直接编译失败）。
3. 未初始化与空指针风险：allBooks未提前初始化，若循环因异常中断，会直接返回null；同时未对getBooks或getValues的返回值做null校验，易触发空指针异常。

修复后的代码（循环版） #

import java.util.ArrayList;
import java.util.Arrays;
import java.util.List;
import java.util.stream.Collectors;

public List<Book> updateBooks(String libraryId) {
    // 简化States列表初始化
    List<States> states = Arrays.asList(States.AVAILABLE, States.BORROWED, States.BLOCKED);
    // 提前初始化总列表，避免返回null
    List<Book> allBooks = new ArrayList<>();
    
    for (States state : states) {
        // 假设getBooks返回的是包含getValues()方法的对象，统一命名为BooksResponse
        BooksResponse books = getBooks(libraryId, state.getValue());
        
        // 空指针防护：跳过null的查询结果
        if (books == null || books.getValues() == null) {
            continue;
        }
        
        List<Book> currentBooks = books.getValues().stream()
            .map(book -> {
                Book bookInfo = new Book();
                bookInfo.setId(book.getId());
                bookInfo.setTitle(book.getTitle());
                bookInfo.setState(book.getState());
                bookInfo.setLinks(book.getLinks());
                // 修复笔误：返回创建好的bookInfo
                return bookInfo;
            })
            .collect(Collectors.toList());
        
        // 将当前状态的书籍追加到总列表，而非覆盖
        allBooks.addAll(currentBooks);
    }
    
    return allBooks;
}

更简洁的Stream替代方案 #

如果想完全去掉循环，用Stream API实现更简洁的逻辑：

import java.util.Arrays;
import java.util.List;
import java.util.Objects;
import java.util.stream.Collectors;

public List<Book> updateBooks(String libraryId) {
    return Arrays.asList(States.AVAILABLE, States.BORROWED, States.BLOCKED)
        .stream()
        // 根据每个状态查询书籍
        .map(state -> getBooks(libraryId, state.getValue()))
        // 过滤null的查询结果
        .filter(Objects::nonNull)
        // 获取书籍列表
        .map(BooksResponse::getValues)
        // 过滤null的书籍列表
        .filter(Objects::nonNull)
        // 扁平化流：将多个列表合并为一个流
        .flatMap(List::stream)
        // 转换为目标Book对象
        .map(book -> {
            Book bookInfo = new Book();
            bookInfo.setId(book.getId());
            bookInfo.setTitle(book.getTitle());
            bookInfo.setState(book.getState());
            bookInfo.setLinks(book.getLinks());
            return bookInfo;
        })
        // 收集为最终列表
        .collect(Collectors.toList());
}

关键修复点说明 #

• 修复笔误：确保map操作返回的是创建好的bookInfo对象，而非无关变量。
• 避免覆盖：使用addAll追加结果（循环版）或flatMap合并流（Stream版），保留所有状态的书籍数据。
• 空指针防护：添加null校验，避免因查询结果为空导致的异常，同时保证最终返回的是一个空列表而非null。

内容的提问来源于stack exchange，提问作者Tindona