From 82f1f5d0cf1b51a5d81915e842e01760f404fa74 Mon Sep 17 00:00:00 2001 From: 疯狂的狮子li <15040126243@163.com> Date: 星期三, 20 十月 2021 13:07:16 +0800 Subject: [PATCH] update 优化xxl-job-admin相关pr代码 增加格式化日志输出与docker镜像 --- ruoyi-common/src/main/java/com/ruoyi/common/core/mybatisplus/core/ServicePlusImpl.java | 58 ++++++++++++++++++++++++++++++++-------------------------- 1 files changed, 32 insertions(+), 26 deletions(-) diff --git a/ruoyi-common/src/main/java/com/ruoyi/common/core/mybatisplus/core/ServicePlusImpl.java b/ruoyi-common/src/main/java/com/ruoyi/common/core/mybatisplus/core/ServicePlusImpl.java index 3a461a8..e9c1f19 100644 --- a/ruoyi-common/src/main/java/com/ruoyi/common/core/mybatisplus/core/ServicePlusImpl.java +++ b/ruoyi-common/src/main/java/com/ruoyi/common/core/mybatisplus/core/ServicePlusImpl.java @@ -1,19 +1,19 @@ package com.ruoyi.common.core.mybatisplus.core; import cn.hutool.core.bean.copier.CopyOptions; +import cn.hutool.core.collection.CollUtil; import cn.hutool.core.util.ObjectUtil; import com.baomidou.mybatisplus.core.conditions.Wrapper; import com.baomidou.mybatisplus.core.metadata.TableInfo; import com.baomidou.mybatisplus.core.metadata.TableInfoHelper; import com.baomidou.mybatisplus.core.toolkit.Assert; -import com.baomidou.mybatisplus.core.toolkit.ClassUtils; +import com.baomidou.mybatisplus.core.toolkit.ReflectionKit; import com.baomidou.mybatisplus.extension.service.impl.ServiceImpl; import com.ruoyi.common.core.page.PagePlus; import com.ruoyi.common.utils.BeanCopyUtils; import com.ruoyi.common.utils.reflect.ReflectUtils; import lombok.extern.slf4j.Slf4j; import org.springframework.beans.factory.annotation.Autowired; -import org.springframework.core.ResolvableType; import java.io.Serializable; import java.util.ArrayList; @@ -24,11 +24,14 @@ /** * IServicePlus 瀹炵幇绫� * + * @param <M> Mapper绫� + * @param <T> 鏁版嵁瀹炰綋绫� + * @param <V> vo绫� * @author Lion Li */ @Slf4j @SuppressWarnings("unchecked") -public class ServicePlusImpl<M extends BaseMapperPlus<T>, T, K> extends ServiceImpl<M, T> implements IServicePlus<T, K> { +public class ServicePlusImpl<M extends BaseMapperPlus<T>, T, V> extends ServiceImpl<M, T> implements IServicePlus<T, V> { @Autowired protected M baseMapper; @@ -46,31 +49,26 @@ return entityClass; } - protected Class<T> mapperClass = currentMapperClass(); + protected Class<M> mapperClass = currentMapperClass(); - protected Class<K> voClass = currentVoClass(); + protected Class<V> voClass = currentVoClass(); - public Class<K> getVoClass() { + public Class<V> getVoClass() { return voClass; } @Override - protected Class<T> currentMapperClass() { - return (Class<T>) this.getResolvableType().as(ServicePlusImpl.class).getGeneric(0).getType(); + protected Class<M> currentMapperClass() { + return (Class<M>) ReflectionKit.getSuperClassGenericType(this.getClass(), ServicePlusImpl.class, 0); } @Override protected Class<T> currentModelClass() { - return (Class<T>) this.getResolvableType().as(ServicePlusImpl.class).getGeneric(1).getType(); + return (Class<T>) ReflectionKit.getSuperClassGenericType(this.getClass(), ServicePlusImpl.class, 1); } - protected Class<K> currentVoClass() { - return (Class<K>) this.getResolvableType().as(ServicePlusImpl.class).getGeneric(2).getType(); - } - - @Override - protected ResolvableType getResolvableType() { - return ResolvableType.forClass(ClassUtils.getUserClass(getClass())); + protected Class<V> currentVoClass() { + return (Class<V>) ReflectionKit.getSuperClassGenericType(this.getClass(), ServicePlusImpl.class, 2); } /** @@ -123,6 +121,9 @@ */ @Override public boolean saveAll(Collection<T> entityList) { + if (CollUtil.isEmpty(entityList)) { + return false; + } return baseMapper.insertAll(entityList) == entityList.size(); } @@ -131,6 +132,9 @@ */ @Override public boolean saveOrUpdateAll(Collection<T> entityList) { + if (CollUtil.isEmpty(entityList)) { + return false; + } TableInfo tableInfo = TableInfoHelper.getTableInfo(entityClass); Assert.notNull(tableInfo, "error: can not execute. because can not find cache of TableInfo for entity!"); String keyProperty = tableInfo.getKeyProperty(); @@ -146,10 +150,12 @@ updateList.add(entity); } } - if (updateBatchById(updateList)) { + if (CollUtil.isNotEmpty(updateList) && updateBatchById(updateList)) { row += updateList.size(); } - row += baseMapper.insertAll(addList); + if (CollUtil.isNotEmpty(addList)) { + row += baseMapper.insertAll(addList); + } return row == entityList.size(); } @@ -159,7 +165,7 @@ * @param id 涓婚敭ID */ @Override - public K getVoById(Serializable id, CopyOptions copyOptions) { + public V getVoById(Serializable id, CopyOptions copyOptions) { T t = getBaseMapper().selectById(id); return BeanCopyUtils.oneCopy(t, copyOptions, voClass); } @@ -170,7 +176,7 @@ * @param idList 涓婚敭ID鍒楄〃 */ @Override - public List<K> listVoByIds(Collection<? extends Serializable> idList, CopyOptions copyOptions) { + public List<V> listVoByIds(Collection<? extends Serializable> idList, CopyOptions copyOptions) { List<T> list = getBaseMapper().selectBatchIds(idList); if (list == null) { return null; @@ -184,7 +190,7 @@ * @param columnMap 琛ㄥ瓧娈� map 瀵硅薄 */ @Override - public List<K> listVoByMap(Map<String, Object> columnMap, CopyOptions copyOptions) { + public List<V> listVoByMap(Map<String, Object> columnMap, CopyOptions copyOptions) { List<T> list = getBaseMapper().selectByMap(columnMap); if (list == null) { return null; @@ -199,7 +205,7 @@ * @param queryWrapper 瀹炰綋瀵硅薄灏佽鎿嶄綔绫� {@link com.baomidou.mybatisplus.core.conditions.query.QueryWrapper} */ @Override - public K getVoOne(Wrapper<T> queryWrapper, CopyOptions copyOptions) { + public V getVoOne(Wrapper<T> queryWrapper, CopyOptions copyOptions) { T t = getOne(queryWrapper, true); return BeanCopyUtils.oneCopy(t, copyOptions, voClass); } @@ -210,7 +216,7 @@ * @param queryWrapper 瀹炰綋瀵硅薄灏佽鎿嶄綔绫� {@link com.baomidou.mybatisplus.core.conditions.query.QueryWrapper} */ @Override - public List<K> listVo(Wrapper<T> queryWrapper, CopyOptions copyOptions) { + public List<V> listVo(Wrapper<T> queryWrapper, CopyOptions copyOptions) { List<T> list = getBaseMapper().selectList(queryWrapper); if (list == null) { return null; @@ -225,9 +231,9 @@ * @param queryWrapper 瀹炰綋瀵硅薄灏佽鎿嶄綔绫� */ @Override - public PagePlus<T, K> pageVo(PagePlus<T, K> page, Wrapper<T> queryWrapper, CopyOptions copyOptions) { - PagePlus<T, K> result = getBaseMapper().selectPage(page, queryWrapper); - List<K> volist = BeanCopyUtils.listCopy(result.getRecords(), copyOptions, voClass); + public PagePlus<T, V> pageVo(PagePlus<T, V> page, Wrapper<T> queryWrapper, CopyOptions copyOptions) { + PagePlus<T, V> result = getBaseMapper().selectPage(page, queryWrapper); + List<V> volist = BeanCopyUtils.listCopy(result.getRecords(), copyOptions, voClass); result.setRecordsVo(volist); return result; } -- Gitblit v1.9.3