마이바티스 리팩토링하기 1

이건 시리즈물이다

그렇다

정확히 코드를 리팩토링하는 과정인것이다

하는이유는? 첫째는 내가 만지기 빡씨다는점이다...ㅂㄷㅂㄷ

따로 설명은 넣지않을것이다 왜냐? 설명을 넣을 필요도 없이 간단하기 때문이지!

---

Before

    <!-- 퇴근 기록하기 -->
    <insert id="updateCommuteRecord" parameterType="com.tistory.vessel.api.vo.CommuteRecordVO">
        UPDATE commute_record
        SET pad = pad
        <if test="leave != null and leave > 0">
            , `leave` = FROM_UNIXTIME(#{leave})
        </if>
        <if test="leave_beacon != null">
            <if test="leave_beacon != null">
                , leave_beacon = #{leave_beacon}
            </if>
            <if test="leave_beacon == null">
                , leave_beacon = null
            </if>
        </if>
        <if test="come_beacon != null">
            <if test="come_beacon != null">
                , come_beacon = #{come_beacon}
            </if>
            <if test="come_beacon == null">
                , come_beacon = null
            </if>
        </if>
        <if test="come_match == null">
            <if test="come_match != -2">
            , come_match = #{come_match}
            </if>
            <if test="come_match == -2">
            , come_match = null
            </if>
        </if>
        <if test="leave_match != null">
            <if test="leave_match != -2">
            , leave_match = #{leave_match}
            </if>
            <if test="leave_match == -2">
            , leave_match = null
            </if>
        </if>
        <if test="valid != null">
            , valid = #{valid}
        </if>
        <if test="id != null and id != -1">
            WHERE id = #{id}
        </if>
        <if test="id == null or id == -1">
            WHERE 1=0
        </if>
    </insert>

after

<insert id="updateCommuteRecord" parameterType="com.tistory.vessel.api.vo.CommuteRecordVO">
    UPDATE commute_record
        <trim prefix="SET" suffixOverrides=",">
            <if test="leave != null">`leave` = FROM_UNIXTIME(#{leave}),</if>
            <if test="leave_beacon != null">leave_beacon = #{leave_beacon},</if>
            <if test="come_beacon != null">come_beacon = #{come_beacon},</if>
            <if test="come_match != null">come_match = #{come_match},</if>
            <if test="leave_match != null">leave_match = #{leave_match},</if>
            <if test="valid != null">valid = #{valid},</if>
            <if test="id != null">id = #{id},</if>
        </trim>
<insert>