Как правильно написать коллбек с обработкой ошибки?

Question

[rd100]

Как мне написать код с обработкой ошибки?
Мне казалось, что именно так большинство функций написано в node, но ! new Error выдает false
Почему так и как правильно это делается в модулях?

function func(err, data, cb) {
	if(!err) {
		cb(data);
	} else {
		console.log(err);
	}
}

setTimeout(func.bind(null, new Error('error'), 'hello world', (data) => {
	console.log(data);
}), 3000);

Comments

[Роман]

Все у вас работает как надо

function func(err, data, cb) {
  
  if(!err){
  	console.log('-----')
  } else {
  	console.log('+++++')
  }
}

setTimeout(func.bind(null, new Error('error'), 'hello world', (data) => {
  console.log(data);
}), 3000);

в консоль падает +++++

[rd100]

Роман, так я хочу в элс выводить ошибку, если она у меня выпадет.
А щас у меня всегда срабатывает, что ошибка есть, даже, когда в коде ее нет, а только лишь, как новый экземпляр в аргументе.
Т.е. срабатывает всегда обработка ошибки, даже, когда у нас ее не было бы без new Error

[Роман]

rd100, ну сами подумайте, у вас new Error('error')ВСЕГДА передается в функцию в качестве параметра err, а в самой функции стоит условие if(!err) которое ВСЕГДА не срабатывает, потому что err ВСЕГДА определена

[Роман]

rd100, просто опишите словами что вы пытаетесь сделать и как это по вашему должно работать и тогда я смогу вам помочь.

Например, вы пишите

так я хочу в элс выводить ошибку, если она у меня выпадет.

Но совершенно не понятно где (в какой части кода) вы допускаете появление ошибки

