如何改进这个内部枚举代码？

Question

我有这样的构造：

public class Constants {
 enum SystemA implements Marker {
    ConstOne(1), ConstTwo(2), ConstThree(3);
    SystemA(int i)
   {
      number = i;
   }
   int number;
 }

  enum SystemB implements Marker {
    ConstFour(4), ConstFive(5), ConstSix(6);
  SystemB(int i)
   {
      number =i;
   } 
   int number;
 }
}

我有 Marker 所以我可以传递给这样的方法： method(Constants.SystemA) 或 method(Constants.SystemB)

列出所有枚举值的最佳方法是什么？我还想确保它不会重复任何枚举中的数字。

Answer 1

如果两个枚举的约定相同，则可以将其合并为一个枚举。
但是，为了区分这两个系统，您可以有一个单独的“SystemType”枚举，其中包含 A、B...，并且您可以为 System 枚举中的每个常量分配一个 SystemType。

此外，您的 SystemType 枚举可以提供一个实例方法，该方法将返回给定类型的所有系统，以便您可以根据需要仅迭代该子集。

至于编号，如果序列像您的示例中一样简单，您可以使用枚举常量的 ordinal() 方法，以便您只需依赖常量声明顺序（可能添加一些数学）。我的意思是：

public enum SystemType {
    A, B;

    public Set<System> getSystems() {
        Set<System> systems = EnumSet.noneOf(System.class);
        for (System system : System.values()) {
            if (system.getType() == this) {
                systems.add(system);
            }
        }
        // you could also store this result
        // as an instance variable for caching.
        return systems; 
    }
}

public enum System {
    ConstOne(SystemType.A),
    ConstTwo(SystemType.A),
    ConstThree(SystemType.A),
    ConstFour(SystemType.B),
    ConstantFive(SystemType.B);

    private final SystemType systemType;

    private System(final SystemType systemType) {
        this.systemType = systemType;
    }

    public int getNumber() {
        return ordinal() + 1;
    }

    public SystemType getType() {
        return systemType;
    }

}

如果您确实需要为不同的系统类型定义不同的契约（仅对 SystemX 或其他有意义的方法），这当然是无用的。

附：
当然，System 是一个非常糟糕的名字，因为它隐藏了 java.lang.System 类。

Answer 2

由于您在代码中声明所有枚举，因此您有责任提供不重叠的数字（如果需要）。
更有趣的问题是：为什么需要这个？听起来好像您正在将枚举用于它们并非设计用途的东西。

Answer 3

我将依赖一个类 - 一个工厂类 - 来创建和控制系统类的实例。