Java非法状态异常

Java非法状态异常,java,android,database,Java,Android,Database,我正在关注一个在线教程,但是我得到了一个非法的状态异常。 链接到教程: 以下是数据库类: public class TaskerDbHelper extends SQLiteOpenHelper { private static final int DATABASE_VERSION = 1; // Database Name private static final String DATABASE_NAME = "taskerManager"; // tasks table name p

我正在关注一个在线教程,但是我得到了一个非法的状态异常。 链接到教程:

以下是数据库类:

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

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

你应该把它做完


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

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

你应该把它做完


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

你还没有调用
db.close()输入

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

}
public List getAllTasks(){
.....
}

它可能会导致这个问题,因为当您尝试调用
getWritableDatabase()时。所以调用
db.close()
并在
getAllTasks()
方法中将其关闭。

您还没有调用
db.close()输入

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

}
public List getAllTasks(){
.....
}

它可能会导致这个问题,因为当您尝试调用
getWritableDatabase()时。所以调用
db.close()
并在
getAllTasks()
方法中将其关闭。

您的代码中有两个问题:

A) 您不能在创建时关闭数据库。该方法是打开数据库的一部分,因此数据库应在打开后打开

B) 在每个任务结束时关闭数据库

public void updateTask(Task task) {
    db = get..
    ...
    db.close();
}
但在这样做的同时,您仍然通过
光标使用相同的数据库

db = db.get..
Cursor cursor = db.rawQuery
while (cursor.moveToNext() {
     updateTask();
}
db.close();
SQLiteOpenHelper
将始终为您提供相同的
db
对象,因为只有一个数据库。在一种方法中关闭连接,而在另一种方法中使用连接,也可能导致您的问题

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

尤其是当您分发对
光标的引用时。在关闭游标之前,不能关闭数据库,因为
游标可以使用它知道的数据库连接重新查询它的数据。这将在两种情况下发生:

如果您的代码使用了
cursor.requery()
,则可能是通过坏消息间接使用的

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


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

代码中有两个问题:

A) 您不能在创建时关闭数据库。该方法是打开数据库的一部分,因此数据库应在打开后打开

B) 在每个任务结束时关闭数据库

public void updateTask(Task task) {
    db = get..
    ...
    db.close();
}
但在这样做的同时,您仍然通过
光标使用相同的数据库

db = db.get..
Cursor cursor = db.rawQuery
while (cursor.moveToNext() {
     updateTask();
}
db.close();
SQLiteOpenHelper
将始终为您提供相同的
db
对象,因为只有一个数据库。在一种方法中关闭连接,而在另一种方法中使用连接,也可能导致您的问题

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

尤其是当您分发对
光标的引用时。在关闭游标之前,不能关闭数据库,因为
游标可以使用它知道的数据库连接重新查询它的数据。这将在两种情况下发生:

如果您的代码使用了
cursor.requery()
,则可能是通过坏消息间接使用的

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

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