ICode9

精准搜索请尝试: 精确搜索
首页 > 编程语言> 文章详细

java – 为什么添加超类会打破我的Spring bean?

2019-06-27 10:06:16  阅读:170  来源: 互联网

标签:java spring spring-boot-2 abstract-class


我有一个正常工作的Spring Boot Web应用程序.我注意到两个@Repository bean有很多共同点,所以我使用抽象超类重构它们,现在我的应用程序被破坏了.我经过双重检查,这是我在工作和非工作状态之间唯一的变化.谁能看到我做错了什么?

这是我的工作代码:

public class One { ... }
public class Two { ... }

@Repository
public class RepoOne {
    private final ISource<One> sourceOne;
    private ICache<One> cache;
    @Value("${com.example.lifetime.one}")
    private int lifetime;
    public RepoOne(ISource<One> sourceOne) {
       this.sourceOne = sourceOne;
    }
    @PostConstruct
    public void createCache() {
       Duration lifetime = Duration.ofMinutes(this.lifetime);
       this.cache = new Cache<>(lifetime, sourceOne);
    }
    public One get(String key) {
      return cache.get(key);
    }
}

@Repository
public class RepoTwo {
    private final ISource<Two> sourceTwo;
    private ICache<Two> cache;
    @Value("${com.example.lifetime.two}")
    private int lifetime;
    public RepoOne(ISource<Two> sourceTwo) {
        this.sourceTwo = sourceTwo;
    }
    @PostConstruct
    public void createCache() {
        Duration lifetime = Duration.ofMinutes(this.lifetime);
        this.cache = new Cache<>(lifetime, sourceTwo);
    }
    public Two get(String key) {
        return cache.get(key);
    }
}

@Service
public class RepoService {
    private final RepoOne repoOne;
    private final RepoTwo repoTwo;
    public RepoService(RepoOne repoOne, RepoTwo repoTwo) {
        this.repoOne = repoOne;
        this.repoTwo = repoTwo;
    }
    public void doSomething(String key) {
        One one = repoOne.get(key);
        ...
    }
}

这是我重新设计的代码,我在其中介绍了一个抽象的通用超类.

abstract class AbstractRepo<T> {
    private final ISource<T> source;
    private ICache<T> cache;
    AbstractRepo (ISource<T> source) {
       this.source = source;
    }
    @PostConstruct
    private void createCache() {
       Duration lifetime = Duration.ofMinutes(lifetime());
       this.cache = new Cache<>(lifetime, source);
    }
    protected abstract int lifetime();
    public final T get(String key) {
        return cache.get(key);
    }
}

@Repository
public class RepoOne extends AbstractRepo<One> {
    @Value("${com.example.lifetime.one}")
    private int lifetime;
    public RepoOne(ISource<One> sourceOne) {
       super(source);
    }
    protected int lifetime() { return lifetime; }
}

@Repository
public class RepoTwo extends AbstractRepo<Two> {
    @Value("${com.example.lifetime.two}")
    private int lifetime;
    public RepoTwo(ISource<Two> sourceTwo) {
       super(source);
    }
    protected int lifetime() { return lifetime; }
}

当使用重新分解的代码时,我在AbstractRepo :: get()中得到一个NullPointerException.我已通过调试器确认缓存为空(以及源).但是,我还通过调试器确认创建了RepoOne和RepoTwo的实例并调用了它们的createCache()方法.就好像每个实例的两个实例都被创建,只有一个被初始化.有什么想法吗?

解决方法:

这不是您引入父类的事实,而是您将get方法转换为final方法的事实.

使用@Repository注释的类将获得自动异常转换.通过使用AOP添加此自动异常转换.在Spring中应用AOP的默认机制是使用代理,在这种情况下是基于类的代理.

会发生什么是CgLib通过子类化为您的类创建代理,以便在调用方法时可以添加建议.但是,最终方法不能在子类中重写.这将导致在代理上调用get方法而不是实际实例.

有两种方法可以解决这个问题

>删除final关键字
>引入定义存储库合同的接口.这将导致创建JDK动态代理. JDK动态代理是基于接口的,不需要子类化您的实际类(仅适用于基于类的代理).

标签:java,spring,spring-boot-2,abstract-class
来源: https://codeday.me/bug/20190627/1303514.html

本站声明: 1. iCode9 技术分享网(下文简称本站)提供的所有内容,仅供技术学习、探讨和分享;
2. 关于本站的所有留言、评论、转载及引用,纯属内容发起人的个人观点,与本站观点和立场无关;
3. 关于本站的所有言论和文字,纯属内容发起人的个人观点,与本站观点和立场无关;
4. 本站文章均是网友提供,不完全保证技术分享内容的完整性、准确性、时效性、风险性和版权归属;如您发现该文章侵犯了您的权益,可联系我们第一时间进行删除;
5. 本站为非盈利性的个人网站,所有内容不会用来进行牟利,也不会利用任何形式的广告来间接获益,纯粹是为了广大技术爱好者提供技术内容和技术思想的分享性交流网站。

专注分享技术,共同学习,共同进步。侵权联系[81616952@qq.com]

Copyright (C)ICode9.com, All Rights Reserved.

ICode9版权所有