每天在写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");
|