インターフェイスから継承された異なるクラスの中から選択したいときにOCPに違反しない方法は？

Question

私はInterfaceがISendOutと言っています。これは2つの異なるクラスを継承しています たとえばTransferViaSerialPortとTransferViaWirelessModule（これらの2つのクラスでこのInterfaceを実装することを意味します）。 SerialPortまたはWirelessModuleでデータを送信する方法と、OCPに違反しない方法の間で、ユーザーが（UI内）を選択できるようにするにはどうすればよいですか？ 「Switch Case」または「If/Else」のステートメントを使用したい場合、私はOCPに違反します。

Answer 1

ファクトリパターンを使用する必要があります。ファクトリパターンをダイナミックにするには、にリフレクションを使用し、のクラスのタイプを表示するには、Dictionaryなどのカスタム属性やその他のメソッドを使用できます。

[System.AttributeUsage(System.AttributeTargets.Class)] 
public class DisplayNameAttribute : Attribute 
{ 
    public DisplayNameAttribute(string displayName) 
    { 
     DisplayName = displayName; 
    } 

    public string DisplayName { get; set; } 
} 

public interface ISendOut 
{ 
    void Send(string data); 
} 

[DisplayName("Wireless")] 
public class WirelessSendOut : ISendOut 
{ 
    public void Send(string data) 
    { 
     MessageBox.Show("data sent through wireless."); 
    } 
} 

[DisplayName("Serial")] 
public class SerialSendOut : ISendOut 
{ 
    public void Send(string data) 
    { 
     MessageBox.Show("data sent through serial port."); 
    } 
} 

public static class SendOutFactory 
{ 
    public static ISendOut CreateSendOut(string typeName) 
    { 
     var types = Assembly.GetExecutingAssembly().GetTypes(); 
     var sendOutType = types.First(x => (typeof(ISendOut)).IsAssignableFrom(x) && x.Name == typeName); 
     return (ISendOut) Activator.CreateInstance(sendOutType); 
    } 
} 

public static class SendOutDiscovery 
{ 
    public static IEnumerable<NameType> Discover() 
    { 
     var types = Assembly.GetExecutingAssembly().GetTypes(); 
     var sendOutTypes = types.Where(x => x != typeof(ISendOut) && (typeof(ISendOut)).IsAssignableFrom(x)); 
     return sendOutTypes.Select(type => GetNameType(type)).ToList(); 
    } 

    private static NameType GetNameType(Type type) 
    { 
     var nameType = new NameType 
          { 
           DisplayName = GetDisplayName(type), 
           TypeName = type.Name 
          }; 
     return nameType; 
    } 

    private static string GetDisplayName(Type type) 
    { 
     return ((DisplayNameAttribute)type.GetCustomAttributes(typeof (DisplayNameAttribute), false).First()).DisplayName; 
    } 
} 

public class NameType //for binding in UI 
{ 
    public string DisplayName { get; set; } 
    public string TypeName { get; set; } 
} 

public class SendOutViewModel //sample using in wpf (window contains a combobox) 
{ 
    public SendOutViewModel() 
    { 
     SendTypes = new ObservableCollection<NameType>(SendOutDiscovery.Discover()); 
    } 

    public NameType SelectedSendType { get; set; } //bind to selected item in combobox 

    public ObservableCollection<NameType> SendTypes { get; private set; } //bind to item source of combo 

    public string Data { get; set; } //data to be sent 

    public void Send() 
    { 
     ISendOut sendOut = SendOutFactory.CreateSendOut(SelectedSendType.TypeName); 
     sendOut.Send(Data); 
    } 
} 

後、私はクラスがそれをimplemetsインタフェースを継承しません

[DisplayName("Usb")] 
public class UsbSendOut : ISendOut 
{ 
    public void Send(string data) 
    { 
     MessageBox.Show("data sent through usb."); 
    } 
} 

Answer 2

あなたは、例えば、パラメータとしてISendOutの実装を渡す戦略パターン https://en.wikipedia.org/wiki/Strategy_pattern

http://www.dofactory.com/Patterns/PatternStrategy.aspx#_self1

Answer 3

をチェックコンストラクタに渡し、C＃のdynamic dispatchに "switch case"を実行させます。

これは、インターフェイスが非常に便利な理由です。間接指定があり、OCPを満たすためにdependency injectionを実行できます。

Answer 4

クラス自体は、OCPに違反するが、その実装を使用してUserConfiguredCommunicationModuleクラス（継承を超える賛成組成）

public class UserConfiguredCommunicationModule : ISendOut 
{ 
    public UserConfiguredUserModule(SerialPort serial, WirelessModule wireless) 
    {} 

    public void Send(string data) 
    { 
     if (UserIdentity.Current.PrefersSerial) 
      serial.Send(data); 
     else 
      wireless.Send(data); 
    } 
} 

は（OCPを破壊を防止することができますが、それは簡単に工場を用いて固定することができます作成​​します。 ）。

更新

あなたはそれと間違って何を知っていますか？私はUIにデータを送信する方法をユーザーが選択できるようにしたいと考えています。送信方法として、赤外線経由で送信する方法や、さまざまな方法の中からユーザーに選択させることによって、OCPに違反するユーザーのUIにifステートメントを持たせる必要があります。発送のすべての新しい種類が新しいのif/else条件に

を持つために私を強制しますので私のアプローチは、代わりにISendOutインタフェースが使用されているすべての単一の場所で、一つだけのクラスにOCPの違反を移動します。

私も工場について言及しました。私は工場のパターン（抽象的な工場または工場のどちらの方法も意味しません）を意味します。コンフィグレーション文字列と具象クラスをマップし、UserConfiguredCommunicationModule内のそのファクトリを使用して、適切なISendOut実装を作成することができます。

UserConfiguredCommunicationModule内のサービスロケータパターンを使用して、正しい実装を解決することもできます。

その点はあなたが選択したものであるにもかかわらず、選択プロセスをカプセル化するのに似たクラスUserConfiguredCommunicationModuleが必要です。