憨厚生

----Java's Slave----
***Java's Host***
  BlogJava :: 首页 :: 新随笔 :: 联系 :: 聚合  :: 管理 ::
  165 随笔 :: 17 文章 :: 90 评论 :: 0 Trackbacks
java编程常见BUG---不完善的异常处理

  这样的错误以前我也犯过，也见过不少人这样的写法！下面我也举个例子：
  

 public void writeFile(File f) {
  String content = null;
  try {
   byte[] b = new byte[1024];
   FileInputStream in = new FileInputStream(f);
   in.read(b);
   content = new String(b);
  } catch (Exception e) {
   System.out.println(e.getMessage());
  }

  if (content.indexOf("hello") > -1) {
   System.out.println("yes");
  } else {
   System.out.println("no");
  }
 }


 上面是个简单的方法,代码中有个隐藏的bug。我在维护一个系统的时候就遇到类似的代码，实际中类似的BUG隐藏
的更深！在对系统业务和代码不是很很熟悉的情况下，我推荐如下写法：

 1 public void writeFile(File f) {
 2  String content = null;
 3  try {
 4   byte[] b = new byte[1024];
 5   FileInputStream in = new FileInputStream(f);
 6   in.read(b);
 7   content = new String(b);
 8  } catch (Exception e) {
 9   content="";
10   //如果异常发生的话,content可能为空
11   //下面对content的操作就有可能发生NullPointerException异常
12   System.out.println(e.getMessage());
13  }
14  //下面操作有可能发生NullPointerException异常
15  if (content.indexOf("hello") > -1) {
16   System.out.println("yes");
17  } else {
18   System.out.println("no");
19  }
20 }


 一般来说异常处理不推荐直接system.out.println打印出来！
 几条建议：
 如果无法处理某个异常，那就不要捕获它。
　　☆ 如果捕获了一个异常，请不要胡乱处理它。
　　☆ 尽量在靠近异常被抛出的地方捕获异常。
　　☆ 在捕获异常的地方将它记录到日志中，除非您打算将它重新抛出。
　　☆ 按照您的异常处理必须多精细来构造您的方法。
　　☆ 需要用几种类型的异常就用几种，尤其是对于应用程序异常。
　　☆ 把低层次的异常封装成层次较高程序员较容易理解的异常。
　　☆ 尽量输出造成异常的完整数据
　　☆ 尽量捕获具有特定含义的异常：比如SqlException，而不是简单地捕获一个Exception


  希望对大家有帮助！

参考：
http://www.blogjava.net/usherlight/archive/2006/10/23/76782.html

posted on 2009-12-09 16:59 二胡 阅读(390) 评论(0)  编辑  收藏 所属分类: Java

只有注册用户登录后才能发表评论。
网站导航: