SimpleDateFormat.clone()是否线程安全？遗留代码实现是否存在线程风险？

嘿，这个问题抓得很准，咱们一步步拆解来看：

1. 原代码是否线程安全？ #

首先得明确：SimpleDateFormat本身天生是非线程安全的——它内部持有一个共享的Calendar实例，多线程同时调用其格式化/解析方法时，会因为并发修改这个Calendar导致日期错乱、抛出异常等问题。

但你的代码里，foo()方法是把sdf.clone()的结果传给bar()，也就是说每个调用foo()的线程都会拿到一个独立的SimpleDateFormat克隆实例。如果bar()方法只是用这个克隆实例单独做格式化操作，且没有在多个线程间共享这个克隆实例的话，那**bar()里的格式化操作是线程安全的**。

不过这里有个重要前提：原静态实例sdf本身没有被其他线程直接调用过（比如没有其他代码用sdf.format(...)或sdf.parse(...)）。如果原sdf被其他线程并发使用，那sdf内部的Calendar状态会被打乱，这时候克隆出来的实例也可能带着错乱的状态——好在你的代码里sdf是private static final，只要没有其他代码直接操作它，这个风险就不存在。

2. 多个线程并发调用sdf.clone()，是否会导致克隆对象损坏？ #

这得深入看SimpleDateFormat的clone()实现逻辑：

• SimpleDateFormat的clone()方法会复制自身的核心状态：包括格式字符串、时区，以及内部的Calendar和NumberFormat实例——其中Calendar是通过调用calendar.clone()来复制的（Calendar本身非线程安全，但克隆会创建全新的实例）。
• 关键问题：clone()过程本身没有同步锁。如果原sdf的内部状态（比如Calendar）在克隆时被其他线程修改（比如原sdf被并发调用了格式化方法），那克隆出来的实例就可能带着不一致的状态，后续使用时出现异常或错误结果。

但在你的代码场景里，sdf是private static final，如果没有其他线程直接使用sdf的任何方法（只用来克隆），那原sdf的内部状态是完全稳定的——因为它从来没有被用来执行过格式化/解析操作，Calendar一直保持初始状态。这种情况下，多线程并发克隆sdf是安全的，不会出现克隆对象损坏的问题。

反过来，如果原sdf被其他线程并发使用（比如有地方直接调用sdf.format(...)），那此时并发克隆就可能出问题：格式化操作会修改sdf内部的Calendar，而克隆过程中正好读取这个Calendar的状态，就会导致克隆出的实例带着错乱的Calendar，后续使用时出现格式化错误。

内容的提问来源于stack exchange，提问作者DodgyCodeException