番石榴功能组成

Question

我看到有一个实用程序可以创建 Function，它是 Function 和 Function 的组合;

我有一个类似的情况，但有点不同。

我的第一个函数是 valueOfFunction，它根据我的 BO 类中的键返回一个枚举。

secind 函数使用参数调用 Enum 上的方法，该参数是 BO 对象。

所以它不完全是 A->B->C

这是函数：

private final static class RequestConvertor implements Function<CoreData, List<Request>> {
    private final static Function<String,RequestConvertorEnum> typeConvertor =  valueOfFunction(RequestConvertorEnum.class);

    @Override
    public List<Request> apply(CoreData coreData) {
        RequestConvertorEnum requestConvertorEnum = typeConvertor.apply(coreData.getType());
        return requestConvertorEnum.convertToRequests(coreData);
    }

}

这是枚举的方法：

   private final List<Request> convertToRequests(CoreData coreData) {
        List<PropertyWrapper> properties = getProperties(coreData);
        if (properties.size() == 0) {
            return Collections.emptyList();
        }
        Request request = new Request(coreData.getKey(), properties, new RequestMetaData(
                coreData.getFoo()));
        return newArrayList(request);
    }

有没有更好的方法将这两个函数组合在一起？

Answer 1

我认为 valueOfFunction 的使用首先是不合适的 - 您仅在调用 的另一个函数（它是 private static 成员）中使用它立即应用方法。您应该使用良好的旧 Enum.valueOf(String) 静态方法：

private final static class RequestConvertor 
        implements Function<CoreData, List<Request>> {
    @Override
    public List<Request> apply(CoreData coreData) {
        return RequestConvertorEnum.valueOf(coreData.getType())
                .convertToRequests(coreData);
    }
}

请注意，提供不代表任何枚举值的 String 会抛出 IllegalArgumentException，但在您当前的代码中，您在运行时有潜在的 NullPointerException （如果 Enum 常量不存在，Enums.valueOfFunction 返回 null），我想您已经意识到了这一点（如果您没有意识到，那么您现在已经意识到了） :)）。

此外，关于其余代码的建议很少。如果您不需要可变性，请使用 ImmutableLists 而不是 ArrayLists （我假设你不修改结果，因为 Collections.emptyList 本身是不可变的，所以如果您修改 convertToRequests 方法的结果，它将在运行时失败）。

private final ImmutableList<Request> convertToRequests(CoreData coreData) { // 1.
    List<PropertyWrapper> properties = getProperties(coreData);
    if (properties.size() == 0) {
        return ImmutableList.of(); // 2.
    }
    Request request = new Request(coreData.getKey(), properties, 
            new RequestMetaData(coreData.getFoo()));

    return ImmutableList.of(request); // 3.
}

一些解释：

1. 使用 ImmutableList 作为返回类型来保证行为（不可变性）。
2. 返回空的不可变列表，但比 JDK 的更好。
3. 与 2. 相同，但这里是单例不可变列表。

如果您需要可变性，请将 ImmutableList.of 替换为 Lists.newArrayList 并保持方法签名不变。