一：背景介绍

本篇博客是对项目开发中出现的没有复用思想的反例进行的总结并进行的改进。目的是将经历转变为自己的经验。通过博客的方式分享给大家，大家一起共同进步和提高。

反例

在这里插入图片描述

反例一解读：
这两个归根到底都是查询在线人员，并且返回的数据类型和结构都是一致的。我们完全有条件可以只写一个接口来实现这两个类似的功能。

反例二解读：
这里使用了mybatis动态SQL进行处理，但是对于course_id与class_id完全可以抽出来，作为公共的进行使用。

思路&方案

模拟代码的环境为：spring boot、 spring mvc 、mybatis、mysql

反例一的优化思路和方案

对于反例一的优化思路：两个查询在线人员的的接口我们可以使用一个接口进行实现，两个接口的区别主要是在于入参不一致，这里我们可以通过使用mybatis的动态SQL进行实现。这样我们由之前的两个接口的（两个controller、两个IService、两个ServiceImpl、两个mapper、两个sql）转变成了一个（controller、Iservice、ServiceImpl、mapper、sql）。

反例一优化的模拟代码

Controller层

 /*
     * @description:查询课程内容
     * @author: wangwei
     * @date: 2023/3/7 16:00
     * @param: [courseContent]
     * @return: java.util.List<com.wangwei.mvc.entity.CourseContentEntity>
     **/
    @PostMapping("/queryCourseContent")
    public List<CourseContentEntity> queryCourseContent(@RequestBody CourseContentEntity courseContent){
        return iCourseContentService.queryCourseContent(courseContent);
    }

IService层

List<CourseContentEntity> queryCourseContent(CourseContentEntity courseContent);

ServiceImpl层

	@Resource
    CourseContentMapper courseContentMapper;
    /*
     * @description:查询课程内容
     * @author: wangwei
     * @date: 2023/3/7 15:57
     * @param: [courseContent]
     * @return: java.util.List<com.wangwei.mvc.entity.CourseContentEntity>
     **/
    @Override
    public List<CourseContentEntity> queryCourseContent(CourseContentEntity courseContent) {
        return courseContentMapper.queryCourseContentRecord(courseContent);
    }

mapper层

//通用查询语句
    List<CourseContentEntity> queryCourseContentRecord(CourseContentEntity courseContentEntity);

mapper.xml

<select id="queryCourseContentRecord" resultMap="courseContentMap">
        SELECT id,course_assembly_id,assembly_content,create_time,created_id,created_by,update_time,updated_id,updated_by
        FROM  tar_course_content_info
        WhERE
        is_delete=0
        <if test="id != null"> and id = #{id} </if>
        <if test="courseAssemblyId != null">and course_assembly_id = #{courseAssemblyId}</if>
        <if test="assemblyContent != null">and assembly_content = #{assemblyContent}</if>
        <if test="createdBy != null">and created_by = #{createdBy}</if>
        <if test="updatedBy != null">and updated_by = #{updatedBy}</if>
        <if test="remark != null">and remark = #{remark}</if>
    </select>