维护有感1--字符串拼接老问题

   上个月拿自己的一份大学时候的代码报名javaeye的招聘，但是被robbin指出了string直接相加的毛病。当然现在我一直都不是用直接用+号了，大学的时候没有考虑这么多问题。最近在维护一个项目，修改里面的功能和代码，发现问题多多，千奇百怪。
以后每天写出点供自己警惕，同时有没发现其“精髓”的地方大家讨论。

其中就象我以前一样 的错误  sql 语句用 + 号 大行其道

"select xmbh from wcrxx where xmbh='"
                + request.getParameter("cgbh") + "' and lx='2'";

有些代码知道自己应该用StringBuffer，但是看看下面，真是不知是怎么想的

 if (strBmbh != null && !("".equals(strBmbh)))
        {
            strQueryCondition = tempQuery.append(" and ").append(
                    " lwzz.bmbh like '").append(strBmbh).append(
                    "%' ").toString();
        }

        if (strCglbm != null && !("".equals(strCglbm)))
        {
            strQueryCondition = tempQuery.append(" and ").append(
                    " lwzz.cglbm like '%").append(strCglbm)
                    .append("%' ").toString();
        }
        if (strXklym != null && !("".equals(strXklym)))
        {
            strQueryCondition = tempQuery.append(" and ").append(
                    " lwzz.xklym like '%").append(strXklym)
                    .append("%' ").toString();
        }

        if (strXkml != null && !("".equals(strXkml)))
        {
            strQueryCondition = tempQuery.append(" and ").append(
                    " lwzz.xkml like '%").append(strXkml)
                    .append("%' ").toString();
        }

真可谓每if 必 toString(), 真不知其目的，难道是为了用了StringBuffer我也要创造对象？
呵呵 今天发牢骚先到这，毕竟以后还长，慢慢贴！

评论

1 楼 xfei6868 2009-12-07  

而且有时候发现这样的语句：

sb.append("CGMC=" + request.getParameter("cgmc") + "',");

看上去好像是用了StringBuffer来连接字符串，但是同样的问题是里面用的是+号拼接，同样产生了很多的String对象。
可以看出这个语句的意图其实是想让相关联的写在一起。
其实可以这样编写

sb.append("CGMC='").append(request.getParameter("cgmc")).append("',");

如果你的代码中正有有着这样的问题，其实修改起来也很简单，用Eclipse的替换功能
把' + ' 替换成 ').append('(单引号内为替换部分，+两侧是否有空格，看具体代码)。