Как в Promise.then() передать результат предыдущего then?

Question

[Danbka]

Привет. Вопрос конечно сформулирован коряво, но объяснить можно только на примере.

Итак, работаю с Rutoken Plugin, используя промисы. Есть следующий код:

RuTokenPlugin.enumerateDevices().then(function(devices) {
			// Получил устройства
			return devices;
		}).then(function(devices) {
			// с каждого устройства получаю идентификаторы сертификатов, с помощью Promise.all() передаю их дальше
			var certPromises = [];
			for (var d in devices) {
				certPromises.push(RuTokenPlugin.enumerateCertificates(d, RuTokenPlugin.CERT_CATEGORY_USER));
			}
			return Promise.all(certPromises);
		}).then(function(certificates) {
			// массив с идентификаторами сертификатов
		})

Проблема в том, что, получив идентификаторы сертификатов, я не могу далее воспользоваться методом getCertificate(deviceId, certId), т.к. он принимает не только идентификатор сертификата, но и идентификатор устройства, которого в последнем then() у меня нет.

Как изменить код, чтобы можно было воспользоваться getCertificate()?

Answer 1

[Николай]

Идея примерно такая, device можно закинуть через замыкание.

function getPromise(d) {
    return RuTokenPlugin.enumerateCertificates(d, RuTokenPlugin.CERT_CATEGORY_USER)
        .then(function(cert) {
             return {
                 device: d,
                 cert: cert
            };
        });
}

Comments

[Алексей Уколов]

Да, примерно так я сделал у себя в вопросе, на который дал ссылку. Но в данном случае, мне кажется, проще обойтись глобальной переменной и индексами массива.

[Danbka]

Я правильно понял, что вы имеете ввиду нечто подобное:

function getCertificates(d) {
return new Promise(function(resolve, reject) {
RuTokenPlugin.enumerateCertificates(d, RuTokenPlugin.CERT_CATEGORY_UNSPEC).then(function(certs) {
resolve({device: d, certificates: certs});
})
});
}

var certPromises = [];
for (var d in devices) {
certPromises.push(getCertificates(d));
}
return Promise.all(certPromises);

В таком случае я вроде как действительно получил массив объектов, каждый из которых содержит идентификатор устройства и массив сертификатов

[Алексей Уколов]

Не обязательно создавать промис, раз enumerateCertificates уже его возвращает. Код, который вам предложили, вполне рабочий.

Answer 2

[Алексей Уколов]

Самый простой и понятный способ - сохранять devices в какую-то внешнюю переменную, глобальную для всей цепочки и обращаться к ней.
Можно ли упростить асинхронный код на Promise?

Comments

[Danbka]

Конечно, я думал об этом. Но нужно соответствие "Идентификатор устройства" - "Сертификаты с него". Просто сохранять devices не имеет смысла.

[Алексей Уколов]

Насколько я понимаю, они просто по индексу соотносятся. В последний then же массив массивов приходит? Сложно что-то посоветовать не понимая структуры данных.

[Алексей Уколов]

И каждый массив в массиве certificates соответствует устройству с таким же индексом в массиве devices.

[Алексей Уколов]

Александр +: я вам под вашим ответом написал почему вы неправы.

[Алексей Уколов]

Danbka: вы уверены, что certificates - это именно "массив с идентификаторами сертификатов", а не массив массивов? Судя по коду, должно быть именно так.

[Danbka]

Алексей Уколов: да, это массив массивов. Я не стал описывать такие подробности, не думая, что они важны.

[Алексей Уколов]

Danbka: ну вот, значит вас спасёт глобальная переменная и соответствие индексов в двух массивах.