Java Switch Statememt - 使用抽象类的公共枚举所需的常量表达式

Question

我编写的代码使用了我在webockets上设计的协议,用于创建Android应用程序的项目.

我有一个使用Autobahn库处理websocket通信的类weCommunicationManager.

我有许多变量和标签,用于处理从我的服务器通过websockets到应用程序的收到的JSON消息.

为了使事物分离并在一个地方,我创建了一个抽象类ExICSProtocol,它将标记保存为公共静态成员,以便可以在必要时引用它们.

在收到的消息中是消息类型的整数值,我需要能够打开它以定义如何处理该特定的接收消息.

为此,我在ExICSProtocol类中实现了一个公共枚举,如下所示,

public static enum MESSAGE_TYPE{
    PROTOCOL_HANDSHAKE(0),
    USER_CONNECTED(1),
    USER_DISCONNECTED(2),
    SYSTEM_STATE(3),
    CHANGE_ROOM(4),
    EXAM_START(5),
    EXAM_PAUSE(6),
    EXAM_STOP(7),
    EXAM_XTIME(8),
    SEND_MESSAGE(9),
    SUCCESS(69),
    FAILURE(-1),
    TERMINATE_CONNECTION(-2);

    private int code;

    MESSAGE_TYPE(int code){
        this.code = code;
    }

    public int getCode(){
        return this.code;
    }
}

我试图在wsCommunicationManager代码中使用它,如下所示,

private static void handleMessage(String message){
    try{
        JSONObject messageObject = new JSONObject(message);
        JSONObject messageHeader = messageObject.getJSONObject(ExICSProtocol.TAG_HEADER);
        JSONObject messagePayload = messageObject.getJSONObject(ExICSProtocol.TAG_PAYLOAD);

        int messageType = messageHeader.getInt(ExICSProtocol.TAG_MESSAGE_TYPE);


        switch(messageType){
            case ExICSProtocol.MESSAGE_TYPE.PROTOCOL_HANDSHAKE.getCode():
                //DO SOMETHING HERE
                break;

            ...

            default:
                throw new ExICSException("Unknown Message Type Received");
                break;
        }
    }catch (JSONException e){
        Log.e(TAG, "Failed to parse received message " + message, e);
    }catch (ExICSException e){
        Log.e(TAG, "Excaption Handling Message Occurred", e);
    }
}

我收到错误标记,ExICSProtocol.MESSAGE_TYPE.PROTOCOL_HANDSHAKE.getCode():表示需要一个常量表达式.然而,这应该是不变的？

我尝试了一些不同的东西,将枚举移动到正在使用它的类中; 同样的问题.使enum公共持有私有int,以便可以直接访问; 同样的问题.

我已经看到很多在switch语句中使用枚举的例子,但非常像我.我在声明或初始化中遗漏了什么,或者我正在尝试做一些无效的事情.

我知道有一些相对简单的解决方法,例如将数字类型代码定义为公共静态最终整数,但是如果可能的话,我希望类型代码在MESSAGE_TYPE下保持在一起.

Answer 1

您可以将 getFromCode(int code) 静态方法添加到 MESSAGE_TYPE 枚举中，如下所示：

public static MESSAGE_TYPE getFromCode(int code) {
    for(MESSAGE_TYPE mt : MESSAGE_TYPE.values())
        if(mt.code == code)
            return mt;

    throw new IllegalArgumentException();
}

然后在你的handleMessage中：

MESSAGE_TYPE messageType = MESSAGE_TYPE.getByCode(messageHeader.getInt(ExICSProtocol.TAG_MESSAGE_TYPE));


switch(messageType){
    case PROTOCOL_HANDSHAKE:
        //handle handshake...

Answer 2

正如您所见,调用方法不算作常量表达式.

更清洁的解决方案(IMO)是在枚举中使用静态方法:

// Enum renamed to comply with conventions
public enum MessageType {

    private static final Map<Integer, MessageType> codeMap = new HashMap<>();
    // Can't do this in the constructor, as the codeMap variable won't have been
    // initialized
    static {
        for (MessageType type : MessageType.values()) {
            codeMap.put(type.getCode(), type);
        }
    }

    public static MessageType fromCode(int code) {
        return codeMap.get(code);
    }
}

然后在你的wsCommunicationManager代码中:

int messageCode = messageHeader.getInt(ExICSProtocol.TAG_MESSAGE_TYPE);
MessageType messageType = MessageType.fromCode(messageCode);
if (messageType == null) {
    // Unknown message code. Do whatever...
}

switch(messageType) {
    case MessageType.PROTOCOL_HANDSHAKE:
        ...
}

基本上,尽可能早地从"面向整数"转向"面向对象"的世界.这有点像早期解析基于数字和日期的用户输入而不是传递字符串 - 只是在你的情况下,代理表示是一个int.