抽象工廠模式 - 未使用的代碼

Question

我正在學習設計模式並在HERE中遇到非常奇怪的示例。如果我們有一個類：

public abstract class AbstractFactory { 
abstract Color getColor(String color); 
abstract Shape getShape(String shape) ; 
} 

正如我們可以看到的，有2種類型的方法創建對象：顏色和形狀。這是一個抽象類，所以我們必須建立具體落實這一點，所以讓我們假設我們有：

public class ShapeFactory extends AbstractFactory { 

@Override 
public Shape getShape(String shapeType){ 

// I skip implementation to keep post brief 

} 

@Override 
Color getColor(String color) { 
    return null; // It's useless method in this class! 
} 
} 

和第二實施：

public class ColorFactory extends AbstractFactory { 

@Override 
public Shape getShape(String shapeType){ 
    return null; // It's useless method in this class! 
} 

@Override 
Color getColor(String color) { 

// I skip implementation to keep post brief 

} 
} 

在這裏，我的問題是，在這兩種情況下（具體工廠）有一種完全沒用的方法，並且不存在，但是當我們創建AbstractFactory類時，我們必須實現這兩種方法。在編程中在不需要它的類中創建無用方法不是不好的習慣嗎？應該以其他方式完成，而不是像網站建議的那樣？

Answer 1

@ Michael213 - 您的具體實現不正確。當然，他們不遵循抽象工廠模式。抽象工廠談論產品的家庭。抽象工廠樣本（使用我的假設）看起來像下面的代碼。你的例子只使用一種方法將被濫用模式，並很快就會中斷。

我已經回答類似的問題，請看看到也What are the real benefits of using the Abstract Factory in the following example, instead of the factory method?

public abstract class AbstractFactory { 
     abstract Color getColor(String color); 
     abstract Shape getShape(String shape) ; 
    } 
    /** 
    * CONCRETE FACTORY1 
    */ 
    class HighResolutionFactory extends AbstractFactory{ 
     Color getColor(String color){ 
      return new HighResolutionColor(); 
     } 
     Shape getShape(String shape){ 
      return new HighResolutionShape(); 
     } 
    } 

    /** 
    * CONCRETE FACTORY2 
    */ 
    class LowResolutionFactory extends AbstractFactory{ 
     Color getColor(String color){ 
      return new LowResolutionColor(); 
     } 
     Shape getShape(String shape){ 
      return new LowResolutionShape(); 
     } 
    } 
    class Color{} // ABSTRACT PRODUCT 1 
    class Shape{} // ABSTRACT PRODUCT 2 
    class HighResolutionColor extends Color{}// CONCRETE PRODUCT1 FACT 1 
    class HighResolutionShape extends Shape{}// CONCRETE PRODUCT2 FACT 1 
    class LowResolutionColor extends Color{}//... 
    class LowResolutionShape extends Shape{} 

Answer 2

是的，該教程似乎並不是最好的方面。 雖然它仍然被視爲工廠設計模式，但並不理想。

Answer 3

AbstractFactory錯誤。你不必考慮製造不同物體的工廠。爲每種不同的類型製造單獨的工廠是正確的。

public interface AbstractColorFactory { 
    public Color getColor(String color); 
} 

public interface AbstractShapeFactory { 
    public Shape getShape(String shape); 
} 

public class ColorFactory implements AbstractColorFactory { 
    public Color getColor(String color) { 
     return .... 
    } 
} 

public class ShapeFactory implements AbstractShapeFactory { 
    public Shape getShape(String shape) { 
     return .... 
    } 
}