码迷,mamicode.com
首页 > 其他好文 > 详细

记录一次代码优化

时间:2016-12-13 07:49:31      阅读:241      评论:0      收藏:0      [点我收藏+]

标签:time()   gettime   对比   userinfo   erb   最好   调用   type   分享   

前不久的项目时间紧张,为了尽快完成原型开发,写了一段效率相当低的代码。

最近几天闲下来,主动把之前的代码优化了一下:)

 
标签:Java、Mybatis、MySQL
概况:本地系统从另外一个系统得到实体类集合List<UserEvent>,但是实体中只有eventId信息,其他属性值均为空。
需要从数据库中查询数据,完善List<UserEvent>的信息并返回。
相关业务表以及对应的实体类,如下图。(为了回避项目信息,相关业务内容均省略,以下表名、实体名、代码变量名等均用字母ABC代替。)
技术分享

原处理

1.先来看代码,乍一看逻辑清晰,符合正常思维习惯。
但是仔细查看发现,for循环中的每步处理都和数据查询有关。假设有10次循环,每次循环中有4次数据库连接查询,一共需要连接数据库40次。
每次数据库连接都需要一定的开销,随着循环量不断增加,处理时间将成倍增长,造成资源浪费。
 1  String eventId = "";
 2  String aTime = "";
 3  for (UserEvent event : userEventList) {
 4       eventId = event.getEventId();
 5   
 6       // 获取业务B信息
 7       List<EntityB> listB = mapperB.selectBInfoByEventId(eventId);
 8       event.setListB(listB);
 9   
10      // 获取业务C信息
11      List<EntityC> listC = mapperC.selectCInfoByEventId(eventId);
12      event.setListC(listC);
13  
14      // 查看是否有业务B处理
15      EntityB entityB = mapperB.selectBInfoByPrimary(phone, eventId);
16      event.setIsActionB(null == entityB ? "false" : "true");
17  
18      // 获取业务A和用户信息
19      User userInfo = mapperA.selectEventUserInfoByEventId(eventId);
20      if(null != userInfo){
21          aTime = userInfo.getTime();
22          event.setTime(aTime == null ? "" : sdfMd.format(sdfYmd.parse(aTime)));
23                      event.setUserInfo(userInfo);
24       }
25  }

 

2.再来看查询语句。
业务表A和业务表B没有复杂的查询。只有业务表C使用了一个子查询,来获取表内自身数据引用的信息。
各业务表数据都需要关联到用户表User。
 1   <select id="selectBInfoByEventId" parameterType="String" resultType="EntityA">
 2     SELECT
 3       B.phone     AS phone,
 4       B.time      AS time,
 5       U.name      AS userName
 6     FROM table_b B
 7     LEFT JOIN user U ON U.phone = B.phone
 8     WHERE B.event_id = #{eventId}
 9     ORDER BY B.time ASC
10   </select>
11   <select id="selectCInfoByEventId" parameterType="String" resultType="EntityC">
12     SELECT
13       C.cmtId,
14       C.referId,
15       C.time,
16       U.name   AS userName
17       ( SELECT TU.name FROM table_c TC
18         LEFT JOIN user TU ON TU.phone = TC.phone
19         WHERE TC.cmt_id = TC.refer_id
20        ) AS referName
21     FROM table_c C
22       LEFT JOIN user U ON C.phone = U.phone
23     WHERE C.event_id = #{eventId}
24     ORDER BY C.time ASC
25   <select id="selectEventUserInfoByEventId" parameterType="java.lang.String" resultType="User">
26       SELECT
27           U.name,
28           U.picId,
29           A.time
30        FROM table_a A
31        LEFT JOIN user U ON U.phone = A.phone
32       WHERE A.event_id = #{eventId}
33   </select>

 

优化分析

  1. 在代码结构上,要避免在for循环中作查询处理。考虑将查询参数evenId从for循环中提取出来,做批量查询,然后再将查询结果设定到对应的实体类中。
  2. 在业务上,对于每一个UserEvent中的eventId,业务表A中必定对应有一条记录,而在业务表B和业务表C中则未必有与这个eventId关联的数据。因此,可以将业务表A作为主表,通过eventId与另外几个表关联查询。查询次数也由原来的至少四次减少为一次查询。
  3. 对于联合查询的结果,以UserEvent作为查询结果的实体类,使用Mybatis中的collection、association来处理结果映射。
  4. 另外,各业务表的查询中都有与用户表User的关联,考虑将各业务信息的查询处理创建为视图。这样不仅能简化联合查询中SQL语句,也可以隔离基础表的数据。

 

优化后的代码 

  int eventSize = userEventList.size();
  List<String> eventIds = new ArrayList<String>(); 
  // 如果考虑去掉重复数据,可以使用集合Set,但是作为Mybatis的出入参数,最后还是需要将Set转化为List。
