###最近都在优化之前的代码:
发现每个模块的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>-->