Где лучше использовать промиссы, а не колбек?

Question

[Елена]

Народ, где лучше использовать промиссы, а не колбек?

app.use('/', (req, res) => res.status(404).send({ message: 'Запрашиваемый ресурс не найден' }));

router.get('/', (req, res) => {
  readerFunction(usersPath, res).on('open', () => {
    res.writeHead(200, { 'Content-type': 'application/json; charset=utf-8' });
    readerFunction(usersPath, res).pipe(res);
  });
});

router.get('/:id', ((req, res) => {
  readerFunction(usersPath, res).on('open', () => {
    const users = jsonParse(usersPath);
    const user = users.find((item) => item._id === req.params.id);
    if (!user) res.status(404).send({ message: 'Нет пользователя с таким id' });
    res.send(user);
  });
}));

const readerFunction = (fileName, res) => {
  const reader = fs.createReadStream(fileName, { encoding: 'utf8' });
  reader.on('error', () => {
    res.status(500).json({ error: 'На сервере произошла ошибка' });
  });
  return reader;
};
const jsonParse = (fileName) => JSON.parse(fs.readFileSync(fileName, { encoding: 'utf8' }));

Comments

[Robur]

И к каким именно строчкам этой простыни относится ваш вопрос?

[Егор Живагин]

Я не вижу тут коллбэков, только обработчики

[Елена]

Robur, :-) Из всего этого кода. Я как понимаю, что где-то лучше изменить на промиссы, а где-то оставить колбек. Вот и не знаю, где именно изменить

[Елена]

Егор Живагин, вот и я не понимаю, где надо изменить на промиссы

[Robur]

Elena0394, так может и не надо. С чего вы взяли что вам "надо"?
1. Если библиотека написана так что надо передавать колбек а не промис - надо передавать колбек.
2. Все эти функции на промисы просто так не меняются - у них совсем другая логика, промисы срабатывают только один раз, а эти - вызываются много раз. То что вы куда то передаете функцию, не значит что это "callback hell" который надо менять на промисы. Ваш код вообще другой структуры и логики, он нормальный, оставьте его в покое ).

[Елена]

Robur, спасибо) В ревью было подмечено, что лучше при работе с файлами использовать промиссы. а не колбеки. А работа с файлами, как я понимаю, происходит в этих методах:

const reader = fs.createReadStream(fileName, { encoding: 'utf8' });
const jsonParse = (fileName) => JSON.parse(fs.readFileSync(fileName, { encoding: 'utf8' }));

а именно: createReadStream и readFileSync.
Конечно мне не сказали, прям жестко исправить, а посоветовали, что можно лучше сделать. Вот хочу понять, а как сделать через промиссы. Здесь я совсем не понимаю где использовать промиссы. Вот есть у меня такой кусок кода:

getUserInfo = () => {
    return fetch(`${this.options.baseUrl}/users/me`, this.options.token)
      .then(res => res.ok ? res.json() : Promise.reject(res.status))
      .catch(err => console.log(`Ошибка: ${err}`))
  };

здесь я понимаю, что делают промиссы, а вот в тех кусах кода - не понимаю

[Robur]

Elena0394,

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

можно воспользоваться https://nodejs.org/api/util.html#util_util_promisi...

можно таким кодом

const readPromise = new Promise((resolve, reject) => {
fs.readFile(filename, (err, data) => {
  if (err) reject(err);
  else resolve(data);
});
})

readPromise.then(...).catch(...)

по сути promisify что-то такое и делает.