Sequelize правильно выполнение нескольких создает + обновления

Question

У меня есть задание cron, которое сбрасывает список элементов на веб-сайте, а затем вставляет или обновляет записи в базе данных. Когда я очищаю страницу, я хочу создать записи для новых, которые еще не были созданы, иначе обновите существующие. В настоящее время я делаю что-то вроде этого:

// pretend there is a "Widget" model defined 

function createOrUpdateWidget(widgetConfig) { 
    return Widget.find(widgetConfig.id) 
     .then(function(widget) { 
      if (widget === null) { 
       return Widget.create(widgetConfig); 
      } 
      else { 
       widget.updateAttributes(widgetConfig); 
      } 
     }); 
} 

function createOrUpdateWidgets(widgetConfigObjects) { 
    var promises = []; 

    widgetConfigObjects.forEach(function(widgetConfig) { 
     promises.push(createOrUpdateWidget(widgetConfig)); 
    }); 

    return Sequelize.Promise.all(promises); 
} 


createOrUpdateWidgets([...]) 
    .done(function() { 
     console.log('Done!'); 
    }); 

Это, кажется, работает хорошо, но я не уверен, если я делаю это «правильно» или нет. Все обещания, которые выполняют взаимодействия с БД, нужно запускать поочередно или как я их определяю? Есть ли лучший способ сделать это?

Answer 1

Что вы делаете довольно идиоматических и прекрасно, единственное место для совершенствования, чтобы использовать тот факт, Sequelize использует Bluebird обещаний, так что вы получите .map бесплатно, который позволяет конвертировать:

function createOrUpdateWidgets(widgetConfigObjects) { 
    var promises = []; 

    widgetConfigObjects.forEach(function(widgetConfig) { 
     promises.push(createOrUpdateWidget(widgetConfig)); 
    }); 

    return Sequelize.Promise.all(promises); 
} 

в:

function createOrUpdateWidgets(widgetConfigObjects) { 
    return Sequelize.Promise.map(widgetConfig, createOrUpdateWidget) 
} 

кроме этого незначительного улучшения - вы правильно цепочки обещаний и, кажется, правильно повесить его.