// 此处直接使用List,因为在业务上排除了重复数据的可能性。
for (int i = 0; i < eventSize; i++) { eventIds.add(userEventList.get(i).getEventId()); } Map<String, Object> paramsMap = new HashMap<String, Object>(); paramsMap.put("eventIds", eventIds); paramsMap.put("phone", phone); List<UserEvent> eventInfoList = eventMapper.selectUserEventInfo(paramsMap); // 将查询结果转化为Map存储,方便调用 Map<String, UserEvent> eventInfoMap = new HashMap<String, UserEvent>(); for(UserEvent event : eventInfoList){ eventInfoMap.put(event.getEventId(), event); } UserEvent newEvent = null; String aTime = null; for(UserEvent event : roadEventList){ // 从查询结果Map中取出补充信息,保存到原UserEvent对象中 newEvent =eventInfoMap.get(event.getEventId()); if(null != newEvent ){ aTime = newEvent.getTime(); event.setTime(aTime == null ? "" : sdfMd.format(sdfYmd.parse(aTime ))); event.setIsActionB(newEvent.getIsActionB() == null ? "false" : newEvent.getIsActionB()); event.setUserInfo(newEvent.getUserInfo()); event.setListB(newEvent.getListB()); event.setListC(newEvent.getListC()); } }

 

    <resultMap id="UserMap" type="User">
        <result column="name" property="name" />
        <result column="picId" property="picId" />
        <result column="time" property="time" />
    </resultMap>
    
    <resultMap id="BMap" type="EntityB">
        <id column="bPhone" property="phone" />
        <result column="bUserName" property="userName" />
         <result column="bTime" property="time" />
     </resultMap>
    
     <resultMap id="CMap" type="EntityC">
         <id column="cmtId" property="cmtId" />
         <result column="referId" property="referId" />
         <result column="cUserName" property="userName" />
         <result column="referName" property="referName" />
         <result column="cTime" property="time" />
     </resultMap>
    
     <resultMap id="EventResultMap" type="com.xxxx.bean.UserEvent">
         <id column="eventId" property="eventId" />
         <result column="time" property="time" />
         <result column="isActionB" property="isActionB" />
         <association property="userInfo" resultMap="UserMap" />
         <collection property="listB" resultMap="BMap" />
         <collection property="listC" resultMap="CMap" />
     </resultMap>
 
     <select id="selectUserEventInfo" resultMap="EventResultMap">
       SELECT
           A.eventId,
           A.time,
           A.name,
           A.picId,
           CASE WHEN B.phone = #{phone} THEN "true" ELSE "false" END AS isActionB,
           B.phone          AS bPhone,
           B.userName       AS bUserName,
           B.time           AS bTime,
           C.cmtId,
           C.referId,
           C.userName       AS cUserName,
           C.referName      AS referName,
           C.time           AS cTime
        FROM v_table_a A
        LEFT JOIN v_table_b B ON B.eventId = A.eventId
        LEFT JOIN v_table_c C ON C.eventId = A.eventId
       <where>
           A.event_id in
           <foreach collection="eventIds" index="" item="eventId" 
            open
="(" separator="," close=")"> #{eventId} </foreach> </where>; </select>

 

编码时需要注意的几个地方

1. 复杂对象的映射解析

采用resultMap嵌套。其中,collection标签表示映射一个集合,association标签表示映射一个实体类,
标签中的property属性值对应的是,该集合/实体在查询结果对象中的变量名。
 
对于各表中名称相同的字段,需要建立别名,否则解析时无法确定各属性与表字段的对应关系。
如:业务表B和业务表C中都有userName字段,在查询语句中为为字段别名加了前缀来区分。
B.userName AS bUserName, <result column="bUserName"property="userName"/>
C.userName AS cUserName, <result column="cUserName" property="userName" />
 
resultMap中type属性表示标签所包含内容对应映射的Java类。
该属性可以写类的全路径(如:<resultMap id="EventResultMap" type="com.xxxx.bean.UserEvent"> ),
也可以配置为简写的类名(如:<resultMap id="UserMap" type="User"> )。
简写的类名需要在xml配置文件中设置(如下),配好之后的简写类名可以在各个sql.xml中使用。
  <!-- spring-mybatis.xml文件 -->
  <!-- 配置sqlSessionFactory -->
  <bean id="sqlSessionFactory" class="org.mybatis.spring.SqlSessionFactoryBean">
      <property name="dataSource" ref="dataSource" />
    <!-- 将各Java类的简写别名单独放到文件mybatis.xml中,方便修改和管理 -->
     <property name="configLocation" value="classpath:xml/mybatis.xml" />
     <property name="mapperLocations" value="classpath:sql/*.xml" /> </bean>
  <!-- mybatis.xml文件 -->
  <configuration>
      <typeAliases>
          <typeAlias alias="EntityA" type="com.xxxx.model.EntityA" />
          <typeAlias alias="EntityB" type="com.xxxx.model.EntityB" />
          <typeAlias alias="EntityC" type="com.xxxx.model.EntityC" />
          <typeAlias alias="User" type="com.xxxx.model.User" />
      </typeAliases>
  </configuration>

 

2. foreach标签的使用

如果查询接口只有一个参数,参数类型为list,则标签中的collection属性应该设定collection="list";参数类型为数组,则应设定为collection="array"
如果查询接口有多个参数,则最好通过Map来传递各参数。此时,foreach标签的collection属性应设置为,Map中表示集合参数的键。
如上面的代码中,表示集合参数是eventIds,它在Map中的键为"eventIds" ,所以collection="eventIds"
 

处理时间对比

各表数据量在200、300条左右,List<UserEvent>集合记录为13条。
虽然优化后的代码行数有所增加,查询结果解析略微复杂,但是十几条数据的查询已有2秒的差距。
 技术分享

 

记录一次代码优化

标签:time()   gettime   对比   userinfo   erb   最好   调用   type   分享   

原文地址:http://www.cnblogs.com/quiet-snowy-day/p/6166340.html

(0)
(0)
   
举报
评论 一句话评论(0
登录后才能评论!
© 2014 mamicode.com 版权所有  联系我们:gaon5@hotmail.com
迷上了代码!