17370845950

本文详解 sqlite 错误代码 1（`sqlite_error: near "mytableofclothes"`）的根本原因——非法 sql 语句执行，重点指出 `database.query()` 等方法**不能直接执行建表语句**，而需调用 `execsql()`；同时修复 `drop table if exist` 拼写错误、表结构注册缺失及游标资源泄漏等关键问题。

该崩溃日志看似指向表名拼写错误（如 "myTableOfClothes"），但实际根源在于 SQLite 执行逻辑误用：错误日志中 while compiling: myTableOfClothes 表明某处代码正尝试将一个纯表名字符串（而非合法 SQL 语句）交由 SQLite 编译——这绝非 CREATE TABLE 语句，而是极可能在数据库 Helper 的 onCreate() 或 onUpgrade() 中遗漏了 execSQL() 调用，或在其他位置错误地将表名当作 SQL 执行。

✅ 核心问题定位与修复

1. DROP TABLE IF EXIST → 必须为 DROP TABLE IF EXISTS

您在 MyDataBaseContract 中的定义存在拼写错误：

public static final String DROP_CLOTHE_TABLE = "DROP TABLE IF EXIST " + CLOTHE_TABLE_NAME; // ❌ 错误！

应修正为：

public static final String DROP_CLOTHE_TABLE = "DROP TABLE IF EXISTS " + CLOTHE_TABLE_NAME; // ✅ 正确
public static final String DROP_USER_TABLE = "DROP TABLE IF EXISTS " + USER_TABLE_NAME;

SQLite 对关键字大小写不敏感，但 EXIST 是无效关键字，会导致 SQLITE_ERROR。

2. 表结构未在 SQLiteOpenHelper 中创建

当前 MyDataBaseContract 仅定义了建表语句字符串（CLOTHE_TABLE_STRUCTURE 等），但未在 MyDataBaseHelper 的 onCreate() 中执行它们。这是崩溃的真正起点。请确保您的 MyDataBaseHelper 类类似如下：

public class MyDataBaseHelper extends SQLiteOpenHelper {
    public MyDataBaseHelper(Context context) {
        super(context, MyDataBaseContract.DATABASE_NAME, null, MyDataBaseContract.DB_VERSION);
    }

    @Override
    public void onCreate(SQLiteDatabase db) {
        // ✅ 关键：必须使用 execSQL() 执行 DDL 语句（CREATE/DROP）
        db.execSQL(MyDataBaseContract.USER_TABLE_STRUCTURE);
        db.execSQL(MyDataBaseContract.CLOTHE_TABLE_STRUCTURE);
    }

    @Override
    public void onUpgrade(SQLiteDatabase db, int oldVersion, int newVersion) {
        // 升级时安全删除旧表并重建（注意：此操作会清空数据）
        db.execSQL(MyDataBaseContract.DROP_USER_TABLE);
        db.execSQL(MyDataBaseContract.DROP_CLOTHE_TABLE);
        onCreate(db); // 重建新结构
    }
}

⚠️ 重要提醒：query()、insert()、update() 等方法仅用于 DML（数据操作）；CREATE TABLE、DROP TABLE 等 DDL（数据定义）语句必须使用 execSQL()。

3. getAllClothesByOwnerId() 存在严重性能与逻辑缺陷

当前实现先查全表再内存过滤，且未使用 WHERE 子句，导致：

• 性能低下（尤其数据量大时）；
• cursor.getColumnIndex(...) 可能返回 -1（列不存在时），引发 IllegalArgumentException；
• 游标未在异常路径下关闭，存在泄漏风险。

✅ 优化后代码：

public ArrayList getAllClothesByOwnerId(int ownerId) {
    ArrayList clothes = new ArrayList<>();
    String selection = MyDataBaseContract.CLOTHE_OWNER_ID + " = ?";
    String[] selectionArgs = {String.valueOf(ownerId)};

    Cursor cursor = null;
    try {
        cursor = database.query(
            MyDataBaseContract.CLOTHE_TABLE_NAME,
            null,           // columns: null → select all
            selection,      // WHERE clause
            selectionArgs,  // ? binding
            null, null, null
        );

        if (cursor != null && cursor.moveToFirst()) {
            int idIdx = cursor.getColumnIndexOrThrow(MyDataBaseContract.CLOTHE_ID);
            int warmthIdx = cursor.getColumnIndexOrThrow(MyDataBaseContract.CLOTHE_WARMTH);
            int typeIdx = cursor.getColumnIndexOrThrow(MyDataBaseContract.CLOTHE_TYPE);
            int colourIdx = cursor.getColumnIndexOrThrow(MyDataBaseContract.CLOTHE_COLOUR);
            int nameIdx = cursor.getColumnIndexOrThrow(MyDataBaseContract.CLOTHE_NAME);
            int ownerIdx = cursor.getColumnIndexOrThrow(MyDataBaseContract.CLOTHE_OWNER_ID);

            do {
                Clothe clothe = new Clothe(
                    cursor.getInt(idIdx),
                    cursor.getString(warmthIdx),
                    cursor.getString(typeIdx),
                    cursor.getString(colourIdx),
                    cursor.getString(nameIdx),
                    cursor.getInt(ownerIdx)
                );
                clothes.add(clothe);
            } while (cursor.moveToNext());
        }
    } finally {
        if (cursor != null && !cursor.isClosed()) cursor.close();
    }
    return clothes;
}

4. 其他关键注意事项

• onResume() 中重复打开数据库风险：LoginActivity.onResume() 每次切回前台都调用 openDataBase()，但未检查是否已打开。建议在 openDataBase() 内添加判空逻辑：

  public void openDataBase() {
      if (database == null || !database.isOpen()) {
          database = myDataBaseHelper.getWritableDatabase();
      }
  }

• closeDataBase() 后不应再访问 database：btn_login 点击逻辑中 myDataBaseManager.closeDataBase() 后仍可能触发 doesLoginExist() 等操作，导致 IllegalStateException。应在 startActivity() 后统一管理生命周期，或改用 try-with-resources 模式。
• CLOTHE_IMAGE_INDEX 类型建议：当前为 TEXT，若存储的是资源 ID（如 R.drawable.xxx），应改为 INTEGER 更合理。

✅ 总结：四步快速修复清单

1. 修正拼写：IF EXIST → IF EXISTS；
2. 补全建表逻辑：在 MyDataBaseHelper.onCreate() 中调用 db.execSQL(...) 创建两张表；
3. 重写查询方法：所有 query() 调用必须指定有效 WHERE 条件，避免全表扫描，并使用 getColumnIndexOrThrow() + try-finally 安全关闭游标；
4. 校验生命周期：确保 SQLiteDatabase 实例在使用前已打开、使用后正确关闭，避免跨生命周期调用。

完成以上修改后，重新安装应用（旧数据库会被 onCreate() 重建），崩溃将彻底消失。SQLite 错误代码 1 的本质，往往是“把不该当 SQL 执行的字符串送进了编译器”——回归 SQL 执行规范，是解决此类问题的黄金法则。