Question

我正在从Java 8学习一些技巧。我创建了一个简单的列表：

 private void createData() {
        bottles.add(new Whiskey("Jack Daniels", "PL"));
        bottles.add(new Whiskey("Balentains", "PL"));
        bottles.add(new Whiskey("Balentains", "EN"));
        bottles.add(new Whiskey("Balentains", "EN"));
        bottles.add(new Whiskey("Balentains", "GR"));
        bottles.add(new Whiskey("Balentains", "PL"));
        bottles.add(new Whiskey("Balentains", "GR"));
    }

现在我想通过几件事从此列表中获取项目。如果用户提供参数origin，我想用origins过滤此列表，但是当他输入错误的origin时，他应该得到空列表，而当他不输入{{1 }}参数，那么他应该获得整个列表。

我有一种过滤列表中项目的方法：

origin

还有一个获取参数（或不获取参数）并返回结果的主要方法：

 private Optional<List<Whiskey>> getWhiskeyFromCountry(String origin) {
        final List<Whiskey> whiskies = bottles.stream()
                .filter(b -> b.getOrigin().equals(origin))
                .collect(Collectors.toList());

        return whiskies.isEmpty() ? Optional.empty() : Optional.of(whiskies);
    }

问题是我仍然在最后一行使用if statemant，但我不想这样做。我想将此代码更改为清晰实用的代码。我试图用Optionals做一些魔术，但最终我得到了这个，我认为它可以做得更好，更简单。你可以帮帮我吗？还是这段代码不错，我不需要更改任何内容？这个问题更多的是关于干净的代码。

Answer 1

您可以使此方法getWhiskeyFromCountry接受Optional<String>作为参数

private List<Whiskey> getWhiskeyFromCountry(Optional<String> origin)

然后如果Optional为空，则返回空列表或基于filter的列表，如果用户输入了错误的origin，您仍然会得到空列表

return origin.map(o->bottles.stream()
            .filter(b -> b.getOrigin().equals(o))
            .collect(Collectors.toList())).orElse(Collections.EMPTY_LIST);

或者在上面的代码中，您可以进行细微调整以为此方法List返回getWhiskeyFromCountry

 private List<Whiskey> getWhiskeyFromCountry(String origin) {

    return bottles.stream()
            .filter(b -> b.getOrigin().equals(origin))
            .collect(Collectors.toList());
  }

在主方法中，使用Optional.map

Optional<String> origin = Optional.ofNullable(routingContext.request().params().get("filter"));
List<Whiskey> result = origin.map(o->getWhiskeyFromCountry(o))
                             .orElse(bottles);

Java 8代码优化-删除if语句

1 个答案: