时间戳

首页 > 解决方案 > 当一个类具有许多依赖项但需要根据某些条件仅使用其中一些依赖项时的设计方法

java - 当一个类具有许多依赖项但需要根据某些条件仅使用其中一些依赖项时的设计方法

问题描述

我有多个实现接口并返回对象的类。

public interface DataFetcher {
    Data getData(Info info);
}

public class Data {
    private String name;
    private String value;
}


@Component
public class DataPointA implements DataFetcher {
    @Override
    public Data getData(Info info) {
        //..Do some processing
        return new Data("SomeName", valueComputed);
    }
}

现在我有大约 20 个数据点，它们实现了 DataFetcher 类并返回数据对象。

我将所有数据点自动连接到一个类，并根据某些条件使用某些数据点。

@Component
public class DataComputer {
    @Autowired
    private DataPointA dataPointA;

    @Autowired
    private DataPointB dataPointB;
    .
    .
    .

    public void computeData(String inputType, Info info) {
        List<DataFetcher> dataFecthers;
        switch(inputType) {
            case "typeA" : dataFecthers  = ImmutableList.of(dataPointA, dataPointB);
                            break;

            .
            .
            .
            case "typeD" : dataFecthers  = ImmutableList.of(dataPointE, dataPointF, dataPointG);
                            break;
        }

        dataFetcher.forEach(dataPoint -> {
            //Do some processing with  dataPoint.getData(info)
        })
    }
}

可以看出，DataComputer 类将有一个完整的依赖项列表，这些依赖项可能变得无法管理。此外，基于 inputType 使用的数据点也是事先知道的，因此可以将其提取出来。这是我的尝试：

@Component 
public class DataComputationPointDecider {
    @Autowired
    private DataPointA dataPointA;

    @Autowired
    private DataPointB dataPointB;

    .
    .
    .

    @Bean
    public Map<String, List<DataFetcher>> getDataComputationPoints() {
        return new ImmutableMap.Builder<String, List<DataFetcher>>()
            .put("typeA", ImmutableList.of(dataPointA, dataPointB))
            .put("typeD", ImmutableList.of(dataPointE, dataPointF, dataPointG))
            .build();
    }
}

然后我的 DataComputer 依赖项减少了：

@Component
public class DataComputer {
    @Autowired
    private Map<String, List<DataFetcher>> dataComputationPoints;

    public void computeData(String inputType, Info info) {
        List<DataFetcher> dataFecthers = dataComputationPoints.get(inputType);
        dataFetcher.forEach(dataPoint -> {
            //Do some processing with  dataPoint.getData(info)
        })
    }
}

有没有更好的方法来设计这个？

标签： javaspringjava-8

解决方案

我认为您的方法没有任何重大错误。但我建议另一种选择。

您可以决定或说出它可以处理哪些输入类型，而不是维护一个映射一个inputType与一个列表的映射。DataFetcherDataFetcher

但这需要改变DataFetcheras的接口

public interface DataFetcher {
    boolean canHandle(String inputType);
    Data getData(Info info);
}

实现看起来像

@Component
public class DataPointA implements DataFetcher {
    @Override
    boolean canHandle(String inputType) {
         return "typeA".equals(inputType);
    }

    @Override
    public Data getData(Info info) {
        //..Do some processing
        return new Data("SomeName", valueComputed);
    }
}

然后您可以将所有内容DataFetcher作为一个列表注入（并且不需要@Autowired为每个列表添加一个字段）并将其处理为

@Autowired
List<DataFetcher> dataFetchers;

...

dataFetchers.stream()
     .filter(dataFetcher -> dataFetcher.canHandle(inputType))
     .forEach(dataFetcher.getData(info));

优点：

在您当前的方法中，如果您添加一个新的DataFetcher实现，您需要添加一个@AutoWired字段/成员并修改 (getDataComputationPoints) 映射。但是，有了这个，DataFetcher可以处理的 inputTypes 是用它本身指定的，因此您只需要为新的输入类型添加新的类。

参考

按类型将引用 bean 自动装配到列表中

更新：

缺点

  1. 输入类型是在类中指定的，这意味着您无法轻松找到DataFetchers给定输入类型的（数据点）列表。

  2. 如果您需要删除对 inputType 的支持，那么您需要再次访问每个实现（从 中删除该 inputType canHandle）。在您的方法中，它只是简单地删除一个地图条目。

推荐阅读