Java枚举给出不正确的值[重复]

Question

在长期使用 C、C++ 和 C# 之后，我最近开始使用 Java。我无法理解 Java 枚举应该如何工作。 经过一些研究，我创建了以下内容：

public class RedRoad implements Serializable, Parcelable
{
 // ... other parts removed for clarity
  public enum State
  {
    NOT_STARTED(0),
    PART_DONE(1),
    COMPLETED(2);
    private int value;
    private State(int value) {
        this.value = value;
    }
    public void setValue(int value) {
        this.value = value;
    }
    public int getValue() {
        return value;
    }
  }
}

我使用 getValue() 将这些 State 值作为整数存储在 sqlite 数据库中，使用 setValue() 检索它们，然后通过广播发送生成的“Road”对象。然后我这样做：

    switch (road.state) {
                case COMPLETED:
                    pline.getOutlinePaint().setColor(Color.GREEN); break;
                case PART_DONE:
                    pline.getOutlinePaint().setColor(Color.argb(0xFF,0xFF,0xA5,0x00)); break;
                case NOT_STARTED:
                    default: pline.getOutlinePaint().setColor(Color.RED);
            }

但前两种情况永远不会被调用，即使我已经检查过 road.state.getValue() 有时是 1 ，而不是零。 此外，如果我将切换代码更改为：

        switch (road.state.getValue()) {
                case 2:
                    pline.getOutlinePaint().setColor(Color.GREEN); break;
                case 1:
                    Log.d("*** road state ", String.valueOf(road.state.getValue()));
                    Log.e("*** road state ", String.valueOf(road.state));
                    pline.getOutlinePaint().setColor(Color.argb(0xFF,0xFF,0xA5,0x00));
                    break;
                case 0:
                default:
                    pline.getOutlinePaint().setColor(Color.RED);
            }

然后根据要求设置颜色。我在日志中得到了一个非凡的输出：

D/*** road state: 1
E/*** road state: NOT_STARTED

这怎么可能？ NOT_STARTED 被定义为零！

[编辑] 后来发现，除了上述之外（我可以找到解决方法），从异步任务发送到主要活动后，值不正确（'Road'类是可打包的）。如果我在发送之前记录 state.getValue() ，然后在接收之后再次记录，任何非零值都已更改为零。

Answer 1

您在第二个实例中对枚举常量调用String.valueOf，它调用枚举的toString 方法。枚举的默认toString 实现返回name 字段，这使得NOT_STARTED 成为有效的返回值。

枚举本质上是具有有限实现数量的抽象类，每个实现保证每个 JVM 只存在一次。

Answer 2

@AndyTurner 关于不使用 setter 是正确的。我现在在一个专门创建的新应用程序中做了更多的测试，setter 给出了完全不可预测的结果。 所以我使用的解决方案（用于从数据库中的“int”设置值）有点无聊

protected void setState(int i) {
    switch (i)
    {
        case 2:     state = State.COMPLETED;     break;
        case 1:     state = State.PART_DONE;     break;
        default:    state = State.NOT_STARTED;
    }
}

虽然这可行，但在我看来，部分原因是使用 Enum（而不是 int）开始。 而且，由于创建设置器（我最初的）给出了不正确的结果，我想知道为什么 Android Studio（4.1）没有给我一个警告？很高兴就这么多其他（不太重要）的事情向我发出警告。