Телеграмм чат группы nodejs

exports.json = (url) => {
  return new Promise((resolve, reject) => {
    https.get(url, (res) => {
      let data = '';
      res.on('data', (chunk) => { data += chunk; });
      res.on('end', () => {
          resolve(JSON.parse(data,"utf8"));
          console.log("ERRRR")
          reject(e.message);
      });
    })
  });
}

так?

The Fallen Phoenix in Node.js — русскоговорящее сообщество

15:32пожаловаться #1

ᐺ ! ᖇ ᖶ ᒪ

а зачем try catch тогда? внутри промиса

Ну это уже мелочи по сравнению с тем что каналы передачи ошибок друг о друге ничего не знают

ᐺ ! ᖇ ᖶ ᒪ in Node.js — русскоговорящее сообщество

15:32пожаловаться #2

ᐺ

exports.json = (url) => {
  return new Promise((resolve, reject) => {
    https.get(url, (res) => {
      let data = '';
      res.on('data', (chunk) => { data += chunk; });
      res.on('end', () => {
          resolve(JSON.parse(data,"utf8"));
          console.log("ERRRR")
          reject(e.message);
      });
    })
  });
}

так?

у тебя нет объекта с ошибкой поэтому не так

ᐺ ! ᖇ ᖶ ᒪ in Node.js — русскоговорящее сообщество

15:33пожаловаться #3

ᐺ

не знаю хорошо ли это или плохо но существует event-to-promise или типа того модуль

¿hope in Node.js — русскоговорящее сообщество

15:33пожаловаться #4

exports.json = (url) => {
  return new Promise((resolve, reject) => {
    https.get(url, (res) => {
      let data = '';
      res.on('data', (chunk) => { data += chunk; });
      res.on('end', () => {
          resolve(JSON.parse(data,"utf8"));
          console.log("ERRRR")
          reject(e.message);
      });
    })
  });
}

так?

А если там данные придут не в формате json?

Pavel Shakhov (pongo) in Node.js — русскоговорящее сообщество

15:35пожаловаться #5

exports.json = (url) => {
  return new Promise((resolve, reject) => {
    https.get(url, (res) => {
      let data = '';
      res.on('data', (chunk) => { data += chunk; });
      res.on('end', () => {
        try {
          resolve(JSON.parse(data,"utf8"));
        } catch (e) {
          console.log("ERRRR")
          reject(e.message);
        }
      });
    })
  });
}

запустил твой. да ловится ошибка, все норм.

json('http://worldtimeapi.org/wrongurl').then(x => console.log(x)).catch(console.error);

15:36пожаловаться #6

¿hope

А если там данные придут не в формате json?

тогда тоже плохо

The Fallen Phoenix in Node.js — русскоговорящее сообщество

15:36пожаловаться #7

exports.json = (url) => {
  return new Promise((resolve, reject) => {
    https.get(url, (res) => {
      let data = '';
      res.on('data', (chunk) => { data += chunk; });
      res.on('end', () => {
          resolve(JSON.parse(data,"utf8"));
          console.log("ERRRR")
          reject(e.message);
      });
    })
  });
}

так?

Явный конструктор промисов это антипаттерн

Pavel Shakhov (pongo) in Node.js — русскоговорящее сообщество

15:36пожаловаться #8

The Fallen Phoenix

Явный конструктор промисов это антипаттерн

пруф

The Fallen Phoenix in Node.js — русскоговорящее сообщество

15:36пожаловаться #9

Не без причины. Используй промисифай на функцию с коллбеком

The Fallen Phoenix in Node.js — русскоговорящее сообщество

15:36пожаловаться #10

пруф

В прикладной разработке*

¿hope in Node.js — русскоговорящее сообщество

15:37пожаловаться #11

The Fallen Phoenix

Не без причины. Используй промисифай на функцию с коллбеком

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

Pavel Shakhov (pongo) in Node.js — русскоговорящее сообщество

15:37пожаловаться #12

The Fallen Phoenix

В прикладной разработке*

все еще не вижу причины

15:37пожаловаться #13

exports.json = (url) => {
  return new Promise((resolve, reject) => {
    https.get(url, (res) => {
      let data = '';
      res.on('data', (chunk) => { data += chunk; });
      res.on('end', () => {
        resolve(JSON.parse(data,"utf8"));
      });
    }).on('error', (e) => {
      reject(e.message);
    })
  });
}

вот теперь ловится

The Fallen Phoenix in Node.js — русскоговорящее сообщество

15:37пожаловаться #14

пруф

Во первых, там же в видео выше. Во вторых, разраб Bluebird про это писал.

Pavel Shakhov (pongo) in Node.js — русскоговорящее сообщество

15:37пожаловаться #15

ты тот код видел? там данные получаются чанками, как ты его промисфаешь?

Pavel Shakhov (pongo) in Node.js — русскоговорящее сообщество

15:38пожаловаться #16

exports.json = (url) => {
  return new Promise((resolve, reject) => {
    https.get(url, (res) => {
      let data = '';
      res.on('data', (chunk) => { data += chunk; });
      res.on('end', () => {
        resolve(JSON.parse(data,"utf8"));
      });
    }).on('error', (e) => {
      reject(e.message);
    })
  });
}

вот теперь ловится

а где ловится ошибка парсинга json?

The Fallen Phoenix in Node.js — русскоговорящее сообщество

15:38пожаловаться #17

ты тот код видел? там данные получаются чанками, как ты его промисфаешь?

Так промис тут создаётся также один

15:39пожаловаться #18