マイファクタをリファクタリングする：派生クラスでキャストすることを避ける

Question

まずタイトルを残念に思うが、私は自分の問題を正確に記述する方法を知らない。私はそれがコードを通してよりよく説明されることを願っています。

public abstract class AB { 
    public MyModel Model; 
} 

public class A : AB { 
    public A() { 
    Model = new MyModelA(); 
    } 

    public void AMethod() { 
    var model = (MyModelA) model; // I have to do this all place 
    } 

    public void AnotherMethod() { 
    var model = (MyModelA) model; // same here 
    model.NewInt = 123; 
    } 
} 

public abstract class MyModel { 

} 

public class MyModelA : MyModel { 
    // new properties 
    public int NewInt {get;set;} 
} 

派生クラスから新しいプロパティを使用するために、私はキャストをしなければならないが、私はすべての場所の上にそれを同じ時間を使用する必要がある場合、それは醜いです、コードを見てみましょう。

私が思う方法は、別のプロパティ：public MyModelA _tmpを宣言してから、それをコンストラクタ_tmp = (MyModelA) Modelにキャストし、Modelの代わりに使用します。

他にもこれを行う方法はありますか？ ありがとう！

Answer 1

あなたは、基本クラスは一般的なことができます：

public abstract class ServiceBase<TModel> where TModel : new() { 
    protected ServiceBase() { Model = new TModel(); } 
    public TModel Model { get; private set; } 
} 

public class AService : ServiceBase<MyModelA> { 
    ... 
} 

Answer 2

あなたは派生クラスで自分のモデルの参照を維持することができます。

public abstract class AB { 
    public MyModel Model; 
} 

public class A : AB { 
     MyModel MyModel; 

    public A() { 
      MyModel = new MyModelA(); 
      Model = MyModel; 
    } 

    public void AMethod() { 
      //just use MyModel 
    } 

    public void AnotherMethod() { 
    MyModel.NewInt = 123; 
    } 
} 

public abstract class MyModel { 

} 

public class MyModelA : MyModel { 
    // new properties 
    public int NewInt {get;set;} 
} 

Answer 3

_tmpと解決策は、そのマニュアルキャストを書くことであなたをのRID常に、しかし、奇妙なオブジェクトデザインの問題が残っています。

私はあなたのNewIntが、MyModelにも存在していた何らかの機能を実行していると思います（そうしないと、新しいクラスを作成する方が良いでしょう）。私はMyModelAが何も新しいものを公開する必要がないようにその機能をカプセル化できないのだろうかと思います。これは、そのような一般化を可能にするためにABの定義を変更することを意味するかもしれない。

答えは、ドメインを理解することなく構文的にもOOPパターンでも簡単に見つかりません。おそらくあなたはそれについていくつかの詳細を提供することができます。