A股上市公司传智教育(股票代码 003032)旗下技术交流社区北京昌平校区

 找回密码
 加入黑马

QQ登录

只需一步,快速开始

© 小兮网络 初级黑马   /  2013-12-28 15:41  /  1375 人查看  /  3 人回复  /   1 人收藏 转载请遵从CC协议 禁止商业使用本文

每天在写Java程序,其实里面有一些细节大家可能没怎么注意,这不,有人总结了一个我们编程中常见的问题。虽然一般没有什么大问题,但是最好别这样做。另外这里提到的很多问题其实可以通过Findbugs来帮我们进行检查出来。
字符串连接误用
错误的写法:

    String s = "";  
  • for (Person p : persons) {  
        s += ", " + p.getName();  
  • }  
  • s = s.substring(2); //remove first comma
正确的写法:

    StringBuilder sb = new StringBuilder(persons.size() * 16); // well estimated buffer
  • for (Person p : persons) {  
        if (sb.length() > 0) sb.append(", ");  
  •     sb.append(p.getName);  
  • }
错误的使用StringBuffer
错误的写法:

    StringBuffer sb = new StringBuffer();  
  • sb.append("Name: ");  
    sb.append(name + '\n');  
  • sb.append("!");  
    ...  
  • String s = sb.toString();
问题在第三行,append char比String性能要好,另外就是初始化StringBuffer没有指定size,导致中间append时可能重新调整内部数组大小。如果是JDK1.5最好用StringBuilder取代StringBuffer,除非有线程安全的要求。还有一种方式就是可以直接连接字符串。缺点就是无法初始化时指定长度。
正确的写法:

    StringBuilder sb = new StringBuilder(100);  
  • sb.append("Name: ");  
    sb.append(name);  
  • sb.append("\n!");  
  • String s = sb.toString();
或者这样写:
  • String s = "Name: " + name + "\n!";
测试字符串相等性
错误的写法:

    if (name.compareTo("John") == 0) ...  
  • if (name == "John") ...  
    if (name.equals("John")) ...  
  • if ("".equals(name)) ...
上面的代码没有错,但是不够好。compareTo不够简洁,==原义是比较两个对象是否一样。另外比较字符是否为空,最好判断它的长度。
正确的写法:

    if ("John".equals(name)) ...  
  • if (name.length() == 0) ...  
  • if (name.isEmpty()) ...
数字转换成字符串
错误的写法:

    "" + set.size()  
  • new Integer(set.size()).toString()  
正确的写法:
  • String.valueOf(set.size())
利用不可变对象(Immutable)
错误的写法:

    zero = new Integer(0);  
  • return Boolean.valueOf("true");



评分

参与人数 1技术分 +1 收起 理由
乔兵 + 1

查看全部评分

3 个回复

倒序浏览
赞一个  呵呵 
回复 使用道具 举报
学习了,支持一个
回复 使用道具 举报
顶一个!
回复 使用道具 举报
您需要登录后才可以回帖 登录 | 加入黑马