Почему не резолвится промис?

Question

[Сева]

Cобственно, вот.
Куда копать, что читать? Спасибо.

var dc = function(data) {

    return new Promise(function(resolve, reject) {

        var filename = 'Test';

        var contract = function() { ... }

        var policy = function() { ... }

        var invoice = function() { ... }
        
        contract().then(invoice().then(policy().then(function() {
            console.log(filename); // Test
            resolve(filename); // UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 2): ReferenceError: filename is not defined
        })))
    })
}

Comments

[Николай]

Для начала избавились бы от callback hell'a.

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

Без полного кода ничего сказать нельзя, код должен работать.

P.S. Можно писать так, это намного читаемее:

contract()
.then(invoice)
.then(policy)
.then(function() {
    resolve(filename);
});

[catHD]

Зря вы игнорируете предупреждение о отсутствие reject обработки

Answer 1

[Risent Veber]

then(policy().
вместо этого возможно нужно написать
then(policy).
А вообще без кода функций contract, policy, invoice трудно судить, к тому же если contract возвращает promise - можно обойтись без создания внешнего promise.

UPDATE: переписал бы так

function dc(data) {
    var filename = 'Test';

    function contract() {
    	return new Promise(function(resolve, reject) {
    		var stream = ...

     		stream.on('finish', function() {
                console.log('stream is done');
                resolve();
            });

        })
    }

    function policy() {
    	var stream = ...
    	var filename = ...

        return new Promise(function(resolve, reject) {
            stream.on('finish', function() {
                console.log('Policy is done');
                resolve(filename);
            });

        })
    }

    function invoice() {
    	var stream = ...

        return new Promise(function(resolve, reject) {
            stream.on('finish', function() {
                console.log('Invoice is done');
                resolve();
            });
        })
    }

    return contract().then(invoice).then(policy).then(function() {
        console.log(filename);
        return filename;
    }).catch(function(error){ console.error(error); });
}

P.S. читать лучше YDKJS

Comments

[Сева]

Все эти функции возвращают промис, который резолвится пустым, просто resolve()
filename нигде не переопределяется, должно браться из замыкания (и берется, потому что в консоль все верно выводится же).

[Risent Veber]

Сева: покажите мне код

[Сева]

Risent Veber: сбросил на почту

[Risent Veber]

Сева: готово - обновил ответ

[Сева]

Risent Veber: ничего не изменилось, filename is not defined

[Risent Veber]

Сева: такого не может быть - значит ошибка в другом файле, который использует этот

[Сева]

Risent Veber: там все просто, как валенок:

app.post('/generate', function(req, res) {
doc.document(req.body).then(function() {
res.send(200);
});
})

[Risent Veber]

Сева: короче учись дебажить найди именно ту строчку на которую ругается интерпретор еще лучше - разбей свою простыню на несколько файлов