循环内的承诺答案

【问题标题】：Promise inside a loop循环内的承诺
【发布时间】：2015-08-12 01:59:13
【问题描述】：

在下面的代码中，我有一个无限循环，我不知道它为什么会发生。我最好的猜测是因为里面的函数是async 循环不会等待它，所以循环永远不会停止。解决此问题的最佳方法是什么？

 var generateToken = function(userId) {
    return new Promise(function(resolve, reject) {
        User.findOne({userId: userId}, function(err, user) {
            if (user !== null) {
                var loop = true;
                while (loop) {
                    var token = Common.randomGenerator(20);
                    (function(e) {
                        User.find({tokens: e}, function(err, result) {
                            if (err) {
                                loop = false;
                                reject('Error querying the database');
                            } else {
                                if (result.length === 0) {
                                    if (user.tokens === undefined) {
                                        user.tokens = [];
                                    }
                                    user.tokens.push(e);
                                    loop = false;
                                    resolve();
                                }
                            }
                        });
                    })(token);
                }
            } else {
                return reject('UserNotFound');
            }
        });
    });
};

这个函数接收一个 userId（User.findOne() 用于查找用户，如果没有具有该 id 的用户，则拒绝该承诺）并为该用户创建一个唯一随机令牌（@987654326 @) ，将其添加到保存在 MongoDB 中的用户实体中，并将其返回给调用者。

(注意有一些反对票说这个问题与this one 相同，这不是因为我的代码中已经有一个闭包并且它仍然不起作用。那个问题更多关于如何将循环变量绑定到闭包）

【问题讨论】：

如果内部查找结果长度不为 0，什么会将循环设置为 false？
有点感觉如果我们确切地知道这样做的目标是什么，所有这些都可以浓缩为一个查询。 mongo 有一个非常强大的属性和子文档过滤系统。
For-loop and async callback in node.js?的可能重复
@KevinB 实际上这段代码是为了学习（不是一个真正的项目，我现在关心它的性能。即使我可以用其他方式做到这一点，问题仍然是一样的）它包括 2查询不是我不确定是否可以在 Mongo 的一个查询中完成的查询，但看看它是如何工作的可能会很有趣。
@doldt ：不，我在帖子的最后一行添加了评论。

标签： javascript node.js promise

【解决方案1】：

你是正确的，你不能像你试图做的那样循环。

Javascript 是单线程的。只要主线程在您的while(loop) 语句中循环，其他任何事情都没有机会运行。如果您的主线程本身正在更改loop 变量，这一切都会好起来的，但这并不是正在发生的事情。相反，您试图在异步响应中更改 loop 变量，但是由于您的主线程正在循环，因此永远无法处理异步响应，因此您的 loop 变量永远不会被更改，因此是非生产性的无限循环。

要修复，您必须更改为不同的循环结构。一种常见的设计模式是创建一个本地函数，其中包含您想要重复的代码。然后，运行您的异步操作，如果在异步结果处理程序中，您决定要重复该操作，您只需从其中再次调用本地函数。因为结果是异步的，所以堆栈已展开，从技术上讲，这不是堆栈建立的递归。它只是启动函数的另一个迭代。

我对您的代码中的逻辑有点困惑（那里有零个 cmets 来解释它），所以我不完全确定我的理解是否正确，但总体思路如下：

var generateToken = function(userId) {
    return new Promise(function(resolve, reject) {
        User.findOne({userId: userId}, function(err, user) {
            function find() {
                var token = Common.randomGenerator(20);
                User.find({tokens: e}, function(err, result) {
                    if (err) {
                        reject('Error querying the database');
                    } else {
                        if (result.length === 0) {
                            if (user.tokens === undefined) {
                                user.tokens = [];
                            }
                            user.tokens.push(e);
                            resolve();
                        } else {
                            // do another find until we don't find a token
                            find();
                        }
                    }
                });
            }

            if (user !== null) {
                find();
            } else {
                reject('UserNotFound');
            }
        });
    });
};

我应该注意到您对User.findOne() 操作的错误处理不完整。

仅供参考，不断查询直到获得result.length === 0 的整个逻辑似乎很奇怪。逻辑看起来很奇怪，它闻起来像“在紧密循环中轮询数据库”，这通常是一件性能很差的事情。如果我们在更高的层次上理解整体问题，我怀疑会有更有效的方法来解决这个问题。

【讨论】：

Because the result is async, the stack has unwound and this isn't technically recursion with a stack build up. It's just launching another iteration of the function.你能详细说明这个想法吗？
@ArianHosseinzadeh - 包含函数完成执行，堆栈完全展开。然后异步操作完成并使用干净的堆栈调用回调。
@ArianHosseinzadeh - 这个答案是给你的吗？
一有时间我会尝试并告诉你
@ArianHosseinzadeh - 没有。 find() 没有返回值，所以返回它的返回值没有用。它调用resolve() 或reject() 或启动另一个find()。这就是它结束的三种方式。

【解决方案2】：

就学习如何解决此类问题而言，您可能需要查看 async 库 (https://github.com/caolan/async)。它提供了非常直观的方式来处理这样的异步情况，大多数熟悉同步迭代和 javascript 基础知识的人都可以理解，几乎可以想象任何风格的异步迭代，并且被广泛使用并且非常好记录在案。

【讨论】：