优化

README.md

@@ -1,18 +1,19 @@
 ## RuoYi-Vue-Ks
 
-RuoYi-Vue-Ks基于RuoYi-Vue优化改造，集成mybatisplus、knife4j、Hutool、lombok等组件，同步修改代码生成，旨在方便快速开发，将定期与RuoYi-Vue同步更新。
+RuoYi-Vue-Ks基于RuoYi-Vue优化改造，集成mybatisplus、knife4j、hutool、lombok等组件，同步修改代码生成，旨在方便快速开发，将定期与RuoYi-Vue同步更新。
 
 主要修改如下：
 
-* 集成mybatisplus实现mybatis增强
-* 集成knife4j实现swagger文档增强
-* 集成Hutool提供Java工具类库
-* 集成lombok插件简化实体类代码
+集成mybatisplus实现mybatis增强
+集成knife4j实现swagger文档增强
+集成hutool提供java工具类库
+集成lombok插件简化实体类代码
 
 特别说明：
 
-* 建数据表必须包含5个默认字段（create_by、create_time、update_by、update_time、remark）
-* 如需使用逻辑删除功能，只需在数据表中添加deleted字段（0表示未删除，1表示已删除）
+* 建数据表必须包含5个默认字段（create_by、create_time、update_by、update_time、remark）。
+* 如需使用逻辑删除功能，只需在数据表中添加deleted字段（0表示未删除，1表示已删除）。
+* 生成主子表增删改查代码时，子表的mapper接口、xml文件需要单独生成。
 
 <p align="center">
 	<img alt="logo" src="https://oscimg.oschina.net/oscnet/up-d3d0a9303e11d522a06cd263f3079027715.png">

ruoyi-generator/src/main/resources/vm/java/serviceImpl.java.vm

@@ -5,7 +5,6 @@ import org.springframework.stereotype.Service;
 #if($table.sub)
 import java.util.Arrays;
 import java.util.List;
-import java.util.ArrayList;
 import com.ruoyi.common.utils.StringUtils;
 import org.springframework.transaction.annotation.Transactional;
 import org.springframework.beans.factory.annotation.Autowired;
@@ -57,9 +56,9 @@ public class ${ClassName}ServiceImpl extends ServiceImpl<${ClassName}Mapper, ${C
     {
         List<${ClassName}> ${className}List = ${className}Mapper.selectList(new QueryWrapper<${ClassName}>(${className}));
         if (StringUtils.isNotNull(${className}List)) {
-            for (${ClassName} temp${className} : ${className}List) {
-                List<${subClassName}> ${subclassName}List = ${className}Mapper.select${subClassName}By${pkColumn.capJavaField}(${pkColumn.javaField});
-                temp${className}.set${subClassName}List(${subclassName}List);
+            for (${ClassName} temp : ${className}List) {
+                List<${subClassName}> ${subclassName}List = ${className}Mapper.select${subClassName}By${pkColumn.capJavaField}(temp.get${pkColumn.capJavaField}());
+                temp.set${subClassName}List(${subclassName}List);
             }
         }
         return ${className}List;
@@ -106,10 +105,8 @@ public class ${ClassName}ServiceImpl extends ServiceImpl<${ClassName}Mapper, ${C
     public int delete${ClassName}By${pkColumn.capJavaField}s(${pkColumn.javaType}[] ${pkColumn.javaField}s)
     {
         List<${pkColumn.javaType}> ${pkColumn.javaField}List = Arrays.asList(${pkColumn.javaField}s);
-        if (StringUtils.isNotNull(${pkColumn.javaField}List)) {
-            for (${pkColumn.javaType} ${pkColumn.javaField} : ${pkColumn.javaField}List) {
-                ${className}Mapper.delete${subClassName}By${subTableFkClassName}(${pkColumn.javaField});
-            }
+        for (${pkColumn.javaType} ${pkColumn.javaField} : ${pkColumn.javaField}List) {
+            ${className}Mapper.delete${subClassName}By${subTableFkClassName}(${pkColumn.javaField});
         }
         return ${className}Mapper.deleteBatchIds(${pkColumn.javaField}List);
     }