スイッチからのケース数を減らす

Question

私は次のコードを持っています。これは、タイプのライターベースのインスタンスを取得するファクトリー関数のようなものです。種類はenumです。

public Writer getWriter(WriterTypeEnum type){ 
    switch(type){ 
    case A: new AWriter() break; 
    case B: ... break; 
    case C: ... break; 
    ... 
    } 
} 

問題は約30 caseです。それを減らしたり、まったく実装しないために私は何ができますか？

戦略パターンを使用して過去にこのようなケースを解決しましたが、ここではアプリ全体で古いenumが使用されています。もう一つの問題は、私はこの列挙体に春の豆を入れることができないということです。スイッチケースのインスタンスの一部はbeanです。

WriterTypeEnumには、UIから送信されるIDがあり、そのIDに基づいて適切なインスタンスを判断する必要があります。

Answer 1

ライター名のルックアップマップを作成してから、リフレクションを使用してインスタンスを作成するのはどうですか？

文字列writerNameJustFound = mapOfWriters.get（type）;

オブジェクトインスタンス= Class.forName（writerNameJustFound）.newInstance（）;あなたはセットアップあなたの列挙型をすることができます

Answer 2

は例

public enum YourEnum{ 
    FIRST(){ 
     public Writer getWriter(){ 
      return null; 
     } 
    }, ... // other enums 

    public abstract Writer getWriter(); 

} 

ため、抽象メソッド

public abstract Writer getWriter(); 

その後、すべてのインスタンスがそのメソッドを実装する必要がありますを持っているか、休憩を省略することができあなたのスイッチでは、繰り返しの文がある場合

Answer 3

これまでのソリューションはすべて歓迎されていて、おそらくいくつかは私のケースではうまくいくでしょう。

解決策は、Set<WriterMapping>という別のオブジェクトを作成することでした。このオブジェクトでは、私はWriterEnumTypeとその対応者Writerの値を格納します。

Answer 4

私が遭遇する別のタイプの答えは、継承を使用することです。つまり、WriterTypeEnumをWriterTypeにすることです。

public Writer getWriter(WriterType type){ 
    return type.getWriter(); 
} 

こうすれば、WriterTypeの正しい子を渡すだけで、getWriter内のコードには絶対に触れることはありません。 問題は、そのような解決策が実装可能かどうかにかかわらず、文脈にあります。