Mtk's Blog: [Android] SQLiteDatabase created and never closed

How to slove the problem "SQLiteDatabase created and never closed"

SQLiteを使ったアプリを作っていたら、ごく稀にタイトルに書いたエラーがログに出力されていることに気がつきました。
ただし、同じ操作をしても再現できません。
ソースをよく見たらCursorのインスタンスをclose()し忘れている箇所が何カ所かありました。
おそらくclose()忘れが原因です。
Cursorが不要になったら直ちにclose()するようにしました。

以下のソースはint型の値をSQLiteに読み書きするキー・バリュー方式(?)のクラスです。
このクラスは汎用性が高いのでMtkはアプリケーションの設定情報を保存するためにを使い回してます。
キャッシュが肥大化しないようにVACUUMもします。

package jp.fujiu.AndroidApp.SqliteSample;

import android.content.ContentValues;
import android.content.Context;
import android.database.Cursor;
import android.database.SQLException;
import android.database.sqlite.SQLiteDatabase;
import android.database.sqlite.SQLiteOpenHelper;
import android.os.AsyncTask;

public class SqliteSample extends SQLiteOpenHelper {
 private static final int VERSION = 1;
 private static final String FILE_NAME = "data.sqlite";
 private static final String INT_TABLE = "INT_TABLE";
 private static final String CREATE_INT_TABLE 
  = "CREATE TABLE " + INT_TABLE + "(" 
  + "_id INTEGER PRIMARY KEY AUTOINCREMENT, "
  + "KEY TEXT NOT NULL UNIQUE, "
  + "VALUE INTEGER NOT NULL"
  + ");";
 private static final String KEY = "KEY";
 private static final String VALUE = "VALUE";
 
 private boolean isModefied = false;

 public SqliteSample(Context context) {
  super(context, FILE_NAME, null, VERSION);
 }

 @Override
 public void onCreate(SQLiteDatabase db) {
  db.beginTransaction();
  try {
   db.execSQL(CREATE_INT_TABLE);
   db.setTransactionSuccessful();
  } finally {
   db.endTransaction();
  }
 }
 
 public Integer getInteger(String key){
  Integer value = null;
  
  String[] columns = {VALUE};
  String selection = "KEY='" + key + "'";

  SQLiteDatabase db = getReadableDatabase();
  Cursor c = db.query(INT_TABLE, columns, selection, null, null, null, null);
  c.moveToFirst();
  if (c.getCount()!=0){
   value = new Integer(c.getInt(0));
  }
  /* Cursorを忘れずにclose() */
  c.close();
  
  return value;
 }
 
 
 public synchronized void setInt(String key, int value){
  SQLiteDatabase db = getWritableDatabase();
  db.beginTransaction();
  try {
   ContentValues values = new ContentValues();
   values.put(KEY, key);
   values.put(VALUE, value);
   String[] whereArgs = {key};
   int affected = db.update(INT_TABLE, values, "KEY=?", whereArgs);

   //UPDATEされてなければINSERT
   if (affected==0){
    db.insert(INT_TABLE, null, values);
   }
   db.setTransactionSuccessful();
   
   isModefied = true;
  } finally {
   db.endTransaction();
  }
 }

 @Override
 public void close() {
  if (isModefied){
   //テーブルの変更があったらVACUUM処理を非同期で実行
   AsyncTask<Object, Object, Object> task = new AsyncTask<Object, Object, Object>(){
    @Override
    synchronized protected Object doInBackground(Object... params) {
     SQLiteDatabase db = getWritableDatabase();
     try {
      db.execSQL("VACUUM " + INT_TABLE);
      
      /* REINDEXが必要ならここで実行してください */
      //TODO
      //db.execSQL("REINDEX " + INT_TABLE);
     } catch (SQLException e) {
      e.printStackTrace();
     }
     return null;
    }

    @Override
    protected void onPostExecute(Object result) {
     super.onPostExecute(result);
     //非同期処理が終了したら
     //SQLiteOpenHelperを忘れずにclose()
     SqliteSample.super.close();
    }
   };
   task.execute((Object)null);
  } else {
   //テーブルの変更がなければ
   //SQLiteOpenHelperを直ちにclose()
   SqliteSample.super.close();
  }
 }

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

上記のソースのc.close()をわざと消して試してみましたが、エラーが起こらないことが多いです。
再現性がないので自動でがclose()してくれてるケースがあるのかもしれません。

SqliteSampleの使い方
１．int型データの保存方法
setIntメソッドで保存します。第１引数はキーとなる文字列、第2引数は保存する値です。
キーが既に保存されていれば値を上書きし、保存されていなければ新規に登録します。

SqliteSample sql = new SqliteSample(getApplicationContext());
sql.setInt("value1", 1);
sql.setInt("value2", 2);
sql.setInt("value3", 3);
････
sql.close();

２．int型データの取り出し方法
getIntメソッドで値を取り出します。キーに対応する値がなければgetIntの第2引数の値が返されます。

SqliteSample sql = new SqliteSample(getApplicationContext());
Integer value1 = sql.getInteger("value1");
if (value1!=null){
 //データありの処理
} else {
 //データなしの処理
}
Integer value2 = sql.getInteger("value2");
if (value2!=null){
 //データありの処理
} else {
 //データなしの処理
}
Integer value3 = sql.getInteger("value3");
if (value3!=null){
 //データありの処理
} else {
 //データなしの処理
}
････
sql.close();

以上、参考になれば幸いです。

現在公開中
「お天気ロイド」
「Eye元気！」
「Wi-Fi sync screen」
いずれもキャッシュを肥大化させないためにVACUUMしています

Mtk's Blog

2011年3月5日土曜日

[Android] SQLiteDatabase created and never closed

0 件のコメント:

コメントを投稿