ベースタイプのみに基づいて派生オブジェクトを作成する方法は？例えば

Question

私はこのクラスを持っている：

public abstract class Device 
{ 
    public Device() 
    { 
    } 

    public Device(XmlDevice xmlDevice) 
    { 
     // creating device by xmlDevice 
    } 
} 

public class Modem : Device 
{ 
    public Modem() 
    { 
    } 

    public Modem(XmlModem modem) 
    { 
     // creating modem by xmlModem 
    } 
} 

public class Plc : Device 
{ 
    public Plc() 
    { 
    } 

    public Plc(XmlPlc plc) 
    { 
     // creating plc by xmlPlc 
    } 
} 


public abstract class XmlDevice 
{ 
    public XmlDevice() 
    { 
    } 
} 

public class XmlModem : XmlDevice 
{ 
    public XmlModem() 
    { 
    } 
} 

public class XmlPlc : XmlDevice 
{ 
    public XmlPlc() 
    { 
    } 
} 

私はXmlDeviceオブジェクトのリストを持っている - どのように私はそのタイプに基づいてデバイスオブジェクトを作成することができますか？私はこのようにすることができます：

foreach(xmlDevice in xmlDevicesList) 
{ 
    if(xmlDevice is XmlModem) 
     devicesList.Add(new Modem((XmlModem)xmlDevice)); 
    else if(xmlDevice is XmlPlc) 
     devicesList.Add(new Plc((XmlPlc)xmlDevice)); 
} 

しかし、私は "is"ステートメントとキャストオブジェクトを避けたいと思います。これはもっとクリーンな方法ですか？

Answer 1

あなたの現在の実装に対するあなたの嫌悪感は十分に確立されています。私はあなたが新しいデバイスタイプを追加するとすぐに中断することを約束できます。さらに悪いことに、コンパイラは、インスタンス型チェックが網羅的であることを意図していることを知る方法がないため、エラーを知らせることによってあなたを助けません。あなたがXml*クラスを変更することができる場合

、あなたは単に派生クラスでCreateDeviceXmlDeviceで抽象メソッドを導入し、それをオーバーライドすることができます：

devicesList = xmlDevicesList.ConvertAll(x => x.CreateDevice()); 

：

public abstract class XmlDevice 
{ 
    // ... 
    public abstract Device CreateDevice(); 
    // ... 
} 

public class XmlModem : XmlDevice 
{ 
    // ... 
    public override Device CreateDevice() 
    { 
     return new Modem(this); 
    } 
    // ... 
} 

は今、あなたは、単にこれを行うことができます

おそらく、この方法の最大の利点は、CreateDeviceメソッドを実装せずにコンパイラが新しいデバイスタイプを作成することを免れないことです。