SQLiteException: no such table (code1)の解決方法がわからない。

Question

###前提・実現したいこと
Androidで資材管理をするアプリを開発しています。
アプリ内ですでに作成したSQLiteのDatabaseを利用して、資材の一覧をExpandableListViewで実装しようと考えています。
###発生している問題・エラーメッセージ
ExpandableListViewChildData.classの11行目のqueryメソッドにおいて、すでに作成したDatabaseを読み込んで、SELECT文を実行しようとするとSQLiteException: no such table: Name(code1)というエラーが発生。

###該当のソースコード

public class DBOpenHelper extends SQLiteOpenHelper {
    private static final String DB_FILE_NAME = "HAVI.db";
    private static final String DB_NAME = "HAVI";
    private static final int DB_VERSION = 1;

    private Context context;
    private File dbPath;
    private boolean createDatabase = false;

    public DBOpenHelper(Context context) {
        super(context, DB_NAME, null, DB_VERSION);
        this.context = context;
        this.dbPath = context.getDatabasePath(DB_NAME);
    }

    @Override
    public synchronized SQLiteDatabase getReadableDatabase() {
        SQLiteDatabase database = super.getReadableDatabase();
        if(createDatabase) {
            try {
                database = copyDatabase(database);
            } catch (IOException e) {

            }
        }
        return database;
    }

    @Override
    public synchronized SQLiteDatabase getWritableDatabase() {
        SQLiteDatabase database = super.getWritableDatabase();
        if(createDatabase) {
            try {
                database = copyDatabase(database);
            } catch (IOException e) {

            }
        }
        return database;
    }

    private SQLiteDatabase copyDatabase(SQLiteDatabase database) throws IOException {
        database.close();

        InputStream input = context.getAssets().open(DB_FILE_NAME);
        OutputStream output = new FileOutputStream(this.dbPath);
        copy(input, output);


        createDatabase = false;

        return super.getWritableDatabase();
    }

    @Override
    public void onCreate(SQLiteDatabase db) {
        super.onOpen(db);
        this.createDatabase = true;
    }

    public void onUpgrade(SQLiteDatabase db, int oldVersion, int newVersion) {
        onUpgrade(db, oldVersion, newVersion);
    }

    private static int copy(InputStream input, OutputStream output) throws IOException {
        byte[] buffer = new byte[1024 * 4];
        int count = 0;
        int n = 0;
        while(-1 != (n = input.read(buffer))) {
            output.write(buffer, 0, n);
            count += n;
        }

        output.flush();
        input.close();
        output.close();

        return count;
    }
}

public class ExpandableListViewChildData {
    public static List<List<Map<String, String>>> allChildList = new ArrayList<List<Map<String, String>>>();
    private static List<Map<String, String>> ChildList = new ArrayList<Map<String, String>>();
    private static Map<String, String> ChildData = new HashMap<String, String>();
    public static String[] key = {"TITLE"};

    public static List<List<Map<String, String>>> getData(String place, SQLiteDatabase db) {
        String[] column = {"name", "expendable"};
        String where = "place1=? OR place2=?";
        String[] whereArgs = {place, place};
        Cursor cursor = db.query("Name", column, where, whereArgs, null, null, null, null);
        String[] expendable = {"A", "B"};

        try {
            for(String ex : expendable) {
                if (cursor.moveToFirst()) {
                    do {
                        String name = cursor.getString(cursor.getColumnIndex("name"));
                        String expendableDatabase = cursor.getString(cursor.getColumnIndex("expendable"));
                        if (ex.equals(expendableDatabase)) {
                            ChildData.put(key[0], name);
                        }
                    } while(cursor.moveToNext());
                }
                ChildList.add(ChildData);


            }
            allChildList.add(ChildList);
        } finally {
            db.close();
        }

        return allChildList;
    }
}

###試したこと
デバッグして、Databaseがコピーされていることを確認。SQLite Browserでデータベースを作成して、それをターミナルでtableが存在してることを確認。

###補足情報(言語/FW/ツール等のバージョンなど)
API 21

いろいろ調べてみましたが、よくわかりませんでした。ぜひご教授ください。

Answer 1

データベースのクローズは、SQLiteDatabase#closeではなく、SQLiteOpenHelper#closeを使ってはいかがですか?

SQLiteOpenHelperのソースコードを見る限り、一度データベースをオープンするとそのデータベースをメンバに保持し、それを使いまわしているようです。
それが原因かどうかまでは確認できませんでしたが、動作しないのであれば疑うべきだと思います。
ご自身でもソースコードを見てご確認してみてください。

