目录
- 一、问题
- 二、优化建议
- 三、Repository模式
一、问题
最近在做代码重构,代码工程采用了Controller/Service/Dao分层架构,Dao层使用了Mybatis-Plus框架。
在查看Service层时发现如下代码:
@Service public class SampleServiceImpl implements SampleService { @Resource private SampleMapper sampleMapper; @Override public SampleTo findById(Long id) { Sample sample = this.sampleMapper.selectOne(Wrappers.<Sample>lambdaQuery() //仅查询指定的column .select(Sample::getId, Sample::getName, Sample::getDate) //查询条件 id = #{id} .eq(Sample::getId, id) ); return (SampleTo) BaseAssembler.populate(sample, new SampleTo()); } @Override public PageInfo<SampleTo> findListByDto(SampleQueryDto sampleQueryDto) { //开启分页 PageHelperUtil.startPage(sampleQueryDto); //查询分页列表 List<Sample> sampleList = this.sampleMapper.selectList(Wrappers.<Sample>lambdaQuery() //查询条件 id = #{id} .eq(Objects.nonNull(sampleQueryDto.getId()), Sample::getId, sampleQueryDto.getId()) //查询条件 name like concat('%', #{name}, '%') .like(StringUtils.hasText(sampleQueryDto.getName()), Sample::getName, sampleQueryDto.getName()) //查询条件 type = #{type} .eq(StringUtils.hasText(sampleQueryDto.getType()), Sample::getType, sampleQueryDto.getType()) //查询条件 date >= #{startDate} .ge(Objects.nonNull(sampleQueryDto.getStartDate()), Sample::getDate, sampleQueryDto.getStartDate()) //查询条件 date <= #{endDate} .le(Objects.nonNull(sampleQueryDto.getEndDate()), Sample::getDate, sampleQueryDto.getEndDate())); //转换分页结果 return PageHelperUtil.convertPageInfo(sampleList, SampleTo.class); } @Override public List<SampleTo> findListByCondition(String name, String type) { List<Sample> sampleList = this.sampleMapper.selectList(Wrappers.<Sample>lambdaQuery() //查询条件 name like concat('%', #{name}, '%') .like(StringUtils.hasText(name), Sample::getName, name) //查询条件 type = #{type} .eq(StringUtils.hasText(type), Sample::getType, type) ); return BaseAssembler.populateList(sampleList, SampleTo.class); } @Override public SampleDetailTo findDetailById(Long id) { return this.sampleMapper.findDetail(id); } @Override public Integer add(SampleAddDto sampleAddDto) { Sample sample = new Sample(); BaseAssembler.populate(sampleAddDto, sample); return this.sampleMapper.insert(sample); } }
dao层代码:
public interface SampleMapper extends BaseMapper<Sample> { //SQL脚本通过XML进行定义 SampleDetailTo findDetail(@Param("id") Long id); }
如上Service代码中直接使用Mybatis-Plus框架提供的Wrapper构造器,写的时候是挺爽,不用再单独为SampleMapper接口写XML脚本了,直接在Service类中都完成了,但是我不推荐这种写法。
分层架构的本意是通过分层来降低、隔离各层次的复杂度,
各层间仅通过接口进行通信,层间仅依赖抽象接口,不依赖具体实现,
只要保证接口不变可以自由切换每层的实现。
上述Service类中直接引用了Dao层实现框架Mybatis-Plus中的Wrappers类,尽管Servcie层依赖了Dao层的Mapper接口,但是Mapper接口中的参数Wrapper却是Dao层具体实现Mybatis-Plus所独有的,试想我们现在Dao层用的Mybatis-Plus实现,后续如果想将Dao层实现切换为Spring JPA,那Mybatis-Plus中Wrapper是不都要替换,那Servcie层中的相关Wrapper引用也都要进行替换,我们仅是想改变Dao实现,却不得不把Servcie层也进行修改。同时Service层本该是写业务逻辑代码的地方,但是却耦合进了大量的Wrapper构造逻辑,代码可读性差,难以捕捉到核心业务逻辑。
二、优化建议
那是不是Mybatis-Plus中的Wrapper就不能用了呢?我的答案是:能用,只是方式没用对。
Wrapper绝对是个好东西,方便我们构造Sql,也可以将我们从繁琐的XML脚本中解救出来,但是不能跨越层间界限。
优化建议如下:
- 移除Servcie中的Wrapper使用
- Java8+之后接口提供了默认方法的支持,可通过给Dao层Mapper接口添加default方法使用Wrapper
- 单表相关的操作 - 通过Dao层Mapper接口的default方法直接使用Wrapper进行实现,提高编码效率
- 多表关联的复杂操作 - 通过Dao层Mapper接口和XML脚本的方式实现
优化后的Service层代码如下:
@Service public class SampleServiceImpl implements SampleService { @Resource private SampleMapper sampleMapper; @Override public SampleTo findById(Long id) { Sample sample = this.sampleMapper.findInfoById(id); return (SampleTo) BaseAssembler.populate(sample, new SampleTo()); } @Override public SampleDetailTo findDetailById(Long id) { return this.sampleMapper.findDetail(id); } @Override public PageInfo<SampleTo> findListByDto(SampleQueryDto sampleQueryDto) { //开启分页 PageHelperUtil.startPage(sampleQueryDto); //查询分页列表 List<Sample> sampleList = this.sampleMapper.findList(sampleQueryDto); //转换分页结果 return PageHelperUtil.convertPageInfo(sampleList, SampleTo.class); } @Override public List<SampleTo> findListByCondition(String name, String type) { List<Sample> sampleList = this.sampleMapper.findListByNameAndType(name, type); return BaseAssembler.populateList(sampleList, SampleTo.class); } @Override public Integer add(SampleAddDto sampleAddDto) { Sample sample = new Sample(); BaseAssembler.populate(sampleAddDto, sample); return this.sampleMapper.insert(sample); } }
优化后的Dao层代码:
public interface SampleMapper extends BaseMapper<Sample> { default Sample findInfoById(Long id) { return this.selectOne(Wrappers.<Sample>lambdaQuery() //仅查询指定的column .select(Sample::getId, Sample::getName, Sample::getDate) //查询条件 id = #{id} .eq(Sample::getId, id) ); } default List<Sample> findList(SampleQueryDto sampleQueryDto) { return this.selectList(Wrappers.<Sample>lambdaQuery() //查询条件 id = #{id} .eq(Objects.nonNull(sampleQueryDto.getId()), Sample::getId, sampleQueryDto.getId()) //查询条件 name like concat('%', #{name}, '%') .like(StringUtils.hasText(sampleQueryDto.getName()), Sample::getName, sampleQueryDto.getName()) //查询条件 type = #{type} .eq(StringUtils.hasText(sampleQueryDto.getType()), Sample::getType, sampleQueryDto.getType()) //查询条件 date >= #{startDate} .ge(Objects.nonNull(sampleQueryDto.getStartDate()), Sample::getDate, sampleQueryDto.getStartDate()) //查询条件 date <= #{endDate} .le(Objects.nonNull(sampleQueryDto.getEndDate()), Sample::getDate, sampleQueryDto.getEndDate()) ); } default List<Sample> findListByNameAndType(String name, String type) { return this.selectList(Wrappers.<Sample>lambdaQuery() //查询条件 name like concat('%', #{name}, '%') .like(StringUtils.hasText(name), Sample::getName, name) //查询条件 type = #{type} .eq(StringUtils.hasText(type), Sample::getType, type) ); } //SQL脚本通过XML进行定义) SampleDetailTo findDetail(@Param("id") Long id); }
优化后的Servcie层完全移除了对Wrapper的依赖,将Servcie层和Dao层实现进行解耦,同时Dao层通过Java8+接口的默认方法同时支持Wrapper和XML的使用,整合编码和XML脚本的各自优势。
三、Repository模式
经过优化过后,Service层代码确实清爽了许多,移除了Mybatis-Plus的Wrapper构造逻辑,使得Service层可以更专注于业务逻辑的实现。但是细心的小伙伴还是会发现Servcie层仍旧依赖了Mybatis的分页插件PageHelper中的PageHelper类、PageInfo类,PageHelper插件也是技术绑定的(强绑定到Mybatis),既然我们们之前强调了Servcie层与Dao层间的界限,如此在Servcie层使用PageHelper也是越界了,例如后续如果切换Spring JPA,那PageHelper在Servcie层的相关的引用也都需要调整。
真正做到业务和技术解耦,可以参考DDD中的Repository(仓库、资源库)模式:
- 单独定义通用的分页查询参数DTO、分页查询结果DTO(与具体技术解耦)
- 定义Repository接口,仅依赖聚合、通用分页查询参数DTO、分页查询结果DTO
- 定义Repository接口的实现类,具体实现可依赖如Mybatis、JPA等Dao框架,在Repository的具体实现类中完成转换:
- 领域模型(聚合)<==> 数据实体
- 通用分页查询参数DTO、结果DTO <==> Dao框架的分页参数、结果(如PageHelper、IPage等)
DDD映射到代码层面,改动还是比较大的,所以在这次重构代码的过程中并没有真正采用DDD Repository模式,
而是仅从Servcie中移除Mybatis-Plus Wrapper便结束了,虽没有完全将Service层与Dao层实现(PageHelper)解耦,
但在Service层移除Wrapper构造逻辑后,使得Service层代码更清爽,可读性更好了,重构过程的代码改动量也在可接收的范围内。
到此这篇关于Mybatis-Plus Wrapper应该出现在Servcie类中吗?的文章就介绍到这了,更多相关Mybatis-Plus Wrapper内容请搜索自由互联以前的文章或继续浏览下面的相关文章希望大家以后多多支持自由互联!