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

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时他应该得到空列表，当他不给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);
    }

也是一个获取参数（或不获取参数）和响应结果的主要方法：

private void getAll(RoutingContext routingContext) {
        Optional<String> origin = Optional.ofNullable(routingContext.request().params().get("filter"));
        List<Whiskey> result = getWhiskeyFromCountry(origin.orElse("")).orElse(Collections.EMPTY_LIST);

        routingContext.response()
                .putHeader("content-type", "application/json; charset=utf-8")
                .end(Json.encodePrettily(origin.isPresent() ? result : bottles));
    }

问题是我仍然在最后一行使用 if 语句，我不想​​这样做。我想将此代码更改为清晰实用。我试图用 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);