というか、SQLiteOpenHelperを使う意味もなくないですか?
SQLiteOpenHelperのメリットは、データベースの新規作成やアップグレードが楽な点だと思うのですが、データベースを開くたびにデータベースを作りなおしているようですし、メリットは薄いと思うのですが。
(getReadableDatabaseを呼んで、getWritableDatabaseのデータベースを返すのも、気持ち悪いですし)

Answer 2

クエリのテーブル名が間違っている可能性があります。正しい名前ですか？
正しい場合はそもそもデータベースの作成は出来ていない、またパスが間違っている可能性があります。

・データベースのオリジナルはassetsに配置すべきだとおもいます。
・DBOpenHelper#onCreateでファイルのコピーをした方がわかりやすい。

---

今回の原因をまとめると
・assetsからDBのコピーをしようとしたが権限がないため書き込みができていなかった。
・SQLiteOpenHelperの機能でDBファイルがなかったため自動で作成した。
・DBファイルは空になっているため質問の例外が発生した。

やはり確認方法が甘かったように思います。
たぶん、ログに書き込み失敗のエラーログが出ていたと思います。

Comments

[Velonica]

テーブル名の間違い、データベースの作成の有無は確認済みです。データベースのパスはDBOpenHelperのコンストラクタでcontext.getDatabasePath(DB_NAME);を使っているのですが、ここに間違いがあるのでしょうか？

データベースのオリジナルはassetsに入れてます。

DBOpenHelperのonCreateでファイルをコピーしてみましたが、同じエラーが発生しました。

[yona]

DB_NAMEに.dbを追加してみてください。

[Velonica]

回答ありがとうございます。
.dbをつけて実行してみました。しかし、以下のエラーが出てしまいました。java.lang.IllegalStateException: attempt to re-open an already-closed object: SQLiteDatabase: /data/data/com.example.macuser.havi2/databases/HAVI.db

ちなみに一応、DB＿NAME以外にはDB_FILE_NAMEというフィールドに.dbとついたオリジナルのDatabaseファイル名を指定しています。これはcopyDatabaseメソッドにおいて利用しており、オリジナルのデータベースをコピーしています。

[yona]

SQLiteOpenHelper#onCreateは指定したパス上にDBファイルが存在しない場合は新規で空のDBファイルを作成します。
そのため、パスが間違っている、またはDBファイルのコピー処理の実行位置が悪いため空のDBファイルが作られたため、テーブルがないと言われていると考えています。

正直に言うと私はあなたが行ったファイルがあるかの確認した結果を信用していません。
・データベースがコピーされていること確認した方法がわからない。
・データベースにテーブルがあるのを確認したファイルはどのファイルなのか？

そのため下記の方法で確認してください。
・assetsのDBファイルにテーブルが存在すること。
・assetsのDBファイルとコピーしたDBファイルのサイズが一致すること。
・コピーしたDBファイルをさらにDownloaフォルダにコピーし、そのDBファイルにテーブルが含まれていること。

[Velonica]

回答ありがとうございます。
assetsのDBファイルにテーブルが存在しているか、assetsのDBファイルとコピーしたDBファイルのサイズが一致するかどうか、コピーしたDBファイルをさらにDownloadフォルダにコピーし、そのDBファイルにテーブルが含まれるかどうか、確認しました。しかし、３つとも正しかったです。

しかし、この３つを確認するにあたって、今まで実機で実行していたのですが、エミュレーターでアプリを実行してみたところ、SQLiteException: no such table (code1)のエラーは出ずに、adapterに対するNullPointerExceptionが発生しました。これは、もしかして、実機において何かしらの問題があるのでしょうか？

ちなみに、実機はSamsung SC-01F/Android 5.0/API 21です

[yona]

ありえますね。
パーミッションは付いていますか？

また、アプリを一度アンインストールしてから再度実行してみてください。

[Velonica]

パーミッションについて自分なりに調べてみたところ、パーミッション一覧というのは見つけられたのですが、どれをつければ良いのかわかりませんでした。
ぜひご教授ください。

[yona]

ファイル書き込みのパーミッションWRITE_EXTERNAL_STORAGEを追加してください。

[Velonica]

無事解決しました。
長々とおつきあいありがとうございました。

[yona]

おめでとうございます。
やはり、getReadableDatabaseやgetWritableDatabaseにファイルコピーの処理を書くべきではないと思います。

[Velonica]

そうなんですね。勉強になりました。
ありがとうございました。