過去のオープンクローズド原理の取得

Question

私は、ユーザーから提供されたマウスデータに基づいて幾何学的図形を描く簡単なプログラムを持っています。 私はマウスのトラッキング（マウスの移動履歴を持つリストを取得する）とシェイプと呼ばれる の1つのクラスを処理するクラスを持っています。このクラスからCircle、Rectangleなどの余分なShapesを導き出します。それらのすべてが抽象Draw（）関数をオーバーライドします。

すべてうまく動作しますが、ユーザーが希望の形状 を手動で切り替えることができるようにしたいときに問題が発生します。私はマウスのデータを取得し、私はどのような形を描くべきかを知っています。問題は、どのオブジェクトを作成するかを「知る」ようにコードを作成し、適切なパラメータをコンストラクタに渡す方法です。現時点では、形状の派生物を新たに追加することも不可能です。これは間違っています。私はobiouslyのようなコードで出てくるしたくない

：

List<Shape> Shapes = new List<Shape>(); 
// somwhere later 

if(CurrentShape == "polyline"){ 
    Shapes.Add(new Polyline(Points)); 
} 
else if (CurrentShape == "rectangle"){ 
    Shapes.Add(new Rectangle(BeginPoint, EndPoint)); 
} 
// and so on. 

明確に上記のコードはオープン・クローズ原理をvilates。問題は、私がそれを乗り越えるには良いアイデアがないということです。主な問題は、さまざまな図形 には異なるパラメータを持つコンストラクタがあることです。そのため、はるかに面倒です。

私はこれが一般的な問題だと確信していますが、それをどのように回避するのか分かりません。アイデアはありますか？

Answer 1

これは工場だけでなく、注射可能な労働者の工場を求めています。

public class Context { 
    public Point BeginPoint; 
    public Point EndPoint; 
    public List Points; 

    whatever else 
} 

public class ShapeFactory { 

    List<FactoryWorker> workers; 

    public Shape CreateShape(string ShapeName, Context context) 
    { 
     foreach (FactoryWorker worker in workers) 
     if (worker.Accepts(ShapeName)) 
      return worker.CreateShape(context); 
    } 

    public void AddWorker(FactoryWorker worker) { 
     workers.Add(worker); 
    } 
} 

public abstract class FactortWorker { 
    public abstract bool Accepts(string ShapeName); 
    puboic Shape CreateShape(Context context); 
} 

public class PolyLineFactoryWorker : FactoryWorker { 

    public override bool Accepts(string ShapeName) { 
     return ShapeName == "polyline"; 
    } 

    public Shape CreateShape(Context context) { ... } 

} 

このようにしてコードを拡張することができます。新しい工場労働者は自由に作成して工場に追加できます。

Answer 2

すべてのオブジェクトを単一のクラスから派生させるか、同じインターフェイスを実装する必要がある場合は、一般的な方法の1つはfactoryです。しかし、あなたのケースでは、工場自体が拡張可能でなければならないため、単純な工場では十分ではないかもしれません。

interface IShapeMaker { 
    IShape Make(IList<Point> points); 
} 
class RectMaker : IShapeMaker { 
    public Make(IList<Point> points) { 
     // Check if the points are good to make a rectangle 
     ... 
     if (pointsAreGoodForRectangle) { 
      return new Rectangle(...); 
     } 
     return null; // Cannot make a rectangle 
    } 
} 
class PolylineMaker : IShapeMaker { 
    public Make(IList<Point> points) { 
     // Check if the points are good to make a polyline 
     ... 
     if (pointsAreGoodForPolyline) { 
      return new Polyline(...); 
     } 
     return null; // Cannot make a polyline 
    } 
} 

を手にこれらのMakerのクラスを使用すると、メーカーのレジストリ（シンプルList<IShapeMaker>は）彼らに点を通るメーカーを通過させると、停止することができます：それを実装するために

一つの方法は、以下の通りです。あなたがヌル以外の形を返すとき。

あなたはNewShapeとNewShapeMakerのペアを追加し、既存のフレームワークに「でそれらをプラグイン」することができますので、このシステムは、拡張可能なまま：NewShapeMakerがレジストリに入った後は、システムの残りの部分は瞬時に認識して使用することが可能となりますあなたのNewShape。