错误提示:This instance method writes to a static field. This is tricky to get correct
原代码:
@Component
public class SpringContextUtil implements ApplicationContextAware {
private static ApplicationContext applicationContext = null;
@Override
public void setApplicationContext(ApplicationContext applicationContext) throws BeansException {
this.applicationContext = applicationContext;
}
public static ApplicationContext getApplicationContext() {
return applicationContext;
}
public static Object getBean(String beanName) {
return applicationContext.getBean(beanName);
}
public static Object getBean(Class clazz) {
return applicationContext.getBean(clazz);
}
}
改进后的代码:
@Component
public class SpringContextUtil implements ApplicationContextAware {
private SpringContextUtil(){}
private static ApplicationContext applicationContext = null;
private synchronized static void setSyncApplicationContext(ApplicationContext applicationContext) {
SpringContextUtil.applicationContext = applicationContext;
}
@Override
public void setApplicationContext(ApplicationContext applicationContext) throws BeansException {
setSyncApplicationContext(applicationContext);
}
public static ApplicationContext getApplicationContext() {
return applicationContext;
}
public static Object getBean(String beanName) {
return applicationContext.getBean(beanName);
}
public static Object getBean(Class clazz) {
return applicationContext.getBean(clazz);
}
}
提示这个原因是,原来的写法不识线程安全的。所以通过增加synchronized,进行解决。
As an aside, in my opinion Google’s CodePro Analytix plugin is a much faster and more comprehensive tool than FindBugs.
Related:
参考:
https://stackoverflow.com/questions/13388829/write-to-static-field-is-findbugs-wrong-in-this-case