Обещание не работает внутри обратного вызова

Question

Я пишу модуль, который использует API Google, но я обертываю все, что является обратным вызовом, основанным на обещании. Это код проблемной области

file1.js

var File2 = require('file2') 
var api = new File2() 
api.auth().then(auth => { 
    api.search('example').then(res => { 
    ...do some stuff... 
    }) 
}).catch(err => { 
    console.log('1') //Not being run 
    throw err 
}) 

file2.js

class File2(){ 
    auth() { 
    ...works fine and resolves... 
    } 

    search() { 
    return new Promise((resolve, reject) => { 
     googleapi.somemethod(options, (err, res) => { 
     if(err) { 
      console.log('2') // DOES run 
      reject(new Error(err)) 
     } 
     resolve(res.field) //Program crashes here because reject didn't actually reject 
     }) 
    }) 

} 

Вызов auth работает просто отлично, но призыв к search (и более конкретно googleapi.somemethod) не работает, и err определен. Я проверяю ошибку, и console.log('2') запускает, но затем console.log('1') в catch не запускается, ошибка не выбрасывается, и программа разбилась на resolve(res), потому что res не определен. Я пытался поставить ловец ошибки в качестве второго аргумента then вместо использования catch, но до сих пор не работает

api.search('example').then(res => { 
    ...do some stuff... 
}, err => { 
    console.log('2') // Still doesn't run 
    throw err 
}) 

Я бегу Узел v6.2.1

Answer 1

Вы должны вернуть обещание :

var File2 = require('file2') 
var api = new File2() 
api.auth().then(auth => { 
    return api.search('example').then(res => { // return the promise 
    return ... 
    }) 
}).catch(err => { 
    console.log('1') // Not being run 
    throw err 
}) 

Кроме того, если вам не нужно auth внутри search, то вы можете unnest эти обещания:

var File2 = require('file2') 
var api = new File2() 
api.auth().then(auth => { 
    return api.search('example') 
}).then(res => { 
    return ... 
}).catch(err => { 
    console.log('1') //Not being run 
    throw err 
}) 

Answer 2

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

Пожалуйста, обновите от

if(err) { 
    console.log('2') // DOES run 
    reject(new Error(err)) 
} 
resolve(res.field) //Program crashes here because reject didn't actually reject 

в

if(err) { 
    console.log('2') // DOES run 
    reject(new Error(err)) 
} 
else { 
    resolve(res.field) //Program crashes here because reject didn't actually reject 
} 

* обновление * или вы можете сократить свой код

if(err) { 
    console.log('2') // DOES run 
    return reject(err) // no need to new Error object 
} 
resolve(res.field) //Program crashes here because reject didn't actually reject