如何在本地数据库中对比ID与表字段值？附PlanID存在性校验实现

我来帮你梳理下如何正确实现这个需求，顺便优化你当前的代码实现——你的核心需求是检查本地数据库表中的planid是否存在，存在返回true，不存在返回false，目前的代码已经有了雏形，但可以从安全性、健壮性和代码规范上做不少改进：

先说说当前代码的几个小问题 #

• 直接拼接SQL字符串存在SQL注入风险：虽然这里planID是int类型风险较低，但这是数据库操作的不良习惯
• 游标处理不够严谨：c.moveToFirst()如果游标为空可能会引发异常，而且getCount()的效率不如直接判断游标是否有效
• finally块里的数据库关闭代码不完整：db...应该补充为db.close()，否则会导致数据库连接泄漏

优化后的完整实现 #

public Boolean isPlanExisted(int planID) {
    SQLiteDatabase db = null;
    Cursor cursor = null;
    try {
        // 使用参数化查询，避免SQL注入，同时适配字段类型
        String query = "SELECT 1 FROM " + Table_Plan_App + " WHERE " + PAPP_Plan_ID + " = ?";
        db = this.getReadableDatabase();
        
        // 先判断数据库是否成功打开
        if (db == null || !db.isOpen()) {
            return false;
        }
        
        // 用参数替换占位符，自动处理类型转换
        cursor = db.rawQuery(query, new String[]{String.valueOf(planID)});
        
        // moveToFirst()直接返回布尔值：有数据返回true，无则返回false，比getCount()更高效
        return cursor.moveToFirst();
    } catch (Exception e) {
        Log.d("eEmp/DBPlanExisted", "检查plan存在性出错：" + e.toString());
        return false;
    } finally {
        // 确保资源被正确释放，避免内存泄漏
        if (cursor != null) {
            cursor.close();
        }
        if (db != null && db.isOpen()) {
            db.close();
        }
    }
}

关键优化点说明 #

• 参数化查询：用?作为占位符传递参数，彻底杜绝SQL注入风险，这是数据库操作的最佳实践
• 高效的结果判断：cursor.moveToFirst()不需要遍历整个结果集计数，直接判断是否有匹配数据，性能更好
• 完善的资源释放：finally块中确保游标和数据库连接都被关闭，避免资源泄漏
• 代码规范优化：把方法名改成Java驼峰命名的isPlanExisted，更符合行业规范
• 清晰的异常日志：补充了明确的错误描述，方便后续排查问题

调用方式无需修改 #

你原来的调用代码可以继续正常使用：

boolean planExists = dbHandler.isPlanExisted(app_plan_id);

额外提醒 #

• 如果你的PAPP_Plan_ID对应的是整数类型字段，不需要给planID加引号（你原来的代码里加了\""，这会把整数转成字符串查询，可能导致类型不匹配，参数化查询会自动处理类型适配）
• 确保Table_Plan_App和PAPP_Plan_ID这两个常量是正确的表名和列名，避免拼写错误导致的查询失败

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