枚举的toString方法和Java的良好做法

Question

我想请教一下在Java中良好的编码习惯。我想创造一些属性的enumeratoin并覆盖toString()下面的方式来使用它（JSF 1.2是为了用于检索本地化消息）：

package ua.com.winforce.casino.email.util; 

import java.text.MessageFormat; 
import java.util.Locale; 
import java.util.MissingResourceException; 
import java.util.ResourceBundle; 

import javax.faces.context.FacesContext; 

public enum StatisticItems { 
    MESSAGES, 
    VIEWS; 

    private static String BUNDLE_NAME = "messages"; 

    public String toString(){ 
     switch (this) { 
     case MESSAGES: 
      return getLocalizedMsg("messages.title"); 
     case VIEWS: 
      return getLocalizedMsg("views.title"); 
     default: 
      return null; 
     } 
    } 

    private static String getLocalizedMsg(String key, Object... arguments) { 
     Locale locale = FacesContext.getCurrentInstance().getViewRoot().getLocale(); 
     String resourceString; 
     try { 
      ResourceBundle bundle = ResourceBundle.getBundle(BUNDLE_NAME, locale); 
      resourceString = bundle.getString(key); 
     } catch (MissingResourceException e) { 
      return key; 
     } 

     if (arguments == null) { 
      return resourceString; 
     } 
     MessageFormat format = new MessageFormat(resourceString, locale); 
     return format.format(arguments); 
    } 
} 

我的问题是关于很好的做法。把所有这些方法放在enum定义中是否被认为是好的？如果没有，我想了解为什么，当然如何做得更好。

Answer 1

有两点需要在此进行：

1. 如果默认情况下（在返回代码为null）是一个运行时错误，然后用switch是一种容易出错的。还有，我认为较好的两个备选方案：

   1. 使用领域localizationKey，在枚举实例的构造函数初始化，并参考这个按键toString方法
   2. 或者，（对于更复杂的情况下），使toString摘要并强制每个实例通过适当的实现进行覆盖。例如，请参阅this question。

2. 很多人认为toString是用于非常明显的实现，否则只用于调试。 （详细请参阅this question）我的建议：提出一个更具描述性的方法名称，为方便起见，不要重复使用toString。

更新：从Java语义缩放了一点：该逻辑属于视图，而不是模型中，如在下面的评论所指出BalusC。

Answer 2

我将确保所有复杂的逻辑只在INITIALISE时间做一次。

public enum StatisticItems { 

    MESSAGES("messages"), 
    VIEWS("views"); 
    final String asString; 

    StatisticItems(String localised) { 
     asString = getLocalizedMsg(localised + ".title"); 
    } 

    @Override 
    public String toString() { 
     return asString; 
    } 

    private static String BUNDLE_NAME = "messages"; 
    private static final Locale locale = FacesContext.getCurrentInstance().getViewRoot().getLocale(); 
    private static final ResourceBundle bundle = ResourceBundle.getBundle(BUNDLE_NAME, locale); 

    private static String getLocalizedMsg(String key) { 
     String resourceString; 
     try { 
      resourceString = bundle.getString(key); 
     } catch (MissingResourceException e) { 
      return key; 
     } 

     if (arguments == null) { 
      return resourceString; 
     } 
     MessageFormat format = new MessageFormat(resourceString, locale); 
     return format.format(arguments); 
    } 
}