无法访问的代码 - 更新现有行

时间:2014-06-06 16:10:02

标签: java unreachable-code

在下面的代码中db.setTransactionSuccessful();给出错误无法访问的代码。谁能告诉我如何解决这个问题呢?

public boolean updateDiaryEntry(String title, long rowId)
        {

            ContentValues newValue = new ContentValues();
            newValue.put(Constants.TITLE_NAME, title);

            db.beginTransaction();


            return db.update(Constants.TABLE_NAME , newValue , Constants.KEY_ID + "= ?" , 
                new String[]{ Double.valueOf(rowId).toString() })>0;

            db.setTransactionSuccessful();
            db.endTransaction();    

        }

2 个答案:

答案 0 :(得分:2)

您正在返回该行之前的行,该行将退出该函数。

答案 1 :(得分:1)

从函数返回之后有两行代码,这些行永远不会被执行,因为你已经离开了这个函数。这就是您获得无法访问的代码消息的原因。您不希望在return语句后面有代码行:

return db.update(Constants.TABLE_NAME , newValue , Constants.KEY_ID + "= ?" , 
            new String[]{ Double.valueOf(rowId).toString() })>0;   //returned from function on this line

db.setTransactionSuccessful(); //therefore you never get to this line
db.endTransaction();  

相反,你可能想做这样的事情:

db_result = db.update(Constants.TABLE_NAME , newValue , Constants.KEY_ID + "= ?" , 
            new String[]{ Double.valueOf(rowId).toString() })>0;

if(db_result){
     db.setTransactionSuccessful(); //Now you can do this conditional on the result of the update
}
db.endTransaction();
return db_result;

通过创建一个变量来存储更新数据库的结果,您可以在从函数返回之前执行与数据库相关的清理/关闭函数。