Проблема в запуске кода с Promises в javascript - PullRequest
Новая
       97

Проблема в запуске кода с Promises в javascript

0 голосов
/ 03 мая 2020

Описание программы 

Create 3 functions:
FuncA – will receive a string and returns it’s length
FuncB – will receive an array of strings and returns their total lengths (using
funcA) after 2 seconds.
FuncC - will receive an array of arrays of strings and returns their total lengths
(using FuncB)

Мое решение: 

    function funcA(s)
    {
        return s.length
    }

    function funcB(arr)
    {
        return new Promise(resolve =>
            {
                setTimeout(() =>
                {
                    let total = 0;
                    arr.forEach(element => {
                        total += funcA(element)
                    });
                    resolve(total)
                },2000)
            })
    }

    function funcC(arr)
    {      
            return new Promise(resolve =>
                {
                    let isFirst = true
                    //a <=> total
                    let total = arr.reduce(async (a,b) =>
                    {
                        if(isFirst) {
                           isFirst = false
                           return (await funcB(a) + await funcB(b))
                        }
                        else {//a <=> total
                            return (a + await funcB(b))
                        }
                    })
                    resolve(total)
                }) 
    }

Запуск:
fun cC ([["aa", "bbb", "tyui"], ["ccc"], ["dfghj", "aedtfr"]])). Then (x => console.log (x))

Результат: [обещание объекта] 11

В чем проблема?

Ответы [ 2 ]

0 голосов
/ 04 мая 2020

Это действительно запутанно.

  • Не помещайте бизнес-логи c в обратный вызов setTimeout. Разрешите только обещание, затем выполните работу в обратном вызове then или после await.
  • Всегда передавайте начальное значение reduce! Это заставит его работать с пустыми массивами и избавит от необходимости того, что действительно странное isFirst logi c.
  • total уже является обещанием. Не оборачивайте это без необходимости в new Promise!

Эти предложения приведут к 

function funcA(s) { return s.length }

function funcB(arr) {
    return new Promise(resolve => {
        setTimeout(resolve, 2000);
    }).then(() => {
        let total = 0;
        arr.forEach(element => {
            total += funcA(element)
        });
        return total;
    });
}

function funcC(arr) {      
    return arr.reduce(async (a,b) => {
        return await a + await funcB(b)
    }, Promise.resolve(0))
}

Однако reduce на самом деле не подходит для асинхронной работы . Вы должны использовать циклический подход в funcC и reduce в funcB там, где он подходит намного лучше: 

async function funcB(arr) {
    await new Promise(resolve => {
        setTimeout(resolve, 2000);
    });
    return arr.reduce((total, element) => total + funcA(element), 0);
}

async function funcC(arr) {
    let total = 0;
    for (const b of arr) {
        total += funcB(b);
    }
    return total;
}
0 голосов
/ 03 мая 2020

[исправил ответ]

вы должны сделать await a вместо a везде 

function funcC(arr)
{      
    return new Promise(resolve =>
        {
            let isFirst = true
            //a <=> total
            let total = arr.reduce(async (a,b) =>
            {
                if(isFirst) {
                    isFirst = false
                    return (await funcB(await a) + await funcB(b))
                }
                else {//a <=> total
                    return (await a + await funcB(b))
                }
            })
            resolve(total)
        }) 
}
...