记一次SQLiteDatabase的关闭问题

有一天在公司的项目里发现一个异常

java.lang.IllegalStateException: attempt to re-open an already-closed object

后来经过排查发现项目里的所有数据库操作之前都会去获取一次数据库对象getWritableDatabase ，操作完成之后去关闭数据库，像这样：

    public synchronized void insertSelfGroup(SelfGroup.Group group){
        if(group == null){
            return;
        }
        //获取数据库
        SQLiteDatabase db = getWritableDatabase();
        if (db != null) {
            try {
                db.beginTransaction();
                ContentValues cv = group2CV(group);
                db.insert(GROUP_TABLE, null, cv);
                db.setTransactionSuccessful();
            } catch (Exception e) {
                e.printStackTrace();
            } finally {
                db.endTransaction();
                //关闭数据库
                db.close();
            }
        }
    }

正常来说这样是没有问题的，但是如果两个线程同时操作的话就会出现上面的报错，比如线程A调用了改方法，之后线程B也调用了该方法，我们知道同一个SQLiteOpenHelper对象获取到的数据库对象是相同的，如果线程A率先执行完毕执行了close，此时数据库已经关闭，而线程B还没执行完，这时就会报错。

知道了原因就是数据库的关闭问题，那怎么解决呢？

在网上搜了一下关于SQLite数据库的关闭问题发现大部分人建议不关闭数据库，或者在退出App的时候在关闭数据库，但是另一种说法是如果长时间不关闭数据库会出现下面的异常：

Leak found
Caused by: java.lang.IllegalStateException: SQLiteDatabase created and never closed

并且他们建议使用一个计数器，当获取一个数据库对象时计数器+1，关闭的时候-1，如果关闭的时候计数器值为0，也就代表没有数据库操作的时候就关闭数据库，简单使用如下：

public class DatabaseManager{

    private AtomicInteger mOpenCounter = new AtomicInteger();

    private static DatabaseManager instance;
    private static SQLiteOpenHelper mDatabaseHelper;
    private SQLiteDatabase mDatabase;

    public static synchronized void initializeInstance(SQLiteOpenHelper helper) {
        if (instance == null) {
            instance = new DatabaseManager();
            mDatabaseHelper = helper;
        }
    }

    public static synchronized DatabaseManager getInstance() {
        if (instance == null) {
            throw new IllegalStateException(DatabaseManager.class.getSimpleName() +
                    " is not initialized, call initializeInstance(..) method first.");
        }

        return instance;
    }

    public synchronized SQLiteDatabase openDatabase() {
        if(mOpenCounter.incrementAndGet() == 1) {
            // Opening new database
            mDatabase = mDatabaseHelper.getWritableDatabase();
        }
        return mDatabase;
    }

    public synchronized void closeDatabase() {
        if(mOpenCounter.decrementAndGet() == 0) {
            // Closing database
            mDatabase.close();

        }
    }}

本来认为这种方式应该是最优解时我又发现了谷歌文档上对SQLite的一段描述：

这下我凌乱了，到底怎样做才最合理呢？
于是我又去看了一下一些知名数据库框架时怎么做的，首先看了一下Litepal，发现它并没有关闭数据库，只有在删除数据库的时候才会去关闭，我又看了一下GreenDao发现它也没有去关闭数据库，既然这些知名框架都不去关数据库那我们也先不关了吧，至于上面说的SQLiteDatabase created and never closed报错我也没碰到过，但是本着严谨的态度我又看一下谷歌官方推荐的数据库框架Room，但是我在官方提供的demo中也没有看关闭数据库的操作，并且官方文档还给了这样一段建议：

最后暂定结论为不需要关闭数据库 ，如果后面发现问题我再来更新。