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

Question

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

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)
        })
    }
}

有更好的设计方法吗？

Answer 1

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

您可以让 DataFetcher 决定或说出什么输入类型，而不是维护将 inputType 与 DataFetcher 列表进行映射的映射(s) 它可以处理。

但这需要将DataFetcher的接口(interface)更改为

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 作为一个列表注入(inject)(不需要为每个列表添加一个 @Autowired 字段)并将其处理为

@Autowired
List<DataFetcher> dataFetchers;

...

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

优点:

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

引用

Autowire reference beans into list by type

更新:

缺点

1. 输入类型在类内部指定，这意味着您无法轻松找到给定输入类型的DataFetchers(数据点)列表。

2. 如果您需要删除对 inputType 的支持，那么您需要再次访问每个实现(以从 canHandle 中删除该 inputType)。在您的方法中，只需删除一个 map 条目即可。