情報隠蔽と関数プログラミングのコーディングスタイル

Question

Simulatorと呼ばれる単純なクラスを開発していましたが、これは特定の入力にSimulationのリストを適用します。各入力に対して、シミュレーションは、各シミュレーションに関して入力が満たさなければならないいくつかの条件に基づいて、出力を生成するか否かを生成することができる。 Simulatorによって生成される結果は、出力のリストです。

ここにコードがあります。あなたが見ることができるように、入力がSimulationで処理可能である場合

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クラスの内部を公開しています。 processableメソッドで行われたチェック操作は、prepareOutputメソッド内に隠れているはずです。

Simulatorにはprocessableが表示されていますが、私はより機能的なアプローチを適用でき、非常に便利です。

どのアプローチが良いですか？私は行方不明の他の解決策はありますか？

Optional<Output> prepareOutput(Input input) { 

     boolean isProcessable = processable(input); // processable is private 

     if(isProcessable){ 
      // prepare Output 
      return Optional.of(Output); 
     } 
     return Optional.empty(); 

} 

そして、このような何か：あなたはprocessableを非表示にする必要がある場合は

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を呼び出して結果をドロップして、操作が可能かどうかを調べることが常に可能なので、そうでなければ不可能な操作であるようにはなりません。その目的のためにprocessableメソッドを使用する方がはるかにクリーンです。

Answer 2

は、なぜそれを異なるビットない

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