JSF 自定义日期转换器 - 线程安全吗？

Question

我在 JSF 1.2 中创建了一个自定义 Converter 来转换 Date 对象。日期具有非常特殊的格式。我已经使用核心 Java SimpleDateFormat 类实现了我的转换器来进行转换，并使用下面代码注释中显示的格式化程序字符串。这一切都很好。

我的问题是关于线程安全的。 SimpleDateFormat API 文档声明它不是线程安全的。因此，我为转换器对象的每个实例创建了一个单独的日期格式对象实例。但是，我不确定这是否足够。我的 DateFormat 对象存储为 DTGDateConverter 的成员。

问题：两个线程是否会同时访问 JSF 中 Converter 对象的同一个实例？

如果答案是肯定的，那么我的 Converter 可能面临风险。

/**
 * <p>JSF Converter used to convert from java.util.Date to a string.
 * The SimpleDateFormat format used is: ddHHmm'Z'MMMyy.</p>
 * 
 * <p>Example: October 31st 2010 at 23:59 formats to 312359ZOCT10</p>
 * 
 * @author JTOUGH
 */
public class DTGDateConverter implements Converter {

    private static final Logger logger = 
        LoggerFactory.getLogger(DTGDateConverter.class);

    private static final String EMPTY_STRING = "";

    private static final DateFormat DTG_DATE_FORMAT = 
        MyFormatterUtilities.createDTGInstance();

    // The 'format' family of core Java classes are NOT thread-safe.
    // Each instance of this class needs its own DateFormat object or
    // runs the risk of two request threads accessing it at the same time.
    private final DateFormat df = (DateFormat)DTG_DATE_FORMAT.clone();

    @Override
    public Object getAsObject(
            FacesContext context, 
            UIComponent component, 
            String stringValue)
            throws ConverterException {
        Date date = null;
        // Prevent ParseException when an empty form field is submitted
        // for conversion
        if (stringValue == null || stringValue.equals(EMPTY_STRING)) {
            date = null;
        } else {
            try {
                date = df.parse(stringValue);
            } catch (ParseException e) {
                if (logger.isDebugEnabled()) {
                    logger.debug("Unable to convert string to Date object", e);
                }
                date = null;
            }
        }
        return date;
    }

    @Override
    public String getAsString(
            FacesContext context, 
            UIComponent component, 
            Object objectValue)
            throws ConverterException {
        if (objectValue == null) {
            return null;
        } else if (!(objectValue instanceof Date)) {
            throw new IllegalArgumentException(
                "objectValue is not a Date object");
        } else {
            // Use 'toUpperCase()' to fix mixed case string returned
            // from 'MMM' portion of date format
            return df.format(objectValue).toUpperCase();
        }
    }

}

Answer 1

两个线程会同时访问 JSF 中 Converter 对象的同一个实例吗？

取决于您如何使用转换器。如果您使用

<h:inputWhatever>
    <f:converter converterId="converterId" />
</h:inputWhatever>

，那么将为视图中的每个输入元素创建一个新实例，这是线程安全的（除了非常罕见的边缘情况外，最终用户在同一会话的两个浏览器选项卡中拥有两个相同的视图，并同时在两种观点）。

但是，如果您使用

<h:inputWhatever converter="#{applicationBean.converter}" />

，则同一实例将在整个应用程序的所有视图之间共享，因此不是线程安全的。

然而，每次创建转换器时，您都会克隆一个静态 DataFormat 实例。该部分已经不是线程安全的。您可能会面临克隆实例的风险，而其内部状态会因为已在其他地方使用而发生更改。此外，克隆现有实例并不一定比创建新实例便宜。

我建议将其声明为线程本地（即在方法块内），无论您如何使用转换器。如果每次创建 DateFormat 的成本是一个主要问题（您是否分析过它？），那么请考虑将其替换为 JodaTime。

Answer 2

日期格式不同步。它
建议单独创建
为每个线程格式化实例。如果
多个线程访问一种格式
同时，必须同步
外部。

是的，这里不是线程安全的。

将其放在方法本地并为每个线程创建实例