如何通过上下文菜单删除SQLite数据库项？删项异常问题求助

我来帮你排查这个问题，你的代码里有两个关键错误，导致不管你选择哪一项删除，最终都会出现不符合预期的删除行为：

1. 上下文菜单未正确获取选中项的位置与数据 #

你在onContextItemSelected方法里直接使用了itemPos变量，但这个变量并没有关联到当前上下文菜单触发的条目位置。正确的做法是从menuInfo中提取当前选中项的上下文信息，拿到对应的位置和数据。

修复后的onContextItemSelected代码示例：

@Override 
public boolean onContextItemSelected(MenuItem menuItem) {
    // 获取当前选中项的上下文信息
    AdapterContextMenuInfo menuInfo = (AdapterContextMenuInfo) menuItem.getMenuInfo();
    int selectedPosition = menuInfo.position;
    
    // 从Adapter中获取选中的条目（这里假设你的Adapter数据源是包含ID的自定义对象，比如ToDoItem）
    ToDoItem selectedItem = (ToDoItem) mAdapter.getItem(selectedPosition);
    
    switch (menuItem.getItemId()) { 
        case R.id.delete_menu_item: 
            // 传入该条目对应的数据库ID进行删除
            ToDoDB.deleteData(selectedItem.getId()); 
            // 从Adapter中移除选中的条目
            mAdapter.remove(selectedItem); 
            return true; 
        case R.id.quit_menu_item: 
            finish(); 
            return true; 
        default: 
            return super.onContextItemSelected(menuItem); 
    } 
}

2. 数据库删除逻辑存在严重错误 #

你的deleteData方法逻辑完全不对：

• 你先查询了全表数据，但根本没用到这个Cursor，只是判断是否有数据就执行删除；
• 你把列表项的toString()结果当作数据库的KEY_ID传入，但KEY_ID应该是自增的整数ID，和列表项内容完全不匹配；
• 数据库关闭的位置不合理，容易导致资源泄漏。

修复后的deleteData方法：

// 改为传入整数类型的ID，更符合数据库ID的类型
public int deleteData(int itemId) { 
    SQLiteDatabase database = this.getWritableDatabase(); 
    int deleteResult = 0;
    try {
        // 直接根据ID精准删除对应的数据行
        deleteResult = database.delete(TABLE_NAME, KEY_ID + "=?", new String[]{String.valueOf(itemId)}); 
    } finally {
        // 确保数据库连接最终被关闭，避免泄漏
        if (database != null && database.isOpen()) {
            database.close();
        }
    }
    return deleteResult; 
}

额外建议：使用自定义对象作为数据源 #

为了让列表项和数据库数据更好地关联，建议创建一个自定义的ToDoItem类来存储每条数据的ID和内容：

public class ToDoItem {
    private int id;
    private String content;

    public ToDoItem(int id, String content) {
        this.id = id;
        this.content = content;
    }

    // Getter方法
    public int getId() {
        return id;
    }

    public String getContent() {
        return content;
    }

    // ListView显示时会调用toString，返回内容即可
    @Override
    public String toString() {
        return content;
    }
}

这样你的Adapter就可以使用List<ToDoItem>作为数据源，从数据库查询数据时，把Cursor中的ID和内容封装成ToDoItem对象，后续删除时就能准确拿到对应条目的数据库ID。

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