共通フィールドを基本クラスに移動するタイミングは？

現在、2つの派生クラスがAあり、B、両方に共通のフィールドがあり、基本クラスに入れるかどうかを判断しようとしています。

基本クラスから参照されることはありません。また、道路のある時点で別のクラスが派生した場合C、を持たない場合、_field1「最低特権」（または何か）のプリンシパルは違反されませんだった？

public abstract class Base
{
    // Should _field1 be brought up to Base?
    //protected int Field1 { get; set; }
}

public class A : Base
{
    private int _field1;
}

public class B : Base
{
    private int _field1;
}

public class C : Base
{
    // Doesn't have/reference _field1
}

それはすべて、あなたが解決しようとしている正確な問題に依存します。

具体的な例を考えてみましょう。抽象基本クラスがVehicleあり、現在、具体的な実装BicycleとがありCarます。あなたは移動を考えているnumberOfWheelsからBicycleとCar車両に。これをする必要がありますか？番号！すべての車両に車輪があるわけではないからです。Boatクラスを追加しようとすると、クラスが壊れるということを既に伝えることができます。

さて、もしあなたの抽象基底クラスがそうだったなら、そこにメンバー変数WheeledVehicleを持つことは論理的numberOfWheelsです。

ご覧のとおり、単純なyesまたはnoの答えではないため、同じロジックを問題に適用する必要があります。

論理的に言えば、サブクラスで複製されたフィールドと基本クラスで共通のフィールドを配置する以外に、3つのオプションがあります。2つの間で共通のプロパティを持つ階層に新しいサブクラスを導入することです。@Peteは完全にそこに行くことなくこれをほのめかします。

@Peteの例を使用して、元の基本クラスから派生するWheeled Vehicleの（場合によっては抽象的な）サブクラスを導入しますが、2つのサブクラスはそこから派生します。したがって、元の基本クラスはホイールで汚染されていませんが、ホイールの共通性はDRYです（ホイールを持つサブクラス間で繰り返されません）。

もちろん、これはあなたの目的にとってはやり過ぎかもしれませんが、クラス階層メカニズムによってサポートされています。

ここで悪魔の擁護者を演じます。

今は何もしないでください。

乾燥していますか？いいえ。しかし、後で簡単に元に戻せない時期尚早な抽象化よりも、少し重複させる方が良いでしょう。プロパティを共通の基本クラスに移動するリファクタリングは簡単です。反対に行くことはありません。成り行きを見守る。

この種の決定をするとき、私は「3の規則」を使用する傾向があります。たとえば、3つの異なる場所で同じことを繰り返した後、それだけでチェーンを上に移動することを検討します。注意：あなたは2歳です。

一般的に、基本クラスに移動します。ここには「はい/いいえ」の目標はないと思います。なぜなら、ここにはトレードオフがあるからです。未使用のフィールドを運ぶことと複雑さを減らすことです。

私は通常、共有される可能性のあるものをすべて含む「重い」基本クラスを好みます。すべての派生クラスで子孫のシリアル化メソッドが必要ないため、これによりファイルへのシリアル化が簡単になります。ただし、そのような問題や同様の問題がない場合、またはメモリ使用量を削減するためにできる限りのことをする必要がある場合は、必要なフィールドのみを保持することで問題ありません。

フィールドの数が非常に限られている場合、共通フィールドを導入する「中間」クラスは問題ありません。ただし、さまざまな組み合わせで多数のフィールドを使用すると、アプローチによって複雑さが大幅に増加し、それぞれが特定のフィールドセットを導入する多くの中間クラスにつながることに注意してください。これはメンテナンスの問題になる可能性があります。