"Tanrı nesnesi" haline gelmeyecek şekilde bir sınıf tasarlama

Question

Grafik üzerinde bazı işlevler çizmeme izin verecek bir uygulama tasarlıyorum. Her fonksiyon, bu grafik sınıfına geçeceğim bir dizi noktadan çizilecektir.

Bir MyPoint sınıfından miras alınan farklı türlerde noktalar vardır. Bazı tür noktalar için, onları ekranda olduğu gibi basacak, diğerleri göz ardı edilebilecek, diğerleri ekleyebilecek, böylece karmaşık olabilecek karmaşık bir mantık var.

Grafik nasıl çizilir, burada ana sorun değildir. Beni rahatsız eden şey, bu mantıksal mantığın nasıl yapılacağı, bu GraphicMaker sınıfının Tanrı-Nesnesi olarak adlandırılmamasıdır. Böyle bir şey yapmak kolay olurdu

:

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

Nasıl böyle bir şey yapsın? Ben her noktası bir nesne üzerinde çizim mantığı koymak olacağını doğru yol his var (şimdiye Point dan her çocuk sınıf kendisini nasıl çizileceğini bilemez) ancak iki sorunlar ortaya:

1. puan türü vardır olacağını kendilerini çekmeyi öğrenmek için GraphicObject sınıfında bulunan diğer tüm noktaları bilmeniz gerekir.
2. Grafik sınıfındaki birçok yöntem/özellikten yararlanabiliyorum, böylece tüm noktaların Grafik sınıfına bir referansı vardır ve tüm mantıklarını istedikleri gibi yapabilirler, ancak bu büyük bir fiyat değildir. Tanrı sınıfına sahip olmak istemediğin için öde mi?

Answer 1

Sana Önerilen yapmak ve ona diğer noktaların dizi geçen kendini çizmek için her nokta sorumlu kılacak

:

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

sizin GraphicMaker için:

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

(Benim Java biraz paslı, bu yüzden% 100 sözdizimi Java'yı düzeltmeyebilir, ancak önerimi ilettiğini düşünüyorum).

Answer 2

Noktanın her bir alt sınıfının, temel Point sınıfındaki birini geçersiz kılan kendi çizim yöntemine sahip olması doğrudur.

Çizim yöntemi, bazı noktalardan biri olan noktaların listesi de dahil olmak üzere, nokta çizim yöntemlerinde kullanılması gereken her şey için ortak yöntemleri/özellikleri olan grafik nesnesine bir başvuru almalıdır. çizim yöntemleri gerekir.

Grafik sınıfında genel yöntemler oluşturma konusunda neden endişe ediyorsunuz? Kod büyüdükçe, birkaç ekstra görünür yöntem, her şeyi yapan çok büyük bir yöntemden çok daha az kafa karıştırıcıdır.