快速业务通道

审查Java代码的十一种常见错误 - 编程入门网

作者 佚名技术 来源 NET编程 浏览 发布时间 2012-06-21
作符,如下所示:

int x = 4; int y = 5; if (x == y) System.out.println ("Hi"); // This ’if’ test won’t compile. if (x.equals (y)) System.out.println ("Hi");

对象更复杂些,==操作符检查两个引用是否指向同一个对象,而equals方法则实现更专门的相等性检查。

更显得混乱的是由java.lang.Object 所提供的缺省的equals方法的实现使用==来简单的判断被比较的两个对象是否为同一个。

许多类覆盖了缺省的equals方法以便更有用些,比如String类,它的equals方法检查两个String对象是否包含同样的字符串,而Integer的equals方法检查所包含的int值是否相等。

大部分时候,在检查两个对象是否相等的时候你应该使用equals方法,而对于原子类型的数据,你用该使用==操作符。

八、常见错误8#: 混淆原子操作和非原子操作

Java保证读和写32位数或者更小的值是原子操作,也就是说可以在一步完成,因而不可能被打断,因此这样的读和写不需要同步。以下的代码是线程安全(thread safe)的:

public class Example{   private int value; // More code here...   public void set (int x){    // NOTE: No synchronized keyword    this.value = x;   } }

不过,这个保证仅限于读和写,下面的代码不是线程安全的:

public void increment (){   // This is effectively two or three instructions:   // 1) Read current setting of ’value’.   // 2) Increment that setting.   // 3) Write the new setting back.   ++this.value; }

在测试的时候,你可能不会捕获到这个错误。首先,测试与线程有关的错误是很难的,而且很耗时间。其次,在有些机器上,这些代码可能会被翻译成一条指令,因此工作正常,只有当在其它的虚拟机上测试的时候这个错误才可能显现。因此最好在开始的时候就正确地同步代码:

public synchronized void increment (){   ++this.value; }

审查Java代码的十一种常见错误(6)

时间:2011-01-03

九、常见错误9#:在catch 块中作清除工作

一段在catch块中作清除工作的代码如下所示:

OutputStream os = null; try{   os = new OutputStream ();   // Do something with os here.   os.close(); }catch (Exception e){   if (os != null)   os.close(); }

尽管这段代码在几个方面都是有问题的,但是在测试中很容易漏掉这个错误。下面列出了这段代码所存在的三个问题:

1.语句os.close()在两处出现,多此一举,而且会带来维护方面的麻烦。

2.上面的代码仅仅处理了Exception,而没有涉及到Error。但是当try块运行出现了Error,流也应该被关闭。

3.close()可能会抛出异常。

上面代码的一个更优版本为:

OutputStream os = null; try{   os = new OutputStream ();   // Do something with os here. }finally{   if (os != null)    os.close(); }

这个版本消除了上面所提到的两个问题:代码不再重复,Error也可以被正确处理了。但是没有好的方法来处理第三个问题,也许最好的方法是把close()语句单独放在一个try/catch块中。

十、常见错误10#: 增加不必要的catch 块

一些开发者听到try/catch块这个名字后,就会想当然的以为所有的try块必须要有与之匹配的catch块。

C++程序员尤其是会这样想,因为在C++中不存在finally块的概念,而且try块存在的唯一理由只不过是为了与catch块相配对。

增加不必要的catch块的代码就象下面的样子,捕获到的异常又立即被抛出:

try{   // Nifty code here }catch(Exception e){   throw e; }finally{   // Cleanup code here }

不必要的catch块被删除后,上面的代码就缩短为:

try{   // N

凌众科技专业提供服务器租用、服务器托管、企业邮局、虚拟主机等服务,公司网站:http://www.lingzhong.cn 为了给广大客户了解更多的技术信息,本技术文章收集来源于网络,凌众科技尊重文章作者的版权,如果有涉及你的版权有必要删除你的文章,请和我们联系。以上信息与文章正文是不可分割的一部分,如果您要转载本文章,请保留以上信息,谢谢!

分享到: 更多

Copyright ©1999-2011 厦门凌众科技有限公司 厦门优通互联科技开发有限公司 All rights reserved

地址(ADD):厦门软件园二期望海路63号701E(东南融通旁) 邮编(ZIP):361008

电话:0592-5908028 传真:0592-5908039 咨询信箱:web@lingzhong.cn 咨询OICQ:173723134

《中华人民共和国增值电信业务经营许可证》闽B2-20100024  ICP备案:闽ICP备05037997号