1

まず第一に、ここにある同様の投稿はすべて役に立ちませんでした。

AsyncTask では、アプリの起動が最初の起動であるかどうかを確認します。つまり、データベースが存在するかどうかを確認します。テーブルからのクエリでそれを行いたいです。これが実行されているものです:

@Override
public List<Entity> loadAll(Entity markerEntity)
{
    Log.i(TAG, "trying to load all entities of type " + markerEntity.getTable());

    List<Entity> results = new LinkedList<Entity>();

    SQLiteDatabase db = defaultSQLiteOpenHelper.getWritableDatabase();

    Cursor cursor = db.query(markerEntity.getTable(), markerEntity.getAllColumns(), null, null, null, null, null);    
    cursor.moveToFirst();

    while (!cursor.isAfterLast())
    {
        Entity result = markerEntity.createNewInstance(cursor);
        results.add(result);

        cursor.moveToNext();
    }

    db.close();

    return results;
}

IllegalArgumentException: database not openによるクラッシュは、この行で発生します。

SQLiteDatabase db = defaultSQLiteOpenHelper.getWritableDatabase();

メソッド (以下を参照) createAllTablesで正確に発生します。

ドキュメントには次のように記載されています。

読み取りと書き込みに使用されるデータベースを作成および/または開きます。これが初めて呼び出されると、データベースが開かれ、onCreate(SQLiteDatabase)、onUpgrade(SQLiteDatabase, int, int) および/または onOpen(SQLiteDatabase) が呼び出されます。

私の意見では、DefaultSQLiteOpenhelper を実行してすべてのデータベース テーブルを作成する必要がありますが、それは行われません。

public class DefaultSQLiteOpenHelper extends SQLiteOpenHelper implements ISQLiteOpenHelper
{
    private static final String TAG = DefaultSQLiteOpenHelper.class.getSimpleName();

    private static final String DATABASE_NAME = "MY_DATABASE";
    private static final int DATABASE_VERSION = 1;

    @Inject
    public DefaultSQLiteOpenHelper(Context context)
    {
        super(context, DATABASE_NAME, null, DATABASE_VERSION);
    }

    //is called by the framework if the database doesn't exist
    @Override
    public void onCreate(SQLiteDatabase db)
    {
        Log.i(TAG, "no database found. Generating new database...");

        createAllTables(db);    
    }

    @Override
    public void onUpgrade(SQLiteDatabase db, int oldVersion, int newVersion)
    {
         Log.w(TAG, "Upgrading database from version " + oldVersion + " to " + newVersion);
//       db.execSQL("DROP TABLE IF EXISTS " + TABLE_COMMENTS); ///TODO when upgrade
//       onCreate(db);
    }

    //The order must not be changed because sqlite doesn't allow
    //table modification. That means one cannot add constraints afterwards
    //which results in creating tables in a specific order
    private final void createAllTables(SQLiteDatabase db)
    {
        db.beginTransaction();

        //create lookups at first
        db.execSQL(TableFactory.createUsage());
        db.execSQL(TableFactory.createLanguage());

        db.execSQL(TableFactory.createAccount());
        db.execSQL(TableFactory.createPreferences());
        db.execSQL(TableFactory.createLanguageUsageRel());
        db.execSQL(TableFactory.createPantry());
        db.execSQL(TableFactory.createProduct());

        //populate lookups
        db.execSQL(DataFactory.populateUsage());
        db.execSQL(DataFactory.populateLanguage());

        db.setTransactionSuccessful();
        db.endTransaction();
        db.close();
    }
}

問題は、なぜデータベースを開かないのですか? ドキュメントには、そうしなければならないと書かれています。データベースを手動で開くなどの解決策として、汚い回避策は受け入れません。私のコードで正確に間違っているのは何ですか? Androidのバグだとは信じられません。

スタックトレースは次のとおりです。

