1

我想知道我是否走对了路，因为我觉得下面的代码是错误的。对不起，我不知道如何正确命名这个问题。

我有一个特定的 ShapeEntity 类，用于从数据库加载数据。Shape 还有其他具体的类（我将来可以有很多）所以我想使用 LSP 来绘制这些形状，这就是我使用 IShape 抽象的原因。我使用 ShapeEntity 提供的 DB 信息来实例化具体的形状对象。

所以我关心的是 Main() 函数，我只使用简单的 if-else 创建这些形状。这是使用 if-else 块创建“未知”对象的正确方法吗？也许我可以为某种 ShapeService 创建 Shape 对象？换个方式怎么解决？

public class ShapeEntity
{
    int idShape { get; set; }
}

public interface IShape
{
    void Draw();
}

public class Square : IShape
{
    public void Draw() { }
}

public class Rectangle : IShape
{
    public void Draw() { }
}

public class Canvas()
{
    public static void Main()
    {
        List<IShape> Shapes = new List<IShape>();

        foreach(ShapeEntity ShapeItem in ShapeRepository.GetAll())
        {
            if(ShapeItem.idShape == 1)
            {
                Shapes.Add(new Square());
            }
            else if(ShapeItem.idShape == 2)
            {
                Shapes.Add(new Rectangle());
            }
        }
    }

    public void DrawShapesOnCanvas(IList<IShape> Shapes)
    {
        foreach(IShape Shape in Shapes)
        {
            Shape.Draw();
        }
    }
}
4

1 回答 1

5

你应该考虑使用Factory模式，而不是使用Id你应该使用enum

例子：

 public class ShapeFactory
    {
        public static IShape GetShape(ShapeType shapeType)
        {
            switch (shapeType)
            {
                case ShapeType.Square:
                    return new Square();

                case ShapeType.Rectangle:
                    return new Rectangle();
                default:
                    break;
            }

            return null;
        }
    }

    public enum ShapeType
    {
        Square,
        Rectangle
    }
于 2012-07-29T22:02:19.060 回答