Java告警:Finally 块未正常完成

最近开发写java发现好多的警告,Finally 块未正常完成 我不太习惯有这么多的警告,所以我就会调整一下。参考代码如下:
//取得序列号
  public static long get_seq(String seq,String sDataLink){
    Connection conn = db.getConn();
    PreparedStatement stmt = null;
    ResultSet rs = null;
    long lSeq = -1;
    try {
      stmt = conn.prepareStatement("SELECT " + seq +
                                   ".NEXTVAL iSeq FROM DUAL" + sDataLink);
      rs = stmt.executeQuery();
      if (rs.next()) {
        lSeq = rs.getLong("iSeq");
      }
    }
    catch (Exception e) {
      lSeq = -1;
      System.out.println("In pubBusi.get_seq(" + seq + "):" + e);
    }
    finally {
      db.CloseState(rs, stmt);
      db.CloseConnect(conn);
      return (lSeq);
    }
  }


原因:报Finally 块未正常完成 warning原因是,return 不应该写在finally中
    finally {
      db.CloseState(rs, stmt);
      db.CloseConnect(conn);
      return (lSeq);
    }
改为
    finally {
      db.CloseState(rs, stmt);
      db.CloseConnect(conn);
    }
    return (lSeq);
这样就不会有这个警告了

其实try里的内容无论出错与否,最后getints返回的永远是0

测试代码,比较常用的做法是类似

try{
……
return 1;
}catch(Exception e){
e.printStackTrace();
}finally{

}
return 0;
}

如果在return 1之前出错,则进入catch和finally,并最终跳出try-catch-finally,返回0
如果try内没有错,则进入finally并返回1

这是eclipse3后的一个特点,就是认为在finally块中不应该存在return语句。

只是一个警告。

不查看该警告的设置:Eclipse中Window->Preferences->Java->Compiler->Errors/Warnings->Potential programming problems中找到'finally' does not complete normally然后把它的警告级别改为Ignore就可以了。


那么在 finally 中存在 return 是否正确的呢?
正确但是不推荐,因为try{ }里面正常return的语句就无法执行了

 

posted @ 2008-06-01 08:52  小草  阅读(1736)  评论(0编辑  收藏  举报
Google+