0
app.post('/post',async ({body}, res) => {
    const validatePost = await foo(body)
    const postResult = await foo2(body)

    res.status(200).json(validatePost ? postResult : {success:false})
})

foo и foo2 это асинхронные функции. Проблема в том что foo2 выполняется обходя условие, поэтому когда ей передаются неверные параметры - происходит краш. Насколько я знаю await должно противостоять этому.

4

1 ответ 1

2

Код покороче - не всегда надежен, к тому же каждый await необходимо оборачивать в try/catch и выводить ошибки в лог (но не на продакшн). Иначе вы рискуете потерять контекст ошибки, как минимум. Ну и для тестов желательно все кейсы прорабатывать, чтобы потом в продакшене у вас неыбло проблем.

app.post('/post',async ({body}, res) => {
    let validatePost;
    let postResult;

    try {
        validatePost = await foo(body);
    } catch(err) {
        console.error(err);
        res.status(500).json({ success: false });
        return;
    }

    if (!validatePost) {
        res.status(400).json({ success: false });
        return;
    }

    try {
        postResult = await foo2(body);
    } catch(err) {
        console.error(err);
        res.status(500).json({ success: false });
        return;
    }

    res.status(200).json(postResult);
});

или так:

class ValidationError extends Error {}

app.post('/post',async ({body}, res) => {
    let postResult;

    try {
        const validatePost = await foo(body);

        if (!validatePost) {
            throw new ValidationError('Some Validation Error');
        }

        postResult = await foo2(body);
    } catch(err) {
        const status = err instanceof ValidationError ? 400 : 500;
        res.status(status).json({ success: false });
        console.error(err);
        return;
    }

    res.status(200).json(postResult);
});

1
  • 1
    обработчик ошибок выгоднее вынести в экспрессовский миддлваре 10 авг 2021 в 18:16

Ваш ответ

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

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