Какие у вас замечания по этому коду? Как писать правильно?

Question

[qwerty]

Правильно ли я вывожу ошибки? Как бы вы написали этот код?

const http = require('http');

async function getBody(url) {
    return new Promise((resolve, reject) => {
        http.get(url, (res) => {
            if (res.statusCode !== 200) {
                throw new Error('Request Failed.\n' +
                    `Status Code: ${res.statusCode}`);
            }

            res.setEncoding('utf8');
            let rawData = '';

            res.on('data', (chunk) => {
                rawData += chunk;
            });

            res.on('end', () => {
                return resolve(rawData);
            });

        });
    });
}

(async () => {
	try {
		let body = await getBody('http://site.com/');
		console.log(body);
	} catch (err) {
		console.error(err);
	}
})();

Answer 1

[Виталий]

Резольв без ретурн, вместо фроу реджект, в него Нью эррор
Гетбоди писать без асинк, ты возвращаешь промис из обычной функции

Comments

[qwerty]

Спасибо!

async function getYandexDisk(url) {
    return new Promise((resolve, reject) => {
        http.get(url, (res) => {

            if (res.statusCode !== 200) {
                reject(new Error('Request Failed.\n' +
                    `Status Code: ${res.statusCode}`));
            }

            res.setEncoding('utf8');
            let rawData = '';

            res.on('data', (chunk) => {
                rawData += chunk;
            });

            res.on('end', () => {
                let pattern = /href="(https:\/\/yadi.sk.+?)"/;
                let arr = rawData.match(pattern);

                if (arr != null) {
                    resolve(arr[1]);
                } else {
                    reject(new Error('Yandex disk url not found'));
                }
            });

        });
    });
}

[qwerty]

Спасибо. Я как раз насчет этого async и думал, теперь стало понятно

[Виталий]

Egor, getYandexDisk перед ней убери асинк объявление. Ечли ты делаешь через ретурн Нью промис, это не асинк функция

[Виталий]

Egor, поставь равенство для себя между асинк и ретурн промис, это две похожих по сущности записей. Плюс промиса в том что завершение идёт через колбек функции, а значит ты можешь внутри промиса делать всякие вермишели типа эвентэмиттеров и завершать промис в то время как пришел в эмиттер колбек, такие фичи в асинк функцию не засунуть.

[qwerty]

Виталий, да, я убрал, изначально делал с async без промисов (тк читал что при создании асинхронной функции она автоматически возвращает промисы, но я не понял как в таком случае вернуть результат из http.get(){}
Вот итоговый код

const http = require('http');

function getYandexDisk(url) {
    return new Promise((resolve, reject) => {
        http.get(url, (res) => {

            if (res.statusCode !== 200) {
                reject(new Error('Request Failed.\n' +
                    `Status Code: ${res.statusCode}`));
            }

            res.setEncoding('utf8');
            let rawData = '';

            res.on('data', (chunk) => {
                rawData += chunk;
            });

            res.on('end', () => {
                let pattern = /href="(https:\/\/yadi.sk.+?)"/;
                let arr = rawData.match(pattern);

                (arr != null) 
                ? resolve(arr[1])
                : reject(new Error('Yandex disk url not found'));
            });

        });
    });
}

(async () => {
	try {
		let yandexDisk = await getYandexDisk('http://site.com/');
		console.log(yandexDisk);
	} catch (err) {
		console.error(err);
	}
})();

[Тимур Шемсединов]

О Всемогущий Аллах, они не ведают, а Ты - Знающий, Мудрый!

const http = require('http');

const fetch = url => new Promise((resolve, reject) => http.get(url, res => {
  if (res.statusCode !== 200) {
    reject(`Request Failed.\nStatus Code: ${res.statusCode}`);
    return;
  }
  res.setEncoding('utf8');
  const buffers = [];
  res.on('data', chunk => buffers.push(chunk));
  res.on('end', () => resolve(Buffer.concat(buffers)));
}));

fetch('http://site.com/')
  .then(body => console.log(body))
  .catch(err => console.error(err));

[qwerty]

Тимур Шемсединов, для чего нужен return; после reject(`Request Failed.\nStatus Code: ${res.statusCode}`)?

И еще, если функция ничего не возвращает нужно ли возвращать состояние промиса?
Вот пример

function writeFile(fileName, data) {
    return new Promise((resolve, reject) => {
        fs.writeFile(fileName, data, (err) => {
            if (err) reject(new Error(err));
            resolve(); // <======
        });
    });
}

[Тимур Шемсединов]

Egor, после reject нужно выйти из функции, иначе код после ифа исполнится.
Второй вопрос не понятен. У меня объект промиса возвращается.

[qwerty]

Тимур Шемсединов, ну, нужно ли мне писать resolve(); в конце выполнения функции, пустой resolve(). Я просто не пойму работает и так и так)

[Тимур Шемсединов]

Egor, можно писать, но так делать - это лишнее обворачивание, лучше использовать универсальный util.promisify https://nodejs.org/api/util.html#util_util_promisi...