从其他对象创建新对象时 instanceOf 的问题

Question

我有一个类负责从Shape 对象创建Formation 对象。形状正如其名，在画布上绘制的形状（TriangleShape、RectangleShape 等）。

形态类似于形状，但我计划以不同的方式使用它们。

例如，RectangleShape 如下所示：

public class RectangleShape extends Shape {

public RectangleShape() {
    this(0, 0, 0, 0);
}

public RectangleShape(int x, int y, int width, int height) {
    this.x = x;
    this.y = y;
    this.width = width;
    this.height = height;

    this.nPoints = 4;
}

@Override
public void drawShape(Graphics2D g) {
    Color color = g.getColor();
    fillShape(g, new Color(g.getColor().getRed(), g.getColor().getGreen(), g.getColor().getBlue(), 16));
    g.setColor(color);

    g.drawLine(x, y, x + width, y);
    g.drawLine(x, y, x, y + height);
    g.drawLine(x, y + height, x + width, y + height);
    g.drawLine(x + width, y, x + width, y + height);
}

@Override
public String toString() {
    return "Rectangle";
}

@Override
public Shape createCopy() {
    return new RectangleShape();
}

@Override
public void fillShape(Graphics2D g) {
    xPoints = new int[] {
        x,
        x,
        x + width,
        x + width
    };

    yPoints = new int[] {
        y,
        y + height,
        y + height,
        y
    };

    g.fillPolygon(xPoints, yPoints, nPoints);
}
}

我保留了一份声明为List<Shape> = new ArrayList<>(); 的所有绘制形状的列表。

当我需要从一个形状动态创建一个队形时，我的问题就出现了。 第一种方法是创建一个具有如下方法的类：

public static TriangleFormation createFormationFrom(TriangleShape shape) {
    // my code here
}

public static RectangleFormation createFormationFrom(RectangleShape shape) {
    // my code here
}

public static PentagonFormation createFormationFrom(PentagonShape shape) {
    // my code here
}

public static HexagonFormation createFormationFrom(HexagonShape shape) {
    // my code here
}

public static OvalFormation createFormationFrom(OvalShape shape) {
    // my code here
}

问题是当我从列表中检索形状时，它的类型为Shape，如果不将形状向下转换为适当的类，我就无法调用任何这些方法，这会引发使用@987654330 的问题@运算符。

我是否应该将 Shape 和 Formation 合并到一个类中，我是否应该尝试实现一个访问者模式（如果是，在这种情况下将如何实现）或者还有其他我没有想到的东西？

Answer 1

这完全取决于您希望Shapes 和Formations 的分离程度。正如您所提到的，最简单的解决方案是向Shape 接口添加一个Formation createFormation() 方法。

但如果您想将Shapes 和Formations 分开，您将不得不做一些更高级的事情。我建议不要使用访问者模式，因为它非常不灵活。

作为替代方案，考虑添加一个FormationBuilder 类：

public interface FormationBuilder
{
    /**
     * Builds a particular Formation implementation from the given shape.
     * Perhaps the shape is passed into the constructor of the Formation,
     * perhaps this method extracts the necessary information to build the
     * Formation...
     */
    Formation build(Shape shape);
}

这可以与这样的工厂一起使用：

public class FormationFactory
{
    private final Map<Class<? extends Shape>, FormationBuilder> builders
            = new HashMap<Class<? extends Shape>, FormationBuilder>();

    public <T extends Shape> void register(
            Class<T> shapeType, FormationBuilder builder);
    {
        builders.put(shapeType, builder);
    }

    public Formation getFormation(Shape shape)
    {
        return builders.get(shape.getClass()).build(shape);
    }
}

但现在的问题是，Factory 应该在哪里初始化。这是否符合您的需求取决于您的代码结构。