这是一个糟糕的oop设计吗？

想象一下,我有一个名为IVehicle的界面.

从这个界面,我推导出几种具体的类型,如公共汽车和汽车(都可以移动,减速,关闭发动机等).我的界面没有字段.

拥有一个具有字段的类(例如车辆的最高速度)并使用每种具体类型的设计是不是很糟糕？这会是糟糕的设计吗？另外,如果我在这个类中存储状态(例如使用字段),那么它必须是静态的吗？

谢谢

你在谈论一个抽象类,不,它是不错的设计.这就是他们的目标(基本上,接口,但有一些基本的实现).

国家不必静态存储.每次创建Car,Bus等时,都会有一个完整的实现(以其中一个派生类的形式).

我会将TopSpeed接口添加为如下属性:

interface IVehicle
{
    void Move();
    void SlowDown();
    void SwitchOffEngine();

    Int32 TopSpeed { get; }
}

然后在具体实体中实现接口,如下所示:

class Car : IVehicle
{
    public void Move() { }
    public void SlowDown() { }
    public void SwitchOffEngine() { }

    public int TopSpeed
    {
        get { return 120; }
    }
}

class Bus : IVehicle
{
    public void Move() { }
    public void SlowDown() { }
    public void SwitchOffEngine() { }

    public int TopSpeed
    {
        get { return 98; }
    }
}

我认为这将是最好的方法,不需要基类.

我认为这是最好的方法,因为它TopSpeed是一个特定于实现的属性,在基类中没有任何意义.由于基类需要任何实现设置此值,我认为最好更改接口以包含它.

在你的情况下,你没有打破OO-

IVehicle定义"如果你是一辆车你应该做什么"

AbstractVehicle定义"如果你是一辆车,这是你应该做的最小值,这就是你要做的事情"

ConcreteVehicle定义"我是一辆车,这是我做事的方式,因为我是一辆车,我应该做的事情"

您不需要将所有属性都设置为静态.最高速度或最高速度之类的东西可以保持静止,但你可以将它们保持为常量,即你的通话.