JS - dann und wartet auf ein Versprechen

Question

Ein gemeinsames Thema, das ich weiß, aber ich will nur mein Verständnis auf der Welt der JS und Versprechungen bestätigen.

So habe ich das folgende Code-Segment, das darin scheitert, dass der then Block nicht wirklich auf das oben genannte Segment wartet, um beendet zu werden. Ich bekomme nämlich nicht den endgültigen/korrekten Wert von okCheck.

var okCheck = false; 

User.findOne({publicID: id}, function (err, userInfo) { 
     if (userInfo.checked.indexOf(id) > -1){ 
     okCheck = true; 
     } 
}) 
.then(function() { 
     //do some additional stuff using the boolean okCheck 
} 

So dies zu beheben - von dem, was ich verstehe, ich brauche eine return zu verwenden - dies richtig ist?

Ist das korrekt - nämlich bin ich garantiert, dass ich immer den endgültigen Wert von okCheck haben werde?

Danke.

Answer 1

Das ist keine Art und Weise, wie es funktioniert, Sie versprechen, um zurückkehren muss „.then()“ zu verwenden.

var okCheck = false; 

User.findOne({publicID: id}).exec() 
.then(function (userInfo) { 
    if (userInfo.checked.indexOf(id) > -1){ 
     okCheck = true; 
    } 
    return okCheck 
}) 
.then(function (okCheck) { 
    //do some additional stuff using the boolean okCheck 
}) 

Hier gibt ".exec()" Versprechen zurück.

Answer 2

Von dem, was ich verstehe, ich brauche eine return zu verwenden - dies richtig ist?

Ja. Aber return einen Wert zu beeinflussen beeinflusst nichts Timing-bezogenes.

Der Punkt ist, dass der Wert return von einem then Callback der Auflösungswert des Versprechens ist, das es zurückgibt, und das Argument des nächsten Rückrufs in der Kette wird.

Sie sollten nicht äußeren Umfang Variablen von asynchronen Rückrufe ändern. Der richtige Ansatz ist es, Maßnahmen aufzuschieben, bis das Versprechen gelöst hat, nicht zu „warten“ mit allem, was für einen variablen Wert angezeigt werden:

// no `var` here! 
User.findOne({publicID: id}).then(function(userInfo) { 
    return userInfo.checked.indexOf(id) > -1; 
// ^^^^^^ return a boolean 
}).then(function(okCheck) { 
//    ^^^^^^^ receive it here as a parameter 
    … // do some additional stuff using it 
}); 

Answer 3

Zwei Punkte:

• okCheck nicht zuverlässig außerhalb der verwendet werden Kontext von User.findOne's Callback oder einem verketteten .thens Callback, daher hat es keinen Vorteil, es als eine äußere Var zu haben, in der Tat ist es eine schlechte Übung, dies zu tun.
• Sie sollten nicht versuchen, beide einen direkten Rückruf und eine verkettete then() zu verwenden - sie sind Alternativen.

diese Mungo sein Unter der Annahme, und userInfo.checked.indexOf() synchron zu sein, können Sie eine der folgenden schreiben: zu User.findOne()

User.findOne({publicID: id}, function(err, userInfo) { 
    if(err) { 
     console.log(err); 
    } else { 
     if (userInfo.checked.indexOf(id) > -1) { 
      // do additional stuff 
     } else { 
      // do other stuff 
     } 
    } 
}); 

Kette .then() das Versprechen

Pass einen Rückruf zurückgegeben von User.findOne().exec()

User.findOne({publicID: id}}).exec().then(function(userInfo) { 
    if(userInfo.checked.indexOf(id) > -1) { 
     // do additional stuff 
    } else { 
     // do other stuff 
    } 
}, function(err) { 
    console.log(err); 
}); 

Hinweis: Vieles davon gilt immer noch, auch wenn es kein Mongoose ist.