在多个chrome.storage API调用中防止竞争条件的最佳方法是什么?

时间:2013-02-24 10:33:06

标签: javascript google-chrome-extension locking race-condition

  1. 某事要求任务
  2. 其他东西将任务列表从存储中拉出来,并检查那里是否有任务。
  3. 如果有任务,它会删除一个和更小的"任务列表"被收回存放。
  4. 在步骤2和3之间,如果发生多个请求,可能会发生竞争条件,并且相同的任务将被提供两次。

    正确的解决方案是"锁定" "任务表"虽然单个任务是"签出",以防止任何其他请求?

    对性能影响最小的解决方案是什么,例如延迟执行,以及如何使用chrome.storage API在javascript中实现?

    一些代码例如:

    function decide_response ( ) {
        if(script.replay_type == "reissue") {
                function next_task( tasks ) {
                    var no_tasks = (tasks.length == 0);
                    if( no_tasks ) {
                        target_complete_responses.close_requester();
                    }
                    else {
                        var next_task = tasks.pop();
                        function notify_execute () {
                            target_complete_responses.notify_requester_execute( next_task );
                        }
                        setTable("tasks", tasks, notify_execute);
                    }
                }
                getTable( "tasks", next_tasks );
        ...
        }
    ...
    }
    

1 个答案:

答案 0 :(得分:5)

我认为你可以通过利用javascript在上下文中是单线程的事实来管理而不需要锁定,即使使用异步chrome.storage API也是如此。只要您没有使用chrome.storage.sync,即 - 如果云中可能有或没有变化,我认为所有赌注都已关闭。

我会做这样的事情(从袖口上写下来,没有经过测试,没有错误处理):

var getTask = (function() {
  // Private list of requests.
  var callbackQueue = [];

  // This function is called when chrome.storage.local.set() has
  // completed storing the updated task list.
  var tasksWritten = function(nComplete) {
    // Remove completed requests from the queue.
    callbackQueue = callbackQueue.slice(nComplete);

    // Handle any newly arrived requests.
    if (callbackQueue.length)
      chrome.storage.local.get('tasks', distributeTasks);
  };

  // This function is called via chrome.storage.local.get() with the
  // task list.
  var distributeTasks = function(items) {
    // Invoke callbacks with tasks.
    var tasks = items['tasks'];
    for (var i = 0; i < callbackQueue.length; ++i)
      callbackQueue[i](tasks[i] || null);

    // Update and store the task list. Pass the number of requests
    // handled as an argument to the set() handler because the queue
    // length may change by the time the handler is invoked.
    chrome.storage.local.set(
      { 'tasks': tasks.slice(callbackQueue.length) },
      function() {
        tasksWritten(callbackQueue.length);
      }
    );
  };

  // This is the public function task consumers call to get a new
  // task. The task is returned via the callback argument.
  return function(callback) {
    if (callbackQueue.push(callback) === 1)
      chrome.storage.local.get('tasks', distributeTasks);
  };
})();

这将来自使用者的任务请求存储为本地内存中队列中的回调。当新请求到达时,回调将添加到队列中,并且将获取任务列表 iff 这是队列中的唯一请求。否则,我们可以假设队列已经被处理(这是一个隐式锁,只允许一行执行来访问任务列表)。

获取任务列表后,任务将分发给请求。请注意,如果在完成提取之前有更多请求,则可能有多个请求。如果请求多于任务,则此代码仅将null传递给回调。要在更多任务到达之前阻止请求,请在添加任务时保留未使用的回调并重新启动请求处理。如果任务可以动态生成和消耗,请记住也需要在那里防止竞争条件,但这里没有显示。

在存储更新的任务列表之前,防止再次读取任务列表非常重要。为此,在更新完成之前,不会从队列中删除请求。然后我们需要确保处理同时到达的任何请求(可以将调用短路到chrome.storage.local.get(),但为了简单起见,我这样做了。)

这种方法应该非常有效,因为它应该尽可能快地响应任务列表的更新,同时尽可能快地响应。没有明确的锁定或等待。如果您在其他上下文中有任务使用者,请设置一个调用getTask()函数的chrome.extension消息处理程序。