原来屎代码也有它的道理

深藏阁楼爱情的钟 2023-10-13 13:24 68阅读 0赞

@Component
    public class UserHelper implements InitializingBean {
    
        private UserService userService;
        private static UserService userService2;
    
        public UserHelper(UserService userService) {
            this.userService = userService;
        }
    
        @Override
        public void afterPropertiesSet() {
            setUserService2(userService);
        }
    
        private static void setUserService2(UserService userService) {
            userService2 = userService;
        }
    
        public static String getUserNameById(Integer id) {
            return userService2.getUserNameById(id);
        }
    }

当你看到上面的代码时，是不是会露出下面的表情？

![8869e691e6b02c230ab9d2b9aa400488.jpg][]

你肯定会说“这是什么屎代码？”。

其实，这段代码的本意是想定义一个静态的工具类，但是这个工具类需要注入bean对象。

我相信，你肯定在工作中也遇到过类似的需求：工具类里面需要注入bean。

代码之所以演变成上面这个样子，也是有一个过程的。

最初的代码长下面这样:

@Component
    public class UserHelper{
        
        private static UserService userService;
    
        public UserHelper(UserService userService) {
            UserHelper.userService = userService;
        }
    
        public static String getUserNameById(Integer id) {
            return userService.getUserNameById(id);
        }
    }

我相信，这才是大家熟悉的味道。

因为我们这个工具类要提供的是静态方法，因此我们需要把userService声明成类的静态属性。

而userService本身是一个bean，所以我需要把它注入进来。这里，我选的的是构造注入，并且在类上使用@Component注解。

的确，这段代码也能正常工作，可是为啥要改它呢？

如果你在idea里面安装了SonarLint，你就会发现，SonarLint认为这种写法有坏味道。

> Assigning a value to a static field in a constructor could cause unreliable behavior at runtime since it will change the value for all instances of the class.  
> Instead remove the field’s static modifier, or initialize it statically.

意思就是不要在构造函数中对静态成员初始化，因为静态成员是属于类的，在构造方法中对静态成员初始化，可能会影响所有实例。

它建议删除静态修饰符，或者选择进行静态初始化（比如静态初始化块，或者静态方法）。

首先，我们没办法删除静态修饰符，否则静态方法就访问不到了。

那么，我们就尝试用第二种方法：我们可以定义一个setUserService的静态方法，然后在构造方法方法中调用这个方法给userService赋值。

@Component
    public class UserHelper{
    
        private static UserService userService;
    
        public UserHelper(UserService userService) {
            setUserService(userService);
        }
    
        
        public static void setUserService(UserService userService) {
            UserHelper.userService = userService;
        }
        
        public static String getUserNameById(Integer id) {
            return userService.getUserNameById(id);
        }
    }

正当我满心欢喜的认为问题解决了时，SonarLint又给我当头棒喝——我们还有一个问题没有解决。

> Utility classes should not have public constructors

意思就是工具类不应该public的构造方法。

的确，它是对的，工具类确实不需要实例化成员，因此也就不需要声明一个public的构造方法。

但是我们又确实需要这个public构造方法，进行构造注入，咋办呢？

换成setter注入行不行呢？

@Component
    public class UserHelper{
    
        private static UserService userService;
    
    
        @Autowired
        private void setUserService(UserService userService) {
            UserHelper.userService = userService;
        }
        
    
        public static String getUserNameById(Integer id) {
            return userService.getUserNameById(id);
        }
    }

不幸的是有来一个新的问题。

> Instance methods should not write to "static" fields

![e9ba151f8252cbb54daded89f7eb6a3c.jpg][]

这下我们明白了，无论构造方法还是实例方法， 都是不建议来设置静态成员的——道理都是一样，都是为了防止某个实例的行为，影响全部其他实例。

回过头我们再来看看最开始的代码：

@Component
    public class UserHelper {
    
        private UserService userService;
    
        public UserHelper(UserService userService) {
            this.userService = userService;
        }

我们把userService改成了实例方法，用构造注入。但是这个userService没办法被静态方法访问到。咋办呢？我们再定义一个静态的userService2。只需要把userService赋值给userService2就行了。所以这里我们实现了InitializingBean接口。

但是我们并不能直接在afterPropertiesSet方法里面给userService2赋值，因为afterPropertiesSet是实例方法，不要再实例方法里面设置静态成员。

所以我们需要定义一个静态方法setUserService2，通过它给userService2复制。

其实本质上，setUserService2还是在实例方法里面给静态成员赋值。只是这样能绕过sonar的检测而已。

虽然上面的这段代码，sonar没有检测出问题，但是显然它不是好代码——它实在是太绕了。

我们应该反问自己，为什么我一定要使用静态方法？

在没有IOC容器之前，使用静态方法很方便，因为不需要创建实例。但是spring会帮我们创建bean对象，因此我们没有必要固执于静态方法或者说工具类。

我们应该在定义这个UserHelper之初就把它定义成一个常规的bean：

@Component
    public class UserHelper {
    
        private UserService userService;
    
        public UserHelper(UserService userService) {
            this.userService = userService;
        }
    
        public String getUserNameById(Integer id) {
            return userService.getUserNameById(id);
        }
    }

如果你说，我的代码里面已经用了静态方法了，我不想改了，那么你可以这样。

public class UserHelper {
    
        private UserHelper() {}
        
        private static UserService userService;
        private static UserService getUserService() {
            if (userService == null) {
                userService =  Objects.requireNonNull(ContextLoader.getCurrentWebApplicationContext())
                        .getBean(UserService.class);
            }
            return userService;
        }
    
        public static String getUserNameById(Integer id) {
            return getUserService().getUserNameById(id);
        }
    }

[8869e691e6b02c230ab9d2b9aa400488.jpg]: https://img-blog.csdnimg.cn/img_convert/8869e691e6b02c230ab9d2b9aa400488.jpg
[e9ba151f8252cbb54daded89f7eb6a3c.jpg]: https://img-blog.csdnimg.cn/img_convert/e9ba151f8252cbb54daded89f7eb6a3c.jpg