######业务尽可能用sql实现(缩短后台响应时间):最近都在优化之前的代码:发现每个模块的index页都需要优化(很多都在循环里操作数据库:菜!初级!)

 

###最近都在优化之前的代码:

发现每个模块的index页都需要优化(很多都再循环里操作数据库:菜!初级!或者说赶项目没想优化这回事吧)

===》解耦合前提下,能用一条SQL解决的,多测试写SQL。可以减少很多代码。

希望我在疲惫,脑子不好使时候也能坚持吧。(注意休息,保持脑子好使!哈哈)

===》两次版本:

代码:

 /**
     * 版本2:
     * 媒体列表 显示优化。一条SQL。减少请求响应时间。
     * 定义一个VOMap。
     * */
    @Override
    public List<MediaVO> selectAll() {  //列表页:VO。====新建媒体,选择 incomWay,模仿User选 Role。
        List<MediaVO> mediaVOList = ccMediaDOMapper.selectAll();  //推广渠道:字典表,SQL
        return mediaVOList;
    }
/*
//        版本一:(背景:需求变动:新增需求:媒体要显示:媒体下广告位数目。)
//        为了省事。没有考虑效率。直接遍历List<mediaDO>根据mediaId查找广告位数目。
//        循环里操作数据库。导致响应请求很慢。。。
    @Override
    public List<MediaVO> selectAll() {  //列表页:VO。====新建媒体,选择 incomWay,模仿User选 Role。

        List<MediaDTO> mediaDTOList = ccMediaDOMapper.selectAll();  //推广渠道:字典表,SQL
        ArrayList<MediaVO> mediaVOList = new ArrayList<>();
        ArrayList<Integer> mediaIdList = new ArrayList<>();
        for (MediaDTO mediaDTO:mediaDTOList) {

            MediaVO mediaVO = new MediaVO();
            //===========属性赋值:
            DozerBeanMapper dozer = new DozerBeanMapper();
            dozer.map(mediaDTO, mediaVO);//取代手动set。
            mediaVOList.add(mediaVO);//====》VO没有slotCount属性值。

            Integer mediaDTOId = mediaDTO.getId();
            mediaIdList.add(mediaDTOId);
        }
//        ArrayList<MediaVO> slotCountList = ccAdslotDOMapper.getSlotCountListByMediaIdList(mediaIdList);
        List<Map<String,Integer > > slotCountMapList = ccAdslotDOMapper.getSlotCountListByMediaIdList(mediaIdList);
        for(int i = 0;i<slotCountMapList.size();i++){
            Map<String,Integer > slotCountMapThis = slotCountMapList.get(i);
            Integer mediaIdmap = slotCountMapThis.get("mediaId");
            Integer slotCount = slotCountMapThis.get("slotCount");
            for(int j = 0;j<mediaDTOList.size();j++) {

//                if (mediaIdmap.intValue() ==  ){  判断匹配。效率低。首先用SQL。。。slotCountMapList
//
//                }
            }

        }

        return mediaVOList;
    }
*/

SQL:之前版本就是简单关联用户表。基本上就是查询media基本信息的SQL难度级别。

  <select id="selectAll" resultMap="MediaVOMAP">
/** 别指望一次写出来:脑子在线,反复尝试。才写出来的这个SQL。脑子不在线,干脆去休息。 */
    SELECT result_media.*,
    COUNT(result_media.slotId) adslot_amount,
    u1.login_name creator_name,
    u2.login_name updater_name
    FROM (
    SELECT
--     admin_media.*
    <include refid="Base_Column_List" />,
    -- ,COUNT(result.id) adslot_amount
    admin_adslot.id AS slotId
    FROM admin_media
    /** 不可用INNER,要有无广告位都要查出媒体 */
    LEFT  JOIN admin_adslot ON admin_media.id = admin_adslot.media_id AND admin_adslot.is_deleted = 0
    AND admin_media.is_deleted = 0
    ) AS result_media
    /** 必须INNER,媒体必须要显示更新者和创建者 */
    INNER  JOIN sys_user u1 ON result_media.`user_id` = u1.id AND u1.is_deleted = 0
    INNER  JOIN sys_user u2 ON result_media.`user_id_update` = u2.id AND u2.is_deleted = 0

    GROUP BY result_media.id

  </select>
  <!--
  不行。
  <select id="selectAll" resultMap="MediaVOMAP">
    select
    <include refid="Base_Column_List" />,

    u1.login_name creator_name,
    u2.login_name updater_name,
    COUNT(admin_adslot.id) adslot_amount
    FROM admin_media
    INNER  JOIN sys_user u1 ON admin_media.`user_id` = u1.id AND u1.is_deleted = 0
    INNER  JOIN sys_user u2 ON admin_media.`user_id_update` = u2.id AND u2.is_deleted = 0
    INNER  JOIN admin_adslot ON admin_media.id = admin_adslot.media_id AND admin_adslot.is_deleted = 0
    AND admin_media.is_deleted = 0

  </select>-->

 

 

 

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值