Lấy nguyên tắc đóng mở trong quá khứ

Question

Tôi có một chương trình đơn giản vẽ số liệu hình học dựa trên dữ liệu chuột do người dùng cung cấp. Tôi có một lớp học xử lý theo dõi chuột (nó nhận được Danh sách có lịch sử di chuyển chuột) và một lớp học trừu tượng có tên là Hình dạng là . Từ lớp này, tôi nhận được một số hình dạng bổ sung, như Circle, Rectangle, vv - và mỗi một trong số chúng sẽ ghi đè lên hàm Draw() trừu tượng.

Tất cả đều hoạt động độc đáo, nhưng sự cố xảy ra khi tôi muốn người dùng có thể chuyển đổi hình dạng mong muốn Hình theo cách thủ công. Tôi nhận được dữ liệu chuột và tôi biết tôi nên vẽ hình dạng nào. Vấn đề là làm thế nào để làm cho mã để "biết" mà đối tượng nó nên tạo và vượt qua thông số appropiate cho constructor. Cũng không thể vào thời điểm này để thêm các dẫn xuất hình dạng mới, mà là sai lầm một cách không đúng.

tôi obiously không muốn đi ra với mã như:

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. 

Đoạn mã trên rõ ràng vilates Open-Closed Nguyên tắc. Vấn đề là tôi không có bất kỳ ý tưởng tốt nào để vượt qua nó. Vấn đề chính là các hình dạng khác nhau có các hàm tạo với các tham số khác nhau, khiến cho nó trở nên rắc rối hơn nhiều.

Tôi khá chắc chắn rằng đây là một vấn đề phổ biến, nhưng tôi không biết làm thế nào để vượt qua nó. Bạn có ý tưởng không?

Answer 1

Nó cầu xin một nhà máy chứ không chỉ nhà máy mà nhà máy có công nhân tiêm.

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) { ... } 

} 

Cách này mã mở cho tiện ích - công nhân nhà máy mới có thể được tạo tự do và thêm vào nhà máy.

Answer 2

Khi bạn cần tạo các đối tượng có nguồn gốc từ một lớp đơn lẻ hoặc triển khai cùng một giao diện, một cách tiếp cận chung là sử dụng factory. Trong trường hợp của bạn, tuy nhiên, một nhà máy đơn giản có thể không đủ, bởi vì bản thân nhà máy cần phải được mở rộng.

Một cách để thực hiện nó là như sau:

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 
    } 
} 

Với những Maker lớp trong tay, bạn có thể làm cho một registry của nhà sản xuất (một đơn giản List<IShapeMaker>) đi qua các nhà sản xuất truyền cho họ những điểm, và dừng khi bạn lấy lại hình dạng không rỗng.

Hệ thống này còn có thể mở rộng, bởi vì bạn có thể thêm một cặp NewShape và NewShapeMaker, và "cắm chúng trong" vào khuôn khổ hiện có: một lần NewShapeMaker được trong registry, phần còn lại của hệ thống ngay lập tức trở nên sẵn sàng để nhận biết và sử dụng NewShape của bạn.