Почему возвращается пустой массив?

Question

[Дамир Шаниязов]

почему в моменте не пустой
statesDto.push(stateDto)
а в return он пустой?

async getProjectStates(project_id) {
        const states = await State.findAll({
            where: {
                project_id
            },
            order: [
                ['serial_number', 'ASC']
            ]
        })

        let statesDto = []
        if (states) {
             await states.map(async state => {
                 const tasks = await Task.findAll({
                     where: {
                         state_id: state.id
                     },
                     order: [
                         ['createdAt', 'DESC']
                     ]
                 })

                 let tasksDto = []
                 if (tasks) {
                     tasks.map(task => {
                         const taskDto = new TaskDto(task)
                         tasksDto.push(taskDto)
                     })
                 }

                 const stateDto = new StateDto(state)
                 stateDto.tasks = tasksDto
                 statesDto.push(stateDto)
             })
        }
        return statesDto
    }

Comments

[キム ファイブプラス]

наверное потому что return выполняется раньше, не? и вообще дичь какая то с кучей вложенных await. используй промисы.

[Сергей Сергей]

await states.map(async state => { здесь промис создаётся "в никуда", и его не ждут, и return statesDto остаётся пустым. Так вроде.
И Array.map() не так применяется.

[Дамир Шаниязов]

Сергей Сергей, foreach?

[Сергей Сергей]

Применён как forEach. Но так не годится, если нужны результаты по await, нужно return allSettled( map() ).

[WbICHA]

Сергей Сергей, Promise.all же.

[WbICHA]

キム ファイブプラス,

вообще дичь какая то с кучей вложенных await. используй промисы.

Это шутка такая, да?

[WbICHA]

Тут, конечно, лютая дичь, но к самим эвейтам она не имеет никакого отношения.

[Сергей Сергей]

WbICHA, тут какие-то отложенные поиски, вдруг отклонены будут.

[WbICHA]

Сергей Сергей, не должно, он идишки из бд берёт, да и разве файндАлл кинет ошибку, если ничего не найдено?
Конечно, тут ещё вопрос что за бд, но 99%, что это монго с монгусом. Правда, я его последний раз трогал 2 года назад, так что могу что-то упустить.)

Answer 1

[DeFaNJI]

1. await с .map не сработает, пока будет выполняться .map тебе уже вернет пустой массив, что бы этого избежать используй Promise.all(и тут применяй .map)

В твоем случае должно сработать примерно вот так:

async getProjectStates(project_id) {
    const states = await State.findAll({
        where: {
            project_id
        },
        order: [
            ['serial_number', 'ASC']
        ]
    })

    let statesDto = []

    if (states.length) {
        await Promise.all(states.map(async state => {
            const tasks = await Task.findAll({
                where: {
                    state_id: state.id
                },
                order: [
                    ['createdAt', 'DESC']
                ]
            })

            let tasksDto = []

            if (tasks.length) {
                tasks.map(task => {
                    const taskDto = new TaskDto(task)
                    tasksDto.push(taskDto)
                })
            }

            const stateDto = new StateDto(state)

            stateDto.tasks = tasksDto
            
            statesDto.push(stateDto)
        }))
    }

    return statesDto
}

2. Если это mongoose, то в states должен быть массив, а при запросе, который ничего не нашел должен возвращаться пустой массив. Пустой массив в JS все равно будет помечен как true, поэтому уместнее тут было бы делать условие по .length . Точно также с tasks в .map

const arr = []

console.log(Boolean(arr)) // true