1

I am following a tutorial online, but i am getting an illegal state exception. Link to tutorial: http://www.developerfeed.com/android/tutorial/building-todo-list-app-android-using-sqlite

Here is the Database class:

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();
}

}

And here is what the log file says:

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
4

3 回答 3

3

您的代码中有两个问题：

A）您不得关闭onCreate. 该方法是打开数据库的一部分，因此数据库应该在之后打开。源代码

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可以使用它知道的数据库连接重新查询它的数据。这将在两种情况下发生：

如果您的代码cursor.requery()可能通过坏的CursorAdapter.

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

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

于 2013-08-14T10:22:38.067 回答
2

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

getWritableDatabase();

你应该完成它。

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

于 2013-08-14T09:35:10.427 回答
1

你还没有db.close();打电话

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

}

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

于 2013-08-14T09:44:05.383 回答