Question

我在网上关注教程，但我得到了一个非法的州例外。链接到教程： http://www.developerfeed.com/android/tutorial/building-todo-list-app-android-using-sqlite

这是数据库类：

public class TaskerDbHelper extends SQLiteOpenHelper {

private static final int DATABASE_VERSION = 1;

// Database Name
private static final String DATABASE_NAME = "taskerManager";

// tasks table name
private static final String TABLE_TASKS = "tasks";

// tasks Table Columns names
private static final String KEY_ID = "id";
private static final String KEY_TASKNAME = "taskName";
private static final String KEY_STATUS = "status";

public TaskerDbHelper(Context context) {
    super(context, DATABASE_NAME, null, DATABASE_VERSION);
}

@Override
public void onCreate(SQLiteDatabase db) {

    String sql = "CREATE TABLE IF NOT EXISTS " + TABLE_TASKS + " ( "
            + KEY_ID + " INTEGER PRIMARY KEY AUTOINCREMENT, " + KEY_TASKNAME
            + " TEXT, " + KEY_STATUS + " INTEGER)";
    db.execSQL(sql);

    db.close();
}

@Override
public void onUpgrade(SQLiteDatabase db, int oldV, int newV) {
    // Drop older table if existed
    db.execSQL("DROP TABLE IF EXISTS " + TABLE_TASKS);
    // Create tables again
    onCreate(db);
}

// Adding new task
public void addTask(Task task) {
    SQLiteDatabase db = this.getWritableDatabase();

    ContentValues values = new ContentValues();
    values.put(KEY_TASKNAME, task.getTaskName()); // task name
    // status of task- can be 0 for not done and 1 for done
    values.put(KEY_STATUS, task.getStatus());

    // Inserting Row
    db.insert(TABLE_TASKS, null, values);
    db.close(); // Closing database connection
}

public List<Task> getAllTasks() {
    List<Task> taskList = new ArrayList<Task>();
    // Select All Query
    String selectQuery = "SELECT  * FROM " + TABLE_TASKS;

    SQLiteDatabase db = this.getWritableDatabase();
    Cursor cursor = db.rawQuery(selectQuery, null);

    // looping through all rows and adding to list
    if (cursor.moveToFirst()) {
        do {
            Task task = new Task();
            task.setId(cursor.getInt(0));
            task.setTaskName(cursor.getString(1));
            task.setStatus(cursor.getInt(2));
            // Adding contact to list
            taskList.add(task);
        } while (cursor.moveToNext());
    }

    // return task list
    return taskList;
}

public void updateTask(Task task) {
    // updating row
    SQLiteDatabase db = this.getWritableDatabase();
    ContentValues values = new ContentValues();
    values.put(KEY_TASKNAME, task.getTaskName());
    values.put(KEY_STATUS, task.getStatus());
    db.update(TABLE_TASKS, values, KEY_ID + " = ?",new String[] {String.valueOf(task.getId())});
    db.close();
}

}

以下是日志文件的内容：

08-14 14:21:42.133: E/AndroidRuntime(10366): java.lang.RuntimeException: Unable to start activity ComponentInfo{com.example.tasker/com.example.tasker.ViewTask}: java.lang.IllegalStateException: attempt to re-open an already-closed object: SQLiteDatabase: /data/data/com.example.tasker/databases/taskerManager

Answer 1

您的代码中存在两个问题：

A）您不得在onCreate中关闭数据库。该方法是打开数据库的一部分，因此db应该在之后打开。 source-code

public void onCreate(SQLiteDatabase db) {
    ...
    db.close();
}

B）在每项任务结束时关闭数据库。

public void updateTask(Task task) {
    db = get..
    ...
    db.close();
}

但在执行此操作时，您仍然通过Cursor

使用相同的数据库

db = db.get..
Cursor cursor = db.rawQuery
while (cursor.moveToNext() {
     updateTask();
}
db.close();

SQLiteOpenHelper将始终为您提供相同的db对象，因为只有一个数据库。在一种方法中关闭连接而在另一种方法中使用它也可能导致您的问题。

你可以通过使用外部循环中的现有db对象或者根本不关闭updateTask中的数据库来解决任务问题，但实际上最好不要关闭数据库

特别是如果您分发对Cursor的引用。在游标关闭之前，您不能关闭数据库，因为Cursor可以使用它知道的数据库连接重新查询数据。这将在两种情况下发生：

如果您的代码可能通过错误的CursorAdapter间接使用cursor.requery()。

第二种情况并不常见。 Cursor只能看到当前查询的有限窗口。如果将光标移动到该窗口之外，它也将使用数据库连接重新查询所需的部分。典型的小型数据库通常比窗口小得多，因此无需移动窗口，也不会发生隐藏的重新查询。

关闭资源通常是件好事。在大多数情况下，关闭数据库不是。不这样做也没问题。 SQLite确保没有任何不好的事情发生，因为SQLite的事务安全性保证了对数据库的每次修改。

Answer 2

您正在使用db.close（）关闭每个方法末尾的数据库连接。因此，如果您启动另一种方法，则始终必须重新启用该连接。与

getWritableDatabase();

你应该完成它。

或者您不会在每个方法的末尾关闭连接。这完全取决于您的应用程序的用途。

Answer 3

您尚未在

中调用db.close();

      public List<Task> getAllTasks() { 
    .....

}

这可能导致该问题，因为当您尝试在下一个函数中调用getWritableDatabase();时，数据库已经打开。所以请致电db.close();并在getAllTasks()方法中关闭它。

Java非法状态异常

3 个答案: