不同线程调用同一个方法出现的问题

有一个DateUtil类，如下：

package com;
import java.text.SimpleDateFormat;
import java.util.Date;

public class DateUtil {
	private static final SimpleDateFormat yyyyMmDd = new SimpleDateFormat(
			"yyyy-MM-dd");
	public static String getBookInfoDate(int days) {
		Date now = new Date();
		return yyyyMmDd.format(new Date(now.getTime() - (long) days * 24L * 60L
				* 60L * 1000L));
	}

}

现在有两个线程都调用了该方法：
调用如下：
类1调用：debugLog.info("传参-2："+DateUtil.getBookInfoDate(-2));
类2调用：debugLog.info("传参1："+DateUtil.getBookInfoDate(1));
这两个类作为两个线程在同一时刻运行：
运行的结果如下：
2012-11-08|16:05:22.392|RMIScheduler_Worker-2|INFO |传参-2：2012-11-10|(FwcpTrailDueRemindJob.java:40)
2012-11-08|16:05:22.392|RMIScheduler_Worker-1|INFO |传参1：2012-11-10|(FwcpApplyRemindJob.java:40)

在同一时刻调用了同一个方法，虽然参数不一样，但是返回值是一样的。

如果吧DateUtil改为：

package com;

import java.text.SimpleDateFormat;
import java.util.Date;

public class DateUtil {
	public static String getBookInfoDate(int days) {
		Date now = new Date();
		SimpleDateFormat yyyyMmDd = new SimpleDateFormat("yyyy-MM-dd");
		return yyyyMmDd.format(new Date(now.getTime() - (long) days * 24L * 60L
				* 60L * 1000L));
	}

}

或者

package com;

import java.text.SimpleDateFormat;
import java.util.Date;

public class DateUtil {
	private static final SimpleDateFormat yyyyMmDd = new SimpleDateFormat(
			"yyyy-MM-dd");
	public static synchronized String getBookInfoDate(int days) {
		Date now = new Date();
		return yyyyMmDd.format(new Date(now.getTime() - (long) days * 24L * 60L
				* 60L * 1000L));
	}
}

把调用的方法改成上面两个运行出来的结果就正常了
2012-11-08|16:18:07.850|RMIScheduler_Worker-2|INFO |传参-2：2012-11-10|(FwcpTrailDueRemindJob.java:40)
2012-11-08|16:18:07.850|RMIScheduler_Worker-1|INFO |传参1：2012-11-07|(FwcpApplyRemindJob.java:40)

为什么第一种方法就会出现返回值一样？有没有人知道呀

评论

2 楼 pch272215690 2012-12-17  

SimpleDateFormat不是线程安全的，把他写成一个常量大家都用，多线程下就出这问题了。

1 楼 pch272215690 2012-11-08  

package com;

import java.text.SimpleDateFormat;
import java.util.Date;

public class DateUtil {
private static final SimpleDateFormat yyyyMmDd = new SimpleDateFormat(
"yyyy-MM-dd");
public static synchronized String getBookInfoDate(int days) {
Date now = new Date();
return yyyyMmDd.format(new Date(now.getTime() - (long) days * 24L * 60L
* 60L * 1000L));
}
} 

这个格式好