Как выполнить массив промисов в redux?

Question

[Максим Корольский]

export function homePageProfileAction(list, name) {
    return (dispatch) => {
        dispatch(homePageProfileFetchId(list, name));

        let promise = list.map(v => {
            console.log(getUserHome(v));
            return getUserHome(v).then(id => {
                dispatch(homePageProfileFetchSuccess(id, v));
            });
        });

        // Promise.all([promise])
        //     .then((res) => {
        //         console.log(res);
        //         dispatch(homePageProfileFetchSuccess(res, "top"));
        //     })
        //     .catch(err => {console.log(err); });
    };
}

Есть вот такая функция. Я получаю массив с id. Выполняю запрос к базе (это и есть промис)
Но хотелось бы что бы все выполнилось одновременно и задиспачить один успех, вместо 4х
Promise.all я попробовал, но там что то зацикливается. Не могу понять что

Answer 1

[nd0ut]

export function homePageProfileAction(list, name) {
    return (dispatch) => {
        dispatch(homePageProfileFetchId(list, name));

        const promises = list.map(v => getUserHome(v));

        Promise.all(promises)
            .then((res) => {
                dispatch(homePageProfileFetchSuccess(res, "top"));
            })
            .catch(err => {console.log(err); });
    };
}

Comments

[nd0ut]

А вообще, список должен принимать #getUserHome и прокидывать в запрос к API.

[Виталий]

nd0ut: А вот объясните мне , зачем возвращать функцию, которая ничего не возвращает, хотите что-то возвращать, возвращайте промис, а дальше вызывайте функцию с then catch. А еще ответьте мне на вопрос, что будет если хотябы один из промисов улетил в ошибку? На сколько корректно все отработает с промисс алл....

[nd0ut]

Виталий: Так реализуются асинхронные экшены в redux. Возвращаемой функции не обязательно что-либо возвращать, её задача - получить данные и дёрнуть plain экшены. Что касается reject внутри Promise.all, то да, он забьёт на свою работу и вернёт аргумент reject'a. Поэтому я и написал, что лучше прокидывать список id в API, дёргать его один раз и не использовать Promise.all.

[Максим Корольский]

nd0ut да согласен с вами, но в данном случае, это еще не реализовано, приходится выкручиваться

Answer 2

[Владимир Шестаков]

Promise.all вернет массив ответов всех выполненных запросов. Смотри, что в then каждого запроса возвращаешь и как потом обрабатываешь.

export function homePageProfileAction(list, name) {
    return (dispatch) => {
        dispatch(homePageProfileFetchId(list, name));

        let promise = list.map(v => {
            return getUserHome(v);
        });

        Promise.all(promise)
             .then((ids) => {
                 console.log(ids);
                 //dispatch(homePageProfileFetchSuccess(res, "top"));
             })
             .catch(err => {console.log(err); });
    };
}