Java8列表。调用void函数

map 在您的场景中不是正确的操作。 forEach 是,因为它是一个终端操作(因此您不需要在它之后进行任何调用),并且它只消耗 Stream 无需返回任何内容的元素:

users.stream().forEach (o -> sendEmail(o));

或者你可以直接打电话 forEach公司 在 List ,正如Aris所建议的:

users.forEach (o -> sendEmail(o));

不清楚您为什么在这里使用迭代器。

List<User> users = menuPriceSummaryService.findAll()
            .stream()
            .flatMap (mps -> checkPreferences(mps).stream())
            .collect(Collectors.toList());

那就用吧 List.forEach :

users.forEach(o -> sendEmail(o));

如果您不需要 users 之后,您可以替换 collect 以上带有 forEach (当然,还要删除变量声明)。