各位，

我正在寻找一种设计模式，使 UI 线程能够与客户端 SQLite 数据库进行交互，该数据库可能具有批量插入(耗时 10 秒)、快速插入和读取，并且不会阻塞 UI线。

我想知道我是否为此使用了最佳设计模式，因为我最近一直在调试死锁和同步问题，我对我的最终产品不是 100% 有信心。

所有数据库访问现在都通过单例类成为瓶颈。这是伪代码，显示了我如何在我的单例 DataManager 中接近写入:

public class DataManager {

  private SQLiteDatabase mDb;
  private ArrayList<Message> mCachedMessages;

  public ArrayList<Message> readMessages() { 
    return mCachedMessages; 
  }

  public void writeMessage(Message m) {
    new WriteMessageAsyncTask().execute(m);
  }

  protected synchronized void dbWriteMessage(Message m) {
    this.mDb.replace(MESSAGE_TABLE_NAME, null, m.toContentValues());
  }

  protected ArrayList<Message> dbReadMessages() {
    // SQLite query for messages
  }

  private class WriteMessageAsyncTask extends AsyncTask<Message, Void, ArrayList<Messages>> {
    protected Void doInBackground(Message... args) {
       DataManager.this.mDb.execSQL("BEGIN TRANSACTION;");
       DataManager.this.dbWriteMessage(args[0]);
       // More possibly expensive DB writes
       DataManager.this.mDb.execSQL("COMMIT TRANSACTION;");
       ArrayList<Messages> newMessages = DataManager.this.dbReadMessages();
       return newMessages;
    }
    protected void onPostExecute(ArrayList<Message> newMessages) {
       DataManager.this.mCachedMessages = newMessages;
    }
  }
}

亮点:

• 首先:所有公共(public)写操作(writeMessage)都是通过 AsyncTask 发生的，从不在主线程上 线程
• 下一步:所有写操作都同步并包装在 开始交易
• 下一步:读操作是 非同步的，因为它们在写入期间不需要阻塞
• 最后:读操作的结果缓存在main onPostExecute 中的线程

这是否代表了将潜在大量数据写入 SQLite 数据库同时最大限度地减少对 UI 线程的影响的 Android 最佳实践？您在上面看到的伪代码是否存在任何明显的同步问题？

更新

我上面的代码有一个严重的bug，如下:

DataManager.this.mDb.execSQL("BEGIN TRANSACTION;");

该行获取数据库的锁。但是，它是一个 DEFERRED 锁，所以在写入发生之前，other clients can both read and write .

DataManager.this.dbWriteMessage(args[0]);

该行实际上修改了数据库。此时，该锁是一个RESERVED 锁，因此没有其他客户端可以写入。

请注意，在第一次调用 dbWriteMessage 之后，可能会有更多昂贵的数据库写入。假设每个写操作都发生在 protected 同步方法中。这意味着在 DataManager 上获取了一个锁，写入发生，然后锁被释放。如果 WriteAsyncMessageTask 是唯一的编写器，这很好。

现在假设还有一些其他任务也执行写操作，但不使用事务(因为它是快速写入)。它可能看起来像这样:

  private class WriteSingleMessageAsyncTask extends AsyncTask<Message, Void, Message> {
    protected Message doInBackground(Message... args) {
       DataManager.this.dbWriteMessage(args[0]);
       return args[0];
    }
    protected void onPostExecute(Message newMessages) {
       if (DataManager.this.mCachedMessages != null)
         DataManager.this.mCachedMessages.add(newMessages);
    }
  }

在这种情况下，如果 WriteSingleMessageAsyncTask 与 WriteMessageAsyncTask 同时执行，并且 WriteMessageAsyncTask 已经执行了至少一次写入，则 WriteSingleMessageAsyncTask 有可能调用 dbWriteMessage，获取 DataManager 上的锁，但随后被阻止完成其由于保留锁而写。 WriteMessageAsyncTask 反复获取和放弃对 DataManager 的锁定，这是一个问题。

要点:结合事务和单例对象级锁定可能会导致死锁。确保在开始事务之前拥有对象级锁。

对我原来的 WriteMessageAsyncTask 类的修复:

   synchronized(DataManager.this) {
     DataManager.this.mDb.execSQL("BEGIN TRANSACTION;");
     DataManager.this.dbWriteMessage(args[0]);
     // More possibly expensive DB writes
     DataManager.this.mDb.execSQL("COMMIT TRANSACTION;");
   }

更新 2

观看来自 Google I/O 2012 的这段视频: http://youtu.be/gbQb1PVjfqM?t=19m13s

它建议使用内置独占事务然后使用 yieldIfContendedSafely 的设计模式

最佳答案

关于同步/死锁部分，我真的不能说太多，这在很大程度上取决于您的代码的其余部分。由于 DataManager 类并不真正与 UI 交互，您可能希望使用服务 (IntentService) 而不是 AsyncTask。您可以在完成同步后显示通知。如果您不调用 UI 代码，您实际上并不需要 onPostExecute()。

关于android - 在 Android 中写入 SQLite 数据库的推荐设计模式，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/11535222/