は、コード設計を改善するためのステートメント

Question

私は以下のコードの設計を改善するにはどうすればよい

場合を取り除きます「もの」を設定する必要があります。クライアントは、フラグが設定されている場合、各フラグに対して何をすべきかを知っています。私はClientのif文と、単にConfigurationインスタンスのブール値フラグを設定するという概念を取り除きたいと思います。これにもっと「汎用的な」オブジェクト指向のアプローチを提案できますか？

種類は、あなたが落とし穴

は、このを見てみましょう...列挙子のように....その場合には、設計では、より柔軟なものを使用してみてくださいを書いているように見えます

Answer 1

あなたはStrategyパターンを使用することができます。 各Ifは独自のロジックでdoStuffを実装する戦略になります。プロパティ（A、B ...）が設定されている場合は、ストラテジをリストに追加します。あなたは、単に戦略上のループに必要と任意のIFSせずに、それらを実行します。

public class Foo { 
    public Configuration configure() { 
     return new Configuration().withPropertyA().withPropertyB(); 
    } 
} 

class Configuration { 
    Set<StuffStrategy> strategies = new HashSet<StuffStrategy>(); 

    public Configuration withPropertyA() { 
     strategies.add(new PropertyAStrategy()); 
     return this; 
    } 

    public Configuration withPropertyB() { 
     strategies.add(new PropertyBStrategy()); 
     return this; 
    } 

    public void executeStrategies() { 
     for (StuffStrategy strategy : strategies) { 
      strategy.doStuff(); 
     } 
    } 
} 

interface StuffStrategy { 
    public void doStuff(); 
} 
class PropertyAStrategy implements StuffStrategy { 
    @Override 
    public void doStuff() { 
    } 
} 

class PropertyBStrategy implements StuffStrategy { 
    @Override 
    public void doStuff() { 
    } 
} 

class Client { 
    public void someMethod(Configuration config) { 
     config.executeStrategies(); 
    } 
} 

Answer 2

について：

例：

public class Foo { 
    public Configuration configure() { 
    return new Configuration(Config.A); 
    } 
} 

class Configuration { 

enum Config{ 
A,B, NONE} 

    private Config propertyConfig = Config.NONE; 

    public Configuration(Config a) { 
    propertyConfig=a; 
    } 


    public Config getConfig() { 
    return this.propertyConfig; 
    } 

    ... 
} 

class Client { 

    public void someMethod(Configuration config) { 
     switch (config.getConfig()) { 
    case A: 
     System.out.println("a config"); 
     break; 
    case B: 
     System.out.println("b config"); 
     break; 

    default: 
     break; 
    } 
    // ... 
} 

} 

Answer 3

私は確かに、あなたはより多くのパラメータを持つことができる場合は、currenly特に何よりも多くのオブジェクト指向のアプローチを見つけることができるとは考えていませんこれはどういうわけかよく知られているデザインパターンBuilderです。あなたのオブジェクトをConfigurationから外して不変にすることによって、このパターンを正しく実装することができれば、ビルダークラスだけがConfigurationのインスタンスを作成できるはずです。あなたのビルダーは、変更不可能なConfigurationインスタンスを返すbuild()メソッドを持つ可変静的内部クラスでなければなりません。