Java 枚举行为

Question

我们有以下枚举：

public enum ComponentTypes {

    PDIFF(301),
    TDIFF(302),
    TADJ(303);

    private long componentTypeId;

    private ComponentTypes(long componentTypeId){
        this.componentTypeId = componentTypeId;
    }

    public Long getId(){
        return this.componentTypeId;
    }
}

在我们的一个测试设置中，我们执行 c.setComponentTypeId(ComponentTypes.TADJ.getId()) 但当 c.getComponentTypeId() 被调用时测试它抛出 NullPointerException，但 c.setComponentTypeId(303L) 按预期工作。使用枚举来设置值时我缺少什么？

编辑

看起来@Tom 直接指出了长/长的不一致。现在 getId() 返回 long 而不是 Long 它按预期工作。

编辑

看起来我之前说的是错误的自动装箱确实按预期工作在我刷新系统jvm等之后没有问题 - 这对我来说没有任何意义！

Answer 1

你没有说“c”是什么类型，但我怀疑它的设置器没有按照你想象的那样做——NullPointerException表明自动拆箱出了问题。您枚举本身似乎没有问题，尽管当成员为 long 时返回 Long 是一种代码味道。

实际上，使用枚举的 ID 调用 c.setComponentTypeId() 是另一种代码味道。为什么不通过 c.setComponentType() 使用枚举本身？你这样做的方式几乎失去了枚举的所有价值。

Answer 2

您说在测试中调用了 setComponentId，但您没有包含此方法的源代码。您可以粘贴此内容，并展示测试中的“c”对象是如何创建的吗？

汤姆，如果他们使用支持自动装箱的最新 java 版本，那么 long/Long 肯定不重要吗？

Answer 3

你在哪里初始化c？我怀疑这就是问题所在，并且您收到的 NullPointerException 实际上来自 setComponentTypeId 调用而不是 getter。

使用您的枚举代码（无需修改）我运行了以下命令：

ComponentTypes c = ComponentTypes.PDIFF;
c.setComponentTypeId(ComponentTypes.TADJ.getId());
System.out.println(c.getComponentTypeId());

c.setComponentTypeId(303L);
System.out.println(c.getComponentTypeId());

我的输出：

303

303

然而；如果 c 未正确初始化，则当您第一次尝试调用 c.setComponentTypeId 时，您会收到 NullPointerException。