承诺的递归函数不会触发返回语句

Question

如果想要的 slugged 字符串可用，我正在尝试编写一个“slug 验证器”来查询我的数据库。

为此，我使用非递归函数递增一个数字并测试它与我的 slugged 字符串连接。

例子：

• 我想在我的数据库中插入my-slugged-string。我想检查是否 my-slugged-string 可用（尚未使用）。
• 我查询我的 带有my-slugged-string 的数据库，如果它已经存在，请尝试 再次使用my-slugged-string1，如果它已经存在，再试一次 与my-slugged-string2等...
• 如果my-slugged-stringx 不存在，则返回此值。

首先，这是我的代码

import nodeGremlin from 'gremlin';
import gremlin from 'gremlin-template-string';

import slug from 'slug';

import Promise from 'bluebird';


let client = nodeGremlin.createClient(8182, 'localhost');
client = Promise.promisifyAll(client);


module.exports.generateNew = function (label, stringToSlug) {
    let sluggedString = slug(stringToSlug, {lower: true});
    return findAvailable(label, sluggedString);
}

function findAvailable(label, stringToTest, i = ''){
    const mixedString = stringToTest + i;
    let query = `g.V().has(label, '${label}').has('slug', '${mixedString}')`;
    return client.executeAsync(query)
        .then((item) => {
            if (item[0]){
                if (i === '') i = 0;
                return findAvailable(label, stringToTest, ++i);
            }
            // breakpoint
            console.log(mixedString);
            return mixedString;

        })
        .catch((err) => {
            // manage error
        });
}

我这样称呼这段代码

import * as Slug from './slug';
import Promise from 'bluebird';

let slug = Promise.promisifyAll(Slug);

export function create(item = {}, callback) {
    // [...]
    slug.generateNewAsync('item', 'my string to slug and test')
        .then((sluggedstring)=>{
            //bla bla bla
        })
    // [...]
}

我的问题是，即使到达断点（算法有效，我已经对其进行了测试：我可以console.log 正确的值），不会返回任何内容，并且进程不会停止并继续运行eeeveeer ...直到超时！ :) :) :)

我可能在 javascript 承诺原则上遗漏了一些东西。

我会感谢你们的帮助；）

编辑：感谢@Adam，代码已编辑

Answer 1

是的，不会返回任何内容，因为findAvailable 中没有return 语句！在此处添加：

function findAvailable(label, stringToTest, i = ''){
    const mixedString = stringToTest + i;
    let query = `g.V().has(label, '${label}').has('slug', '${mixedString}')`;
    return client.executeAsync(query)
//  ^^^^^^
        .then(…);
}

也就是说，您编写的模块现在已经返回了一个承诺。没有必要对它进行承诺 - 事实上，它不会与承诺一起使用，因为它不需要回调。不要在上面调用promisifyAll，方法调用不要使用Async后缀：

…
export function create(item = {}, callback) {
    …
    return slug.generateNew('item', 'my string to slug and test')
//              ^^^^^^^^^^^
        .then((sluggedstring)=>{
            … 
        });
}