oop - 信息隐藏和函数式编程编码风格

Question

我正在开发一个名为 Simulator 的简单类，它将 Simulation 列表应用于特定输入。对于每个输入，模拟可以产生或不产生输出，这取决于输入对于每个模拟必须满足的某些条件。 Simulator 产生的结果是一个输出列表。

这是代码。

class Simulator {
    final List<Simulation> simulations;

    // Some initialization code...

    List<Ouput> execute(Input input) {
        return simulations
            .stream()
            .filter(s -> s.processable(input))
            .map(s -> s.prepareOutput(input))
            .collect(Collectors.toList());
    }
}

如您所见，我首先验证输入是否可由 Simulation 处理，过滤不能处理的模拟，然后将这些模拟应用于输入。

从面向对象的角度来看，我公开了 Simulation 类的内部结构。 processable 方法完成的检查操作应该隐藏在prepareOutput 方法中。

但是，让 processable 对 Simulator 可见，我可以应用更实用的方法，这非常方便。

哪种方法更好？我还缺少其他解决方案吗？

Answer 1

由于您的类 Simulation 已经公开了可能会失败的操作 prepareOutput，因此当您提供方法 processable 时，没有额外公开提前检测 prepareOutput 操作是否会失败。实际上，提供这样的检查是一个很好的 API 设计，只要提前计算的成本不是太高即可。

您仍然可以考虑在 Simulation 类中提供批量处理操作。

public class Simulation {
    public Output prepareOutput(Input input) {
        …
    }
    public static List<Output> prepareWhenPossible(List<Simulation> list, Input input) {
        return simulations.stream()
            .filter(s -> s.processable(input))
            .map(s -> s.prepareOutput(input))
            .collect(Collectors.toList());
    }
}

重要的是要让调用者清楚地知道它将跳过无法进行操作的元素，而不是实现“全有或全无”的行为。

这仍然不排除公开 processable，如果实现起来很便宜的话。这并不是说这是不可能的操作，因为总是可以调用 prepareOutput 并删除结果来确定该操作是否可行。为此目的使用可处理方法会更简洁。