java - 如何避免重复？

Question

<分区>

Answer 1

当我看到枚举作为 switch 语句或 if-else 阶梯中的条件，并且编写控制语句的人拥有该枚举时，我认为这是一种代码味道。

考虑这个替代方案，而不是打开枚举，为什么不使用你在其他地方做的同样的事情呢？我建议你在这里试试多态性。

首先，如果您没有对枚举有意义的默认实现，请使用一些抽象方法定义您的方向枚举。这与下面示例中的 getWidth() 相同。

其次，添加任何您可能需要的具有合理默认值的非抽象方法。然后，您可以像在普通 Java 多态性中那样覆盖这些方法。这与下面示例中的 getHeight() 相同。

这种方法有很多优点。

1. 如果您想添加更多方向(假设您从 4 个方向变成了 8 个方向，添加了左上、右上、右下、左下)，那么您不必遍历整个代码基础并添加其他 if-else 语句。您只需在枚举中为这些新方向添加特定于实现的代码即可。一切正常。
2. 添加默认行为非常简单，您不必在代码库的多个位置保持同步。
3. 您的调用代码中的代码行会随着您拥有的枚举数的增加而减少。如果您有 10 个枚举，并且必须为每个枚举编写 10 次调用，那么您将有 121 行代码(计算调用和控制语句)，但是在您开始使用多态性之后，您将有 10 行代码(没有控制语句，只有方法调用)。这应该会降低您的圈复杂度。
4. 您清楚地将枚举的功能封装在枚举本身中。在这里你必须要小心一点，如果你对这种方法发疯，枚举最终可能会了解系统的太多部分，但通常我发现这不是问题。

方向枚举

public enum Direction {
    UP {
        public int getWidth() {
            return 50;
        }
        @Override
        public int getHeight() {
            return 100;
        }
    },
    DOWN {
        public int getWidth() {
            return 30;
        }
    };

    public abstract int getWidth();
    public int getHeight() {
        return 10;
    }
}

现在你的调用代码看起来像这样。

box.setAsBox(direction.getWidth(), direction.getSize());
bodyDef.position.set(direction.getPositionVector());
body = gameWorld.world.createBody(direction.getBody());
body.setLinearVelocity(direction.getLinearVelocity());