Асинхронное выполнение кода JS + node на примере?

Question

[Андрей]

Добрый день!
Есть некий код парсера, страница новостей взята за пример.
Хочу понять, насколько код оптимально и правильно написан в отношении очередности выполнения? Особенно интересует его асинхронное выполнение. Возможно есть белее простое решение.

const request = require ('request');
const cheerio = require ('cheerio');
let news = [];
const ps = new Promise((resolve) => {
request ('https://echo.msk.ru/news/',(error, response, body)=>{
    if (!error && response.statusCode == 200){}
        const $ = cheerio.load (body);
        $('h3 a').each(function() {
           news.push($(this).text());
        });
        resolve(news);
        });
});
ps.then(() => {
    console.log(news);
  });

Comments

[DanKud]

Если вам так удобно, то все нормально, но я бы лично предпочел асинхронный код писать через async/await, чтобы работать с полученными данными дальше в основном коде, а не внутри callback-функции, как у вас:

const request = require('request');
const cheerio = require('cheerio');

(async () => {
  let news = [];
  
  const requestBody = await new Promise((resolve) => {
    request('https://echo.msk.ru/news/', (error, response, body) => {
      if (!error && response.statusCode == 200) {
        resolve(body);
      }
    });
  });

  const $ = cheerio.load(requestBody);
  $('h3 a').each(function () {
    news.push($(this).text());
  });
  
  console.log(news);
})();

Ну и раз уж вы собрались писать асинхронный код на промисах, то лучше использовать библиотеку
request-promise-native или axios, чтобы вам сразу же отдавался промис при запросе.

[Андрей]

DanKud, вы правы, похоже это решение оптимально

[Андрей]

DanKud, так будет правильно? Или код не оптимален? Извиняюсь за назойливость, я новичок в программировании.

const axios = require('axios');
const cheerio = require  ('cheerio');
const getNews = async () => {
  try {
    return await axios.get('https://echo.msk.ru/news/');
  } catch (error){
    console.log (error);
  }
}
const resNews = async () => {
  const news = await getNews();
  $ = cheerio.load(news.data);
  $('h3 a').each(function () {
    console.log( $(this).text());
  });
}
resNews();