05-27 17:25:50.370: I/DefaultSQLiteOpenHelper(719): データベースが見つかりません。新しいデータベースを生成しています... 05-27 17:25:50.400: W/dalvikvm(719): threadid=10: キャッチされない例外で終了するスレッド (group=0x40015560) 05-27 17:25:50.400: E/AndroidRuntime(719 ): 致命的な例外: AsyncTask

1 05-27 17:25:50.400: E/AndroidRuntime(719): java.lang.RuntimeException: 実行中にエラーが発生しました

doInBackground() 05-27 17:25:50.400: E/AndroidRuntime(719): android.os.AsyncTask$3.done(AsyncTask.java:200) 05-27 17:25:50.400: E/AndroidRuntime(719) : java.util.concurrent.FutureTask$Sync.innerSetException(FutureTask.java:274) 05-27 17:25:50.400: E/AndroidRuntime(719): java.util.concurrent.FutureTask.setException(FutureTask.java) :125) 05-27 17:25:50.400: E/AndroidRuntime(719): java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:308) 05-27 17:25:50.400: E/AndroidRuntime (719): java.util.concurrent.FutureTask.run(FutureTask.java:138) 05-27 17:25:50.400: E/AndroidRuntime(719): java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor. java:1088) 05-27 17:25:50.400: E/AndroidRuntime(719): java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:581) 05-27 17:25:50.400: E/AndroidRuntime(719): java.lang.Thread.run(Thread.java:1019) 05-27 17:25:50.400: E/AndroidRuntime(719): 原因by: java.lang.IllegalStateException: データベースが開いていません 05-27 17:25:50.400: E/AndroidRuntime(719): android.database.sqlite.SQLiteDatabase.endTransaction(SQLiteDatabase.java:555) 05-27 17:25 :50.400: E/AndroidRuntime(719): android.database.sqlite.SQLiteOpenHelper.getWritableDatabase(SQLiteOpenHelper.java:137) 05-27 17:25:50.400: E/AndroidRuntime(719): com.mydomain.android. base.persistence.PersistenceManager.loadAll(PersistenceManager.java:58) 05-27 17:25:50.400: E/AndroidRuntime(719): com.mydomain.android.base.main.StartupTask.isFirstStartAfterInstallation(StartupTask.java:107) ) 05-27 17:25:50.400: E/AndroidRuntime(719): com.mydomain.android.base.main.StartupTask.doInBackground(StartupTask.java:52) 05-27 17:25:50.400: E/AndroidRuntime(719): com.mydomain.android.base.main.StartupTask.doInBackground(StartupTask.java:18) 05-27 で17:25:50.400: E/AndroidRuntime(719): android.os.AsyncTask$2.call(AsyncTask.java:185) 05-27 17:25:50.400: E/AndroidRuntime(719): java.util で。 concurrent.FutureTask$Sync.innerRun(FutureTask.java:306) 05-27 17:25:50.400: E/AndroidRuntime(719): ... 4 もっと見る306) 05-27 17:25:50.400: E/AndroidRuntime(719): ... 4 以上306) 05-27 17:25:50.400: E/AndroidRuntime(719): ... 4 以上

4

1 に答える 1

5

まず、 でデータベースを閉じていますcreateAllTables()onCreate()から、またはonUpgrade()のデータベースを閉じないでくださいSQLiteOpenHelper

createAllTables()第 2 に、トランザクションで実行されるため、トランザクション ロジックは では不要onCreate()です。とにかく、トランザクションロジックが不適切に実装されているため、これは良いことです。

後者のコメントを明確にするために、トランザクションを行う正しい方法は次のとおりです。

try {
    db.beginTransaction();

    // SQL

    db.setTransactionSuccessful();
}
// optional catch block
finally {
    db.endTransaction();
}

endTransaction()SQL 例外が発生した場合は、呼び出される必要があります。実装ではこれをスキップします。

于 2013-05-27T17:47:33.313 に答える