递归遍历字典尝试计算总单词数

Question

我已经从一个链表构建了一个 trie。每个节点包含一个 char 和一个由 27 个节点组成的数组（字母 + 一个额外的 $ 表示单词结尾的点）。 我尝试编写一个递归方法来计算单词的数量，但它返回 1。我不确定如何修复它或到底出了什么问题。

int recursiveCount(Node* temp, int count)
{
    if (temp->value == '$')
    {
        count++;
    }
    for (int i = 0; i < 27; i++)
    {
        if (temp->array[i] != NULL)
        {
            return recursiveCount(temp->arr[i],count);
        }
    }
return count;
}

Answer 1

您通过值传递count，这意味着当递归解开时它超出范围，并且只有“最远”的一个被返回，因为这是第一个要增加的count，它只是1。通过通过引用代替int recursiveCount(Node* temp, int& temp);

Answer 2

正如其他答案中已经指出的那样，您的问题是您有不同的 count 变量，每个递归调用一个变量。增加一个不会改变其他的。除了传递（非常量）引用之外，您还可以采用更实用的编程方法并从函数中返回计数。当然，您需要总结您进行的所有递归调用的返回计数：

unsigned recursiveCount(Node const * node) /* you don't change the node, so make
                                              it const. Why a pointer btw? A
                                              reference would do fine! */
{
    unsigned count = 0; /* You aren't expecting a negative number of
                           words, are you? So use unsigned. */
    if (node->value == '$')
    {
        count++;
    }
    for (int i = 0; i < 27; i++)
    {
        if (node->array[i] != NULL) /* "array"!? change that to a meaningful
                                       name ... "children" is bad, but not as
                                       bad as "array" ... */
        {
            count += recursiveCount(node->array[i]); /* "arr" was a typo I
                                                        suppose */
        }
    }
    return count; /* consistent indentation, please! */
}

Answer 3

您已将其标记为 C++，所以我很困惑为什么该方法不是 Node.js 的一部分。见封装。

递归代码中的重大错误不是添加在每个后续节点中找到的每个'$'。特别是，您的代码仅返回一次 for 循环调用的计数，所有其他都被丢弃。

for (int i = 0; i < 27; i++)
{
    if (temp->array[i] != NULL)
    {
        return recursiveCount(temp->arr[i],count);
        // This only returns one value,
        // and ignores all the other 26 possible values
        // these should sum together
    }
}

考虑使方法成为节点的一部分。请注意 retVal 如何累积找到的每个“$”。

int Node::recursiveCount()
{
    int retVal = 0; // accumulation

    if (value == '$') 
    { 
        retVal += 1;  // 1 word found here
    }

    // keep searching here, there might still be words with prefix
    for (size_t i = 0; i < 27; i++)
    {
        if (nullptr != nxt[i]) 
        {
            retVal += (nxt[i]->recursiveCount());
            //     ^^ accumulate any other words found
        }
    }
    return (retVal); // return full count
}