调用SQLite数据库update函数后数据未更新，返回值为0

核心问题拆解 #

• WHERE条件逻辑错误：你调用DB.update时用password = ?作为筛选条件，这会只更新当前密码等于新密码的行——完全不符合你要修改指定用户密码的需求，应该用name = ? AND email = ?来定位目标用户。
• 冗余字段写入：ContentValues里添加了name和email，但这两个是查询条件，不需要更新，只需要传入要修改的password字段即可。
• 返回值判断偏差：SQLite的update方法返回受影响的行数，0表示没有匹配到任何行（不是错误），但你的代码只要返回值不是-1就返回true，会导致明明没更新数据却返回true的情况。
• Cursor资源泄漏：查询后的Cursor未关闭，长期运行会导致数据库资源浪费。

修正后的代码 #

public Boolean changePassword(String name, String email, String password) {
    SQLiteDatabase DB = this.getWritableDatabase();
    ContentValues contentValues = new ContentValues();
    // 仅添加需要更新的密码字段
    contentValues.put("password", password);
    
    Cursor cursor = null;
    try {
        cursor = DB.rawQuery("Select * from UserDetails where name = ? and email = ?", new String[]{name, email});
        if (cursor.getCount() > 0) {
            // 用name和email定位要修改的用户
            long result = DB.update("UserDetails", contentValues, "name = ? AND email = ?", new String[]{name, email});
            
            System.out.println("受影响行数= " + result);
            
            // 只有受影响行数>0，才代表更新成功
            return result > 0;
        } else {
            return false;
        }
    } finally {
        // 确保Cursor被关闭，避免资源泄漏
        if (cursor != null) {
            cursor.close();
        }
        DB.close();
    }
}

额外优化建议 #

可以去掉前置的rawQuery查询，直接通过update的返回值判断用户是否存在，减少一次数据库查询，提升效率：

public Boolean changePassword(String name, String email, String password) {
    SQLiteDatabase DB = this.getWritableDatabase();
    ContentValues contentValues = new ContentValues();
    contentValues.put("password", password);
    
    try {
        long result = DB.update("UserDetails", contentValues, "name = ? AND email = ?", new String[]{name, email});
        System.out.println("受影响行数= " + result);
        // result>0：更新成功；result=0：目标用户不存在
        return result > 0;
    } finally {
        DB.close();
    }
}

内容的提问来源于stack exchange，提问作者Diana