Как правильно записать Promise?

Question

[IvanIvanIvanIvanIvan]

У менять есть функция

export const getNews = async () => {
    let news;
    try {
      fetch(`https://newsapi.org/v2/top-headlines?country=us&apiKey='''''''''`)
        .then(res => res.json())
        .then(res => {
            news = res;
        })
    } catch(e){
      throw e;
    }
    return news;
};

Она делает запрос на сервер и получает новости (я проверял в консоль выводит)

Как мне вызвать правильно эту функция, и получить новости

getNews()
						.then(result => {
							console.log(result);
						})
						.catch(error => {
							console.log(error);
						});

У меня result выводит undifined

Answer 1

[Stockholm Syndrome]

export const getNews = async () => {
  let news;
  try {
    news = await fetch(`https://newsapi.org/v2/top-headlines?country=us&apiKey='''''''''`).then(res => res.json());
  } catch(e){
    throw e;
  }
  return news;
};

Answer 2

[Владимир Проскурин]

Не нужно использовать try...catch, вы же используете промисы, а у них есть своя реализация обработки ошибок.

export const getNews = async () => {
    return fetch(`https://newsapi.org/v2/top-headlines?country=us&apiKey='''''''''`)
        .then(res => res.json())
        .then(res => {
            return res;
        });
};

и потом просто используете

getNews()
            .then(news => {
              console.log(news);
            })
            .catch(error => {
              console.log(error);
            });

Comments

[Fedor Rychkov]

Мне кажется в этом варианте async не обязателен, у фетча и так промис обертка.