Android开发中SQLite数据库电影收藏字段无法更新问题求助

• 核心逻辑错误：传入数据库更新方法的电影名称参数完全不匹配
  原代码在点击事件的遍历循环中，预先定义了开头为Selected items : \n的字符串itemSelected，每检测到一个勾选项，就把当前电影名追加到这个字符串尾部，随后直接把这个持续累加的长字符串传给updateFavorites方法的name形参。
  举个实际运行的例子：如果用户勾选了《阿凡达》和《泰坦尼克号》，第一次循环检测到《阿凡达》时，实际传入的name值是Selected items : \n阿凡达\n；第二次循环检测到《泰坦尼克号》时，传入的name值是Selected items : \n阿凡达\n泰坦尼克号\n。数据库中存储的是单部电影的独立名称，完全无法匹配这类带提示文字、拼接了多部影片的长文本，会触发查询无结果的分支，更新操作根本不会执行。
• 代码位置错误：列表适配器初始化放在数据读取的循环内部
  原代码将排序操作、ArrayAdapter创建、ListView绑定全部写在while(data.moveToNext())的遍历循环里，每读取一行数据库数据就重新创建一次适配器、重新排序整个列表、重新给ListView设置适配器，既造成性能浪费，也容易导致ListView的选中位置错位。
• 状态配置缺失：ListView未开启多选模式
  原代码使用了自带复选框的simple_list_item_multiple_choice布局，但没有手动开启ListView的多选模式，默认的单选模式下isItemChecked()方法无法正确读取复选框的选中状态，会出现勾选了电影但程序完全检测不到选中项的问题。
• 冗余逻辑问题：更新方法内重复查询造成资源浪费
  原updateFavorites方法在执行更新前，先通过rawQuery查询匹配记录是否存在，但SQLiteDatabase.update()本身就会返回受影响的行数，当返回值大于0时就代表更新成功，完全不需要额外执行一次查询，还容易造成Cursor未关闭的内存泄漏。

1. 调整DisplayActivity核心逻辑 #

• 数据库读取完成、列表内容全部填充后，再统一排序、创建适配器、绑定ListView
• 遍历勾选项时，每次只把当前位置的单部电影名称作为参数传给更新方法，不做无关的长字符串拼接
• 显式开启ListView的多选模式，保证选中状态可正常检测
• 操作完成后通过Toast反馈执行结果，清空列表选中状态
  修正后的核心代码：

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_display);

    movieNList = (ListView) findViewById(R.id.moviesLV);
    myDb = new DataBaseHelper(this);
    addFavoritesB = (Button) findViewById(R.id.addButton);

    ArrayList<String> theList = new ArrayList<>();
    Cursor data = myDb.getData();

    if (data.getCount() == 0) {
        Toast.makeText(DisplayActivity.this, "The Database is empty", Toast.LENGTH_SHORT).show();
    } else {
        // 先完整遍历所有数据存入列表
        while (data.moveToNext()) {
            theList.add(data.getString(1));
        }
        // 遍历完成后统一排序
        Collections.sort(theList);
        // 一次性创建适配器并绑定列表
        ListAdapter listAdapter = new ArrayAdapter<>(this, 
            android.R.layout.simple_list_item_multiple_choice, theList);
        movieNList.setAdapter(listAdapter);
        // 开启多选模式，保证复选框选中状态可正常检测
        movieNList.setChoiceMode(ListView.CHOICE_MODE_MULTIPLE);
        // 关闭Cursor避免内存泄漏
        data.close();
    }

    buttonAction();
}

public void buttonAction() {
    addFavoritesB.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View v) {
            int updateSuccessCount = 0;
            for (int i = 0; i < movieNList.getCount(); i++) {
                if (movieNList.isItemChecked(i)) {
                    // 直接取当前位置的单部电影名称作为参数，不做长字符串拼接
                    String currentMovieName = (String) movieNList.getItemAtPosition(i);
                    boolean isUpdateSuccess = myDb.updateFavorites(currentMovieName, "yes");
                    if (isUpdateSuccess) {
                        updateSuccessCount++;
                    }
                }
            }
            // 反馈执行结果
            Toast.makeText(DisplayActivity.this, 
                "成功更新" + updateSuccessCount + "部电影为收藏状态", 
                Toast.LENGTH_SHORT).show();
            // 清空列表选中状态
            movieNList.clearChoices();
        }
    });
}

2. 优化DataBaseHelper中的更新方法 #

去掉冗余的前置查询，直接根据更新操作的返回值判断执行结果，简化逻辑同时避免资源泄漏：

public boolean updateFavorites(String name, String favorites) {
    SQLiteDatabase db = this.getWritableDatabase();
    ContentValues contentValues = new ContentValues();
    contentValues.put("FAVORITES", favorites);
    // 直接执行更新操作，返回值为实际受影响的行数
    int updateRowCount = db.update(
        TABLE_NAME, 
        contentValues, 
        "name=?", 
        new String[]{name}
    );
    // 受影响行数大于0即代表更新成功
    return updateRowCount > 0;
}

• 检查数据库表中存储电影名称的列名是否确实为硬编码的name，如果实际列名和代码不匹配，会导致更新条件完全失效
• 检查存量测试数据中，电影名称是否存在首尾空格、隐藏换行符，避免因为字符串不完全匹配导致更新失效
• 调试时可通过Log打印实际传入更新方法的电影名称、更新操作的返回值，快速定位参数不匹配的问题

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