Promise-Abweisung funktioniert nicht innerhalb des Rückrufs

Question

Ich schreibe ein Modul, das die Google-API verwendet, aber alles umschließt, was Callback-basiert in einem Versprechen ist. Dies ist der Code des Problembereichs

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 
     }) 
    }) 

} 

Der Aufruf von auth funktioniert ganz gut, aber der Anruf an search (und insbesondere googleapi.somemethod) ist fehlgeschlagen, und err ist definiert. Ich überprüfe nach Fehler, und console.log('2') läuft, aber dann console.log('1') in catch nicht ausgeführt wird, wird der Fehler nicht ausgelöst, und das Programm stürzte auf resolve(res), weil res nicht definiert ist. Ich habe versucht, zu then den Fehler Fang als zweites Argument setzt stattdessen catch zu verwenden, aber das noch nicht

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

Ich renne Knoten V6.2.1

Answer 1

Sie sollten das Versprechen zurückgeben funktioniert :

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 
}) 

auch wenn Sie nicht auth innerhalb search benötigen, dann können Sie diese Versprechen 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

Aufruf Reject() stoppt nicht Ihr Programm, alle Codes unten werden auch ausgeführt.

Bitte aktualisieren Sie von

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

zu

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

* update * oder Sie können Ihren Code

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