setTimeout(func.bind(null, new Error('error') ...

А тут непонятно зачем вы генерируете объект с ошибкой

if(!err)...

А тут непонятно, зачем у вас условие, которое никогда не выполняется

И так почти по каждой строчке вашего кода одни вопросы и ни намека на то, что вы этим хотели добиться

[rd100]

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

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

Это задание было на собеседовании и его решение было такое, какое я выше написал, но оно не работает и я не понимаю, что там имели в виду или может я что не так понял и не так переписал.

И почитать про это тоже негде, в разделе про Error там и близко подобного примера нет.

Есть предположение, что err должен быть в аргументе cb.

[Роман]

rd100, стало чуть яснее. Уточните пожалуйста два момента:
1. В каком месте ожидается ошибка (где она может произойти)?
2. Ожидаемая ошибка происходит сама по себе или ми ее генерируем на основе анализа каких-то данных или она приходит при вызове какого-то кода (например при в обработчике fetch при невозможности установить соединение?

[rd100]

Роман, шла речь о функции, которая сработает не сразу, а через время, что не позволит трай кетч отработать ошибку и если наша функция выдаст ошибку, то всё закрашится, но мы должны ее вызвать или написать так, что если она выдаст ошибку (которая убьет процесс), мы обработаем эту ошибку на подобии того, как я попытался это сделать, через new Error на подобии того, как работают модули в ноде, где первым аргументом идет ошибка.

После этих слов я сам не понимал, где от меня ждут эту обработку ошибки и завис.
На что мне накидали примерный вид, который я повторил, но он не совсем правильно работает, как вы сами убедились.

[Роман]

rd100, давай глянем модули ноды. Вот ее репозиторий на гитхабе https://github.com/nodejs/node/blob/master

открываю любой js файл, например этот https://github.com/nodejs/node/blob/master/lib/_ht... , и вижу

311-327 строчки

const oncreate = once((err, socket) => {
        if (err) {
          process.nextTick(() => this.emit('error', err));
        } else {
          this.onSocket(socket);
        }
      });

      try {
        const newSocket = optsWithoutSignal.createConnection(optsWithoutSignal,
                                                             oncreate);
        if (newSocket) {
          oncreate(null, newSocket);
        }
      } catch (err) {
        oncreate(err);
      }

что есть обычный try/catch

или вот в этом файле https://github.com/nodejs/node/blob/master/lib/int... строчки 46-54

getConnections(callback) {
    this._request({
      cmd: 'NODE_SOCKET_GET_COUNT',
      key: this.key
    }, 'NODE_SOCKET_COUNT', false, (err, msg) => {
      if (err) return callback(err);
      callback(null, msg.count);
    });
  }

тут вообще классика, eсли метод this._request(...) отработал с ошибкой - то ошибка просто пробрасывается в коллбэк

в других модулях встречаются подобные варианты только с promise.then().catch() но суть от этого не меняется. По факту везде либо ошибка перехватывается с помощью try/catch или promise.then().catch() либо генерится с помощью new error, либо прокидывается/возвращается при вызове различных методов/функций сторонних модулей, но во всех случаях отдается (либо не отдается если не было ошибки) в коллбэк. Если исходить из данной логики, то все у вас верно, разве что саму функцию чуть подправить.

Примерно так

function func(err, data, cb) {
  if(!err) {
    cb(null, data);
  } else {
    cb(err, data);
  }
}


// вызов с ошибкой
setTimeout(func.bind(null, new Error('error'), 'hello world', (err, data) => {
  if(err){
       console.log('ошибка', err);
  }else{
       console.log('данные', data);
  }
}), 2000);

// вызов без ошибки
setTimeout(func.bind(null, null, 'hello world', (err, data) => {
  if(err){
       console.log('ошибка', err);
  }else{
       console.log('данные', data);
  }
}), 4000);

[rd100]

Роман, так, а где происходит инициализация ошибки?
Вот например

function func(err, data, cb) {
  if(!err) {
    cb(null, data);
  } else {
    cb(err, data);
  }
}

Как при запуске func err поймет, что она ошибка?
Если мы явно укажем, как указывали ранее, err -> new Error, то всегда будет срабатывать вариант с ошибкой, тюею else

Можно пожалуйста в fiddle с двумя примерами, где сработает ошибка, а где нет, а то код вроде не изменился.
Просто здесь вы явно указываете в последних примерах, new Error.
А как это сделать так, чтобы мы не указывали, а это за нас сделала наша проверка в if уже.

Я не совсем просто понимаю, как правильно подкрутить new Error в аргумент в виде не new Error, чтобы он вечно не отрабатывал на ошибку, что она якобы есть.
Как вообще Error попадает в err?
А точнее, где и как вызывается функция, что она выбирает параметры аргументов.
Ведь вызвав через таймаут - мы уже вызывали функцию и не передали туда изначально Error, а если передали, то всегда же будет отрабатывать if (err) т.е. замкнутый круг.

[Роман]

так, а где происходит инициализация ошибки?

rd100, так этот вопрос я вам и задаю все то время, что мы тут общаемся, и пока вы не дадите на него ответ, мы так и будем гадать на кофейной гуще

Но все-же я думаю что тут нет никакого волшебства, и в примере просто либо передается перехваченная выше ошибка, либо не передается

// вызов с ошибкой
setTimeout(func.bind(null, new Error('error'), 'hello world', (err, data) => {
  if(err){
       console.log('ошибка', err);
  }else{
       console.log('данные', data);
  }
}), 2000);

// вызов без ошибки
setTimeout(func.bind(null, null, 'hello world', (err, data) => {
  if(err){
       console.log('ошибка', err);
  }else{
       console.log('данные', data);
  }
}), 4000);

в первом случае идет вызов с ошибкой, если она была (поймали ее с помощью try/catch или получили от одной из вызываемых функций или сгенерировали по результатам обработки каких либо данных с помощью new Error. Во втором случае вызываем ее без ошибки. Веди менно так и поступают в примерах с гитхаба, что я скинул выше.

[Роман]

rd100, есть еще вот такая штука, https://nodejs.org/dist./v0.8.18/docs/api/domain.h.... Обратите как используется domain.intercept(), но по факту там под капотом все тоже самое. Я допускаю что в ноде есть какие то механики перехвата ошибок путем подмены контекста или чего то в таком же духе, но мне такие способы неизвестны, а об известных я уже рассказал все что знал.