0

Хочу передать в promise параметр с другого файла. Но из-за асинхроности (как мне кажется) выдаёт неверный ответ.

var promise_get_permissions = function (user_id_current) {
return new Promise(function (resolve, reject) {
        console.log(user_id_current);
        var result_array = [];

        connection.UserRoles
            .findAndCountAll({
                where: {
                    userId: user_id_current
                }
            })
            .then(function (result) {
                var role_id_current;
                for (var i = result.count - 1; i >= 0; i--) {
                    role_id_current = result.rows[i].dataValues["roleId"];

                    connection.RolePermissions
                        .findAndCountAll({
                            where: {
                                roleId: role_id_current
                            }
                        })
                        .then(function (result) {
                            var permission_id_current;
                            //console.log("permissions result:", result);
                            for (var i = result.count - 1; i >= 0; i--) {
                                permission_id_current = result.rows[i].dataValues["permissionId"];

                                connection.Permissions
                                    .findById(permission_id_current)
                                    .then(function (result) {
                                        result_array.push(result.dataValues["code"]);
                                    })
                            }
                        })
                        .then(function () {
                            unique(result_array);
                        });
                }
            })
            .then(function () {
                unique(result_array);
                resolve(result_array);
            })
});
};

Вызываю код, который выше, здесь:

func_auth.promise_get_permissions(request.user)
    .then(function (result) {
        console.log(result);
        for (var i = result.length - 1; i >= 0; i--) {
            get_permissions_array[result[i]] = true;
            // console.log(get_permissions_array[result[i]]);
        }
        console.log(get_permissions_array);
    })
    .then(function(){
        response.render("main.hbs",{
            is_auth_visible: request.isAuthenticated(),
            get_permissions: get_permissions_array
        });
    })

но в итоге result пустой, а в нём должны бытьcodeы Укажите на ошибку или предложите альтернативный вариант. Заранее спасибо!

И еще вопрос! Можно ли заставить ждать пока сработают все thenы?

12
  • Проверьте каскад промисов, какой то then вызывается не в том порядке. 23 авг 2017 в 10:50
  • я пробовал не оборачивать promise в функцию и вызывать как промис, а не как функции. В таком случае всё работает нормально. Но потом мне нужно было передать параметр и ничего не меняя я решил обернуть promise в функцию. И эта функция стала выдавать не верный ответ. 23 авг 2017 в 11:05
  • То есть тот then, где я заполняю массив result_array, не успевает сработать перед resolve. Сначала происходит resolve, потом этот then 23 авг 2017 в 11:11
  • да, нужно это проверить 23 авг 2017 в 11:12
  • законсольте в then (во всех) вашу переменную. И проверьте последовательность 23 авг 2017 в 11:13

1 ответ 1

1

Для того, чтобы цепочка вычислений шла по порядку, нужно чтобы каждый обработчик then возвращал очередной Promise, который надо дождаться.

В приведенном коде в нескольких местах асинхронная функция просто запускается и нигде не отслеживается когда она закончится.

Например:

connection.RolePermissions
    .findAndCountAll(...)

...
connection.Permissions
    .findById(permission_id_current)

Вместо этого, нужно возвращать Promise, которые получены из этих функций.

В случае, когда операции выполняются в цикле, и каждая возвращает Promise, результаты можно собрать в массив и для ожидания окончания всех использовать Promise.all

В итоге может код может выглядеть так:

var promise_get_permissions = function (user_id_current) {
    console.log(user_id_current);

    return connection.UserRoles
        .findAndCountAll({
            where: {
                userId: user_id_current
            }
        })
        .then(function (userRole) {
            var role_id_current;
            var rolePermissions = [];
            for (var i = userRole.count - 1; i >= 0; i--) {
                role_id_current = userRole.rows[i].dataValues["roleId"];

                rolePermissions.push(connection.RolePermissions
                    .findAndCountAll({
                        where: {
                            roleId: role_id_current
                        }
                    }));
            }
            return Promise.all(rolePermissions);
        })
        .then(function (rolePermissions) {

            return Promise.all(rolePermissions.map(rolePermission=>{
                var permissions = [];
                for (var i = rolePermission.count - 1; i >= 0; i--) {
                    var permission_id_current = rolePermission.rows[i].dataValues["permissionId"];

                    permissions.push(connection.Permissions.findById(permission_id_current));
                }
                return Promise.all(permissions);
            }));
        })
        .then(function(permissions){
            return permissions.reduce((acc,permission)=>acc.concat(
                permission.map(p=>p.dataValues["code"])
            ),[]);
        })
        .then(function(resul){
            unique(result);
            return result;
        })
};

Можно обратить внимание, что теперь нет вложенных then и все выполняется последовательно.

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.