views:

76

answers:

2

I know that SimpleDateFormat and NumberFormat are NOT thread safe.
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=4101500

But what about the other Format classes like MessageFormat?

Fortify 360 is flagging the use of "MessageFormat.format(String, Object...)" static method as a "Race Condition - Format Flaw" issue, but when I analyze the the source code of MessageFormat, I saw that in that method, it creates a new local instance of MessageFormat itself.

Is the Java MessageFormat Class thread safe?

+2  A: 

The javadoc for MessageFormat says:

Message formats are not synchronized. It is recommended to create separate format instances for each thread. If multiple threads access a format concurrently, it must be synchronized externally.

So officially, no - it's not thread-safe.

The docs for SimpleDateFormat say much the same thing.

Now, the docs may just be being conservative, and in practice it'll work just fine in multiple threads, but it's not worth the risk.

skaffman
Thanks for showing the JavaDoc, that would be enough for me. When I viewed the source code of MessageFormat, it's even clearer to me why it's not thread safe. That class makes use of NumberFormat and DateFormat, two classes that are Not thread safe.
thenonhacker
+2  A: 

If you are referrring to the method

public static String format(String pattern, Object... arguments)

this is thread-safe since as described in the javadoc it creates a new MessageFormat to do the formatting.

BTW, thats a funny typo in your title 'SimpleThreadFormat' :)

Jörn Horstmann