Стоит ли автоматически оборачивать в try catch?

Question

[alex4answ]

Добрый день, начал замечать что у меня очень много шаблонного кода, особенно в middlewares Express.

const getUser = async (id) => {
  try {
    const response =  await fetch(`/users/${id}`);
    const user = response.json();
    // делать что-то с user
  } catch(err) {
    console.error(err); // к примеру просто в консоль
  }
};

Стоит ли автоматически оборачивать "полезный" код в try catch ?

Что-нибудь типа Декоратора

const tryCatchDecorator = (fn) => {
  return (...args) => {
    try {
      fn(...args);
    } catch(err) {
      console.error(err.message);
    }
  };
};

Или этого

const tryCatchWrapper = (callback) => {
  try {
    callback();
  } catch(err) {
    console.error(err);
  }
};

const getUser = async (id) => {
  const response =  await fetch(`/users/${id}`);
  const user = response.json();
  // делать что-то с user
};

const getUserWithTryCatch =  tryCatchDecorator(getUser); // Первый способ
const getUserWithTryCatch =  tryCatchWrapper((id) => getUser(id)); // Второй способ

getUserWithTryCatch(1);
tryCatchWrapper(getUser(1));

Почему ?

P.S. Речь идет только о шаблонных try/catch

Comments

[Никита]

Стоит ли автоматически оборачивать "полезный" код в try catch ?

А почему бы и нет?

[Дмитрий Беляев]

Если в Вашем коде слишком много try-catch, то у Вашего кода проблемы. ИМХО

[alex4answ]

Дмитрий Беляев, Почему?
Например почти каждый middleware express (если там работа с бд) нужно обернуть в try/catch

Проблема не в том что их много, просто много мест где они нужны и все эти места шаблонные грубо говоря

[Дмитрий Беляев]

alex4answ, потому что try-catch создает неконтролируемый хаос в коде. Программа начинает вести себя непредсказуемо. Баги гарантированы.
Если в программе вылетает исключение (например чтение поля из undefined), я бы предпочел, чтоб программа упала, так я быстрее исправлю причину исключения и скорее всего оно даже на прод не попадет.
Если в программе возникает ошибка (например запрошенных данных нет в базе), то это нормальное состояние программы, с ним нужно работать или возвращать как результат, но никак этот результат не должен оказаться в механизме throw-try-catch

Answer 1

[Robur]

Если это объективно улучшит ваш код - то стоит. Если нет, то не стоит.
Сам по себе такой прием ничего плохого не несет.
Но если у вас это все в мидлварах, то проще написать один обработчик ошибок который будет ловить все исключения в мидлварах, добавить его в express аппу, и остальному коду разрешить бросать исключения.

Comments

[alex4answ]

Robur , У меня стоит обработчик ошибок, но вся проблема в том, что асинхронные ошибки нужно пробрасывать явно, next(error) (что я и делаю в catch, в middleware)
А у меня они почти все асинхронные

[Robur]

alex4answ, https://github.com/davidbanham/express-async-errors/ вот эта штука делает примерно то же самое но для асинхронных функций.

[alex4answ]

Robur, Спасибо, то что нужно