Sonar问题解决：普通方法调用静态属性

最新推荐文章于 2024-08-23 21:30:00 发布

低至一折起

最新推荐文章于 2024-08-23 21:30:00 发布

阅读量2.7k

点赞数 3

CC 4.0 BY-SA版权

分类专栏： Java

本文链接：https://blog.youkuaiyun.com/benben683280/article/details/80273110

Java 专栏收录该内容

29 篇文章

订阅专栏

本文介绍了一种在使用Spring框架时如何避免Sonar警告的方法。通过将非静态方法设置Bean工厂中对静态变量的修改转换为通过静态方法进行，解决了Sonar扫描时提示的问题。这种方法不仅符合编码规范，还提高了代码的安全性和稳定性。

摘要生成于 C知道，由 DeepSeek-R1 满血版支持，前往体验 >

问题：普通方法调用静态属性时，Sonar会认为有问题

这是我们经常用的，实现Spring的Bean工厂，写一个工具类，
开放一个静态方法：获取实体类

public class BeanHelper implements BeanFactoryAware {

    private static BeanFactory factory;

    /**
     * 获取实体类
     * 
     * @param id
     * @return
     */
    public static <T> T getBean(String id) {
        return (T) factory.getBean(id);
    }

    /**
     * 设置bean工厂
     * 
     * @param beanFactory
     */
    @Override
    public void setBeanFactory(BeanFactory beanFactory) {
        BeanHelper.factory = beanFactory;
    }
}

Sonar扫描会报：

Instance methods should not write to “static” fields
Correctly updating a static field from a non-static method is tricky to get right and could easily lead to bugs if there are multiple class instances and/or multiple threads in play. Ideally, static fields are only updated from synchronized static methods.
This rule raises an issue each time a static field is updated from a non-static method.

翻译：

正确地从非静态方法更新静态字段是很棘手的, 如果在运行中有多个类实例和 / 或多个线程, 则可以很容易地导致 bug。理想情况下, 静态字段只从同步静态方法更新。
每次从非静态方法更新静态字段时, 此规则将引发一个问题。

解决办法：对静态属性包装一个静态方法

public class BeanHelper implements BeanFactoryAware {
    private static BeanFactory factory;

    /**
     * 获取实体类
     * 
     * @param id
     * @return
     */
    public static <T> T getBean(String id) {
        return (T) factory.getBean(id);
    }

    private static void setFactory(BeanFactory factory) {
        BeanHelper.factory = factory;
    }

    /**
     * 设置bean工厂
     * 
     * @param beanFactory
     */
    @Override
    public void setBeanFactory(BeanFactory beanFactory) {
        setFactory(beanFactory);
    }
}

包装一个静态 setFactory 方法，Sonar扫描就不会产生问题，由于是静态方法，会被最先初始化，更近一步，可以在 setFactory 方法中，判断 BeanHelper.factory 是否为 null，不为 null 则不赋值，保证不会被多实例或多线程覆盖，此处就不详细写出了。