Thiết kế một lớp theo cách mà nó không trở thành một "đối tượng Thiên Chúa"

Question

Tôi đang thiết kế một ứng dụng cho phép tôi vẽ một số chức năng trên đồ họa. Mỗi chức năng sẽ được rút ra từ một tập hợp các điểm mà tôi sẽ chuyển đến lớp đồ họa này.

Có nhiều loại điểm khác nhau, tất cả đều được kế thừa từ lớp MyPoint. Đối với một số loại điểm nó sẽ chỉ in chúng trên màn hình như chúng, những người khác có thể bị bỏ qua, những người khác được thêm vào, do đó, có một số loại logic liên quan đến họ có thể trở nên phức tạp.

Cách vẽ đồ họa thực sự không phải là vấn đề chính ở đây. Điều làm phiền tôi là làm thế nào để làm cho logic mã như vậy mà lớp GraphicMaker này không trở thành cái gọi là Thiên Chúa-Object.

Nó sẽ là dễ dàng để làm một cái gì đó như thế này:

class GraphicMaker { 
    ArrayList<Point> points = new ArrayList<Point>(); 

    public void AddPoint(Point point) { 
     points.add(point); 
    } 

    public void DoDrawing() { 
     foreach (Point point in points) { 
      if (point is PointA) { 
       //some logic here 
      else if (point is PointXYZ) { 
       //...etc 
      } 
     } 
    } 
} 

Làm thế nào bạn sẽ làm điều gì đó như thế này? Tôi có cảm giác cách chính xác sẽ là đặt logic vẽ lên từng đối tượng Điểm (vì vậy mỗi lớp con từ Point sẽ biết cách vẽ chính nó) nhưng hai vấn đề phát sinh:

1. Sẽ có các loại điểm cần phải biết tất cả các điểm khác tồn tại trong lớp GraphicObject để biết cách tự vẽ.
2. Tôi có thể tạo ra rất nhiều phương pháp/thuộc tính từ lớp đồ họa công khai, để tất cả các điểm có tham chiếu đến lớp Đồ họa và có thể tạo ra tất cả logic của chúng theo ý muốn, nhưng không phải là giá lớn trả tiền vì không muốn có một lớp Thiên Chúa?

Answer 1

tôi sẽ làm như bạn đề nghị và làm cho mỗi điểm chịu trách nhiệm cho việc vẽ bản thân, vượt qua hàng loạt các điểm khác với nó:

interface ICanDraw { 
    void Draw(ArrayList<Point> allPoints); 
} 

public abstract class Point : ICanDraw { 
    ... 
} 

public PoniePoint : Point { 
    public void Draw(ArrayList<Point> allPoints) { 
     // do drawing logic here 
    } 
} 

Đối GraphicMaker của bạn:

public void DoDrawing() { 
    foreach (Point point in points) { 
     point.Draw(points); 
    } 
} 

(My Java là một chút gỉ, vì vậy mà có thể không 100% cú pháp chính xác Java nhưng tôi nghĩ rằng nó chuyển tải đề nghị của tôi).

Answer 2

Bạn đúng là mỗi phân lớp của điểm phải có phương thức vẽ riêng sẽ ghi đè phương thức vẽ trong lớp điểm cơ sở.

Phương thức vẽ phải có tham chiếu đến đối tượng đồ họa, cần có phương pháp/thuộc tính công khai cho bất kỳ thứ gì cần được sử dụng trong phương pháp vẽ điểm, bao gồm danh sách các điểm nếu đó là một trong những điều các phương pháp vẽ cần.

Tại sao bạn quan tâm đến việc tạo phương pháp công khai trên lớp đồ họa? Một khi mã phát triển, một vài phương thức hiển thị thêm sẽ khó hiểu hơn rất nhiều so với một phương thức khổng lồ làm mọi thứ.