代码中的感悟

QQ账号登录	立即注册>>

admin

1244 主题	544 听众	1万金钱

管理员

TA的每日心情

	衰 2021-2-2 11:21

签到天数: 36 天

[LV.5]常住居民I

电梯直达

^#

发表于 2014-04-07 17:31:53 |只看该作者 |正序浏览

废话少说先上代码：
复制内容到剪贴板
代码:
int iPage = 0;
      int iPerPage = 20;
      String sName = "admin";
      String sSql = "select user.c_name name, (select dept.c_name from db_aty_t_aty_dept dept where dept.c_id=user.c_dept) dept, user.c_gzdh gzdh from dt_aty.t_aty_user user where user.c_dept like "
            + sName + " and n_valid = 1 and user.c_dept != -1 limit " + iPage + "," + iPerPage;
大家写的代码是不是这个样子的呢？
我相信很多人会说，我的代码比这个写的好多了。
好的看下面的代码：
复制内容到剪贴板
代码:
int iPage = 0;
      int iPerPage = 20;
      String sName = "admin";
      String sSql =  "select "
                  + "    user.c_name name, "
                  + "    (select dept.c_name "
                  + "       from db_aty_t_aty_dept dept "
                  + "    where "
                  + "       dept.c_id=user.c_dept) dept,"
                  + "    user.c_gzdh gzdh "
                  + "from "
                  + "    dt_aty.t_aty_user user "
                  + "where "
                  + "    user.c_dept like " + sName
                  + "    and "
                  + "    n_valid = 1 "
                  + "    and "
                  + "    user.c_dept != -1 "
                  + "limit " + iPage + "," + iPerPage;
那么这个代码大家满意么？
还是有人会说，还可以改进，那么好的。
继续看代码：
复制内容到剪贴板
代码:
int iPage = 0;
      int iPerPage = 20;
      String sName = "admin";
      String sSql =  "select "
                  + "    user.c_name name, "
                  + "    (select dept.c_name "
                  + "       from db_aty_t_aty_dept dept "
                  + "    where "
                  + "       dept.c_id=user.c_dept) dept,"
                  + "    user.c_gzdh gzdh "
                  + "from "
                  + "    dt_aty.t_aty_user user "
                  + "where "
                  + "    user.c_dept like ?"
                  + "    and "
                  + "    n_valid = 1 "
                  + "    and "
                  + "    user.c_dept != -1 "
                  + "limit ?,?";
这个可以了吧！还是有人会说，我还可以改进，那么恭喜你。
看最后一段代码：
复制内容到剪贴板
代码:
int iPage = 0;
      int iPerPage = 20;
      String sName = "admin";
      StringBuilder sbSql = new StringBuilder();
      sbSql.append("select ");
      sbSql.append("  user.C_name name, ");
      sbSql.append("  (select ");
      sbSql.append("    dept.c_name ");
      sbSql.append(" from ");
      sbSql.append("    db_aty.t_aty_dept dept ");
      sbSql.append(" where ");
      sbSql.append("    dept.c_id=user.c_dept) dept, ");
      sbSql.append("  user.c_gzdh gzdh ");
      sbSql.append("from ");
      sbSql.append("  db_aty.t_aty_user user ");
      sbSql.append("where ");
      sbSql.append("  user.c_dept like ? ");
      sbSql.append("  and ");
      sbSql.append("  n_valid = 1 ");
      sbSql.append("  and ");
      sbSql.append("  user.c_dept != -1 ");
      sbSql.append("limit ?,?");
看过这四段代码大家，可能会有疑问，这是为什么呢？
我总结了几点：
1、大家可能注意到了变量的命名方式，可能大家平时写代码的时候注意不到这个，不过这也是个人的习惯问题的。其实按照自己的命名规则来写没有问题，问题在于我们在工作中写的代码不会只有自己看的。后期维护的人员也会看的，这样可以让维护人员直观的了解变量的数据类型。

2、为什么不用字符拼接而是用通配符？很多的答案我相信都是一致的，没错就是防注入。但是可能很多意识不到还有一个问题就是数据库解析。数据库接受到程序传入的SQL语句的时候，如果是拼接的字符串数据库会自动进行解析工作，那么大量SQL涌入的时候，效率就可想而知了。

3、为什么摒弃了最熟悉的String类型而是用StringBuilder类型。我们首先来看String类型定义的特殊性，如果用new的方式我就不用说了，大家用new的方式定义String类型的时候比较少，即使是那么定义了。大家一定会知道只要是new就一定会创建一个字符串。

字符串拼接的过程，先搜索字符串池中字符串对象。如果存在就会把该字符串地址赋值给定义的字符串变量。如果不存在就会重新创建对象。在这种拼接的过程中创建很多对象，那么对性能要求比较高，而且效率很低下。
还有人可能问了，为什么不是StringBuffer，因为这里不需要线程安全，只要效率。所以选择StringBuilder就可以了。

4、最后一个可能是大家不太注意的问题就是，SQL的格式。一段良好的代码它的可读性是最珍贵的。说一个人有多牛，不是说这个人可以写出别人看不懂的代码，而是看有多少人可以读懂他的代码。所以SQL的格式也非常重要。

看沙发吧！我不知道我写帖子里面带有什么词语了，就是不能通过。

这次就说这么多，我希望大家多多参与进来，不过看昨天大家回复的帖子。很多人愿意被动的接受知识，而不愿意去主动探索去寻找知识。这样的态度是不对的，希望这个帖子可以让更多参与进来，不是我一个人写，而是所有人写。写出自己对编程的思考，自己的想法，自己的技巧。

科帮网 1、本主题所有言论和图片纯属会员个人意见，与本社区立场无关
2、本站所有主题由该帖子作者发表，该帖子作者与科帮网享有帖子相关版权
3、其他单位或个人使用、转载或引用本文时必须同时征得该帖子作者和科帮网的同意
4、帖子作者须承担一切因本文发表而直接或间接导致的民事或刑事法律责任
5、本帖部分内容转载自其它媒体，但并不代表本站赞同其观点和对其真实性负责
6、如本帖侵犯到任何版权问题，请立即告知本站，本站将及时予与删除并致以最深的歉意
7、科帮网管理员和版主有权不事先通知发贴者而删除本文