Как переписать это на промисах?

Question

[jeruthadam]

Как этот код переписать на промисах?

fs.readFile('./data.json', (err, data) => {
  if (err) {
    console.error('unable to read the file')
  }
  else {
    try {
      data = JSON.parse(data)
      console.log(data.name)
    }
    catch (e) {
      console.error('invalid JSON in file')
    }
  }
})

Я сделал так, но работает некорректно, постоянно возвращает первый кетч

const promise = new Promise ((resolve, reject) => {
  fs.readFile('./data.json', (err, data) => {
    if (err) {
      reject()
    }
    else {
      try {
        resolve()
      }
      catch (e) {
        reject()
      }
    }
  })
})

promise
  .then((data) => {
    data = JSON.parse(data)
    console.log(data.name)
  })
  .catch((e) => {
    console.error('unable to read the file')
  })
  .catch(SyntaxError, (e) => {
    console.error('invalid JSON in file')
  })

Answer 1

[Антон]

const {promisify} = require('util');

const fs = require('fs');
const readFileAsync = promisify(fs.readFile); 

readFileAsync(filePath)
  .then((text) => {
      console.log('CONTENT:', text);
  })
  .catch((err) => {
      console.log('ERROR:', err);
  });

Comments

[jeruthadam]

1. откуда взялся readFileAsync? Я не юзаю блуберд
2. у меня 2 эррор хендлинга. как второй делать?

[Антон]

jeruthadam,
1. присмотритесь
2. так, же как и было раньше

[jeruthadam]

1. util нашел
2. так же как раньше не работает.

[jeruthadam]

readFileAsync('./data.json')
  .then(JSON.parse)
  .then((val) => {
    console.log(val.name)
  })
  .catch((err) => {
    console.error('invalid JSON in file')
  })
  .catch((err) => {
    console.error('unable to read the file')
  })

Возвращает только первый эррор.

[Антон]

jeruthadam, попробуйте SyntaxError поставить первым

[jeruthadam]

Антон, без разницы, пробовал

[jeruthadam]

возвращается пустая консоль.

[napa3um]

Катчи не каскадируются автоматом как Тхены при возврате undefined, нужно явно передавать ошибку дальше с помощью Promise.reject или throw:

Promise.reject(new Error('Test error'))
.catch(err => {
    console.log('Error handler 1', err)
    throw err
})
.catch(err => {
    console.log('Error handler 2', err)
    return Promise.reject(err)
})
.catch(err => {
    console.log('Error handler 3', err)
})
.catch(err => {
    console.log('Not executed handler', err)
})

Answer 2

[kn1ght_t]

на ноде ни разу не писал, но попробуйте так:

function readFileAsync(path) {
  return new Promise(function (resolve, reject) {
    fs.readFile(path, function (error, result) {
      if (error) {
        reject(error);
      } else {
        resolve(result);
      }
    });
  });
}

(async function() {
  try {
    const data = await readFileAsync('./data.json');
  } catch(error) {
    console.log(error);
  }
})();

Comments

[jeruthadam]

1. не надо async await, я до них еще не дошел
2. у меня 2 эррор хендлинга

[kn1ght_t]

2. в промисах разве бывает 2 эррор хендлинга?

[jeruthadam]

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

[napa3um]

jeruthadam, ошибка при выполнении промиса всегда только одна, промис либо резолвится, либо реджектится, другого не дано (однако промисы могут каскадироваться, и из обработчика резолва или реджекта можно вернуть новый промис, в том числе отрежекченный, т.е., ошибку).

Answer 3

[0xD34F]

постоянно возвращает первый кетч

Неудивительно - ведь вы ничего не передаёте в resolve. Так что вы парсите не содержимое файла, а undefined.

Comments

[jeruthadam]

В резолв как раз все возвращается как надо. С ошибками непонятн окак быть.

[0xD34F]

У вас написано:

try {
  resolve()
}

Так что непонятно, каким образом data попадает в resolve.

[jeruthadam]

0xD34F, да, не попадает. что вы предлагаете? data добавляю ноль эффекта.

[0xD34F]

"Не попадает", а три минуты назад всё было "как надо"... Что-то я запутался.

Попробуйте указать кодировку при чтении:

fs.readFile('./data.json', 'utf8', (err, data) => ...

[jeruthadam]

0xD34F,

SyntaxError: Unexpected token / in JSON at position 0

[0xD34F]

Попробуйте вырезать BOM перед тем как парсить:
data = JSON.parse(data.replace(/^\uFEFF/, ''))

[jeruthadam]

Шозабред)))

[jeruthadam]

Проблема явно не в этом вообще.

[0xD34F]

jeruthadam, ещё раз - попробуйте вырезать BOM. Или страдайте дальше.

[jeruthadam]

0xD34F, да бред это все, потому что reject тоже не работает.

[0xD34F]

jeruthadam, бред? Что ж, видимо вы и сами способны разобраться со своей проблемой. Удачи!

[jeruthadam]

0xD34F, ну от тебя ее точно не дождешся