Как правильно ловить ошибки в промисах?

Question

[GrigoryMorozov]

Есть вот такая функция для обращения к API:

async api(method, params = {}) {
	// ...

	if (data.response) {
		return data.response;
	}

	else {
		throw new Error(data.error);
	}
}

Предполагается, что сообщения об ошибках должны выводиться в модальном окне. Нужно понять, как наиболее правильно это реализовать. Пока что есть вот такие варианты:

1. Добавить вызов функции modal() прямо прямо перед инструкцией throw;

2. Ловить ошибки с помощью конструкции try ... catch;

async init() {
	// ...

	try {
		let response = await api('users.list');
	}

	catch (e) {
		modal(e.text);
	}
}

3. Ловить ошибки с помощью catch (хотя это не очень соответствует стилю async/await).

async function init() {
	// ...

	let response = await api('users.list').catch(e => modal(e.text));
}

Какой вариант в данной ситуации лучше использовать? Хочется писать как можно меньше кода (способ обработки ошибок не будет зависеть от их характера) и иметь возможность выйти из функции init() после обработки ошибки.

Буду благодарен за ответы.

Answer 1

[Alexandroppolus]

Второй вариант.

Первый очень плох из-за того, что api прибивается гвоздями к какому-то modal, хотя должно заниматься только общением с сервером, в общем, полнейший бардак и никакого разделения на слои. Третий не удобен, потому что в таком виде в случае ошибки response будет равен значению, возвращаемому функцией modal, это надо проверять отдельно и всё равно выходить. Во втором всё как положено - либо работаем с респонсом, либо вываливаемся в catch

Comments

[GrigoryMorozov]

Спасибо за ответ. Вообще, да, использование try ... catch выглядит здесь наиболее подходящим вариантом. И всё же не хотелось бы оборачивать в эту конструкцию каждый запрос к API.

Мне в голову пришла вот такая идея:

async function errorHandler(error) {
  modal(error.text);

  throw error;
}

async function init() {
  // ...

  let response = await api('users.list').catch(errorHandler);
}

Насколько разумно применять вот такой код?

[Alexandroppolus]

В этом случае всё равно придется делать try ... catch, ведь новый промис теперь отклоненный.

[GrigoryMorozov]

Alexandroppolus, попробовал всё это реализовать. Получил тот результат, который хотел.