kotlin与Spring, 默认类,方法,property为final带来的问题--依赖注入失效,NullPointerException异常

来源:互联网 发布:太极线指标软件 编辑:程序博客网 时间:2024/06/17 00:28

问题的由来

在Spring中使用kotlin, 发现依赖注入似乎没有生效,导致NPE.
比如以下的代码,对于发送短信这个功能, 并不需要在主流程中同步发送, 所以就使用了Spring的@Async注解, 可以使这个方法在配置AsyncExecutor中执行.
但是实际调用时, 却发现contactDao为null, 程序报NullPointerException异常. 但是我用java写却没有问题.

@Serviceopen class MessageService {    //根据用户id获取用户联系信息的Dao    @Autowired    lateinit var contactDao: ContactDao    //将message发给一系列的用户ids    @Async    fun sendMessage(contactIds: List<String>, message: String) {        //先根据id获取用户联系信息        contactIds.map { contactDao.getContactById(it) }                //对每个用户发送message                .forEach { contact ->                    sendPhoneMessage(contact.phone,message)                }    }    fun sendPhoneMessage(phoneNumber:String, message:String){        ...    }}

原理

解决这个问题, 就需要对Spring的源码有一定的了解.
简单的说就是kotlin的类, 方法, property默认都是final, 是不能够被子类更改的. 而Spring的cglib代理需要非final类, 方法.

  1. Spring的异步注解(@Async), 事务(@Transactional), 缓存(@Cacheable,@CachePut,@CacheEvict等),Configuration这些其实都要更改方法的执行流程, 也就是说要使用代理来更改.
  2. 对于不指定具体代理方法, 且没有实现接口的类, spring会选择使用cglib来进行代理
  3. cglib代理过程是先根据Bean实例的类,生成一个子类(配置了拦截器), 然后寻找构造器, 构造一个子类实例(并没有对该实例进行任何注入设置, 该实例的属性的都是null).
  4. 代理后生成的实例非final方法调用会转发给原bean, 就可以获得依赖注入的属性值. 但是final方法是不会被代理的, 所以对上面的kotlin bean操作, 那么property肯定是null.

解决方法

所以我们可以手动将@Async注解的方法的kotlin类的类, 所有方法, 所有属性都改为open. 这样cglib代理时就不会有问题了.

open class MessageService {    @Autowired    lateinit open var contactCache: ContactCache    @Async    open fun sendMessage(contactIds: List<String>, message: AlarmMessage) {        contactIds.map { contactCache.getContactById(it) }                .forEach { contact ->                    sendPhoneMessage(contact.phone,message)                }    }    open fun sendPhoneMessage(phoneNumber:String, message:String){        ...    }}

或者使用官方的maven插件,kotlin all open插件.
All-open compiler plugin
配置all open依赖, 使用spring plugin, 就可以将有@Component, @Async, @Transactional, @Cacheable等的类编译为open了.

<plugin>    <artifactId>kotlin-maven-plugin</artifactId>    <groupId>org.jetbrains.kotlin</groupId>    <version>${kotlin.version}</version>    <configuration>        <compilerPlugins>            <plugin>spring</plugin>        </compilerPlugins>        <jvmTarget>1.8</jvmTarget>    </configuration>    <executions>        <execution>            <id>compile</id>            <phase>compile</phase>            <goals>                <goal>compile</goal>            </goals>        </execution>        <execution>            <id>test-compile</id>            <phase>test-compile</phase>            <goals>                <goal>test-compile</goal>            </goals>        </execution>    </executions>    <dependencies>        <dependency>            <groupId>org.jetbrains.kotlin</groupId>            <artifactId>kotlin-maven-allopen</artifactId>            <version>${kotlin.version}</version>        </dependency>    </dependencies></plugin>

总结:

Effective Java第二版第17条: 要么为继承而设计, 并提供文档说明, 要么就禁止继承.kotlin类, 方法默认final应该来源于此. 这条建议的确很好…不过对于我们这种普通开发者来说, 这个改变, 的确造成了很多不便, 特别是使用AOP的框架, 如果对源码不熟悉, 就很容易出错. 对于框架的开发者来说, 这的确是一个很好的实践.

这个问题的讨论可以看这里, kotlin语言的开发者有讨论为什么要将kotlin类和方法设计为final : https://discuss.kotlinlang.org/t/a-bit-about-picking-defaults/1418

0 0