我的霍夫曼编码方法哪里出错了？

Question

我正在尝试编写 Huffman 字符串编码算法。

我的解决方案是这样工作的：

1. 由于字符串中的每个字母都有一个与之关联的特殊二进制代码，因此搜索二叉树，当找到一个字母时，将其添加到带有二进制代码的映射中。 （我出错的地方在这里）
2. 对字符串进行迭代，并为每个字母关联与映射字母的键关联的值。

我没有在某处打印树，即使很难，它也可以帮助你，但这是我得到的字符串 abracadabra，以及我应该得到的：

正确代码：000010000110110101111101011000111110110100111011101100101101110000110000110111100101111101010010

我得到了什么： 00001000111011010110101111010101100011101011010

这是我的代码：

#include <algorithm>
#include <map>

string codes = "";

void getMapCharBinaryCode(Node root, string &prefix, map <char, string> &m){
    if(!root) return;
    if(root->value){
        if(!m.count(root->value)){
            m[root->value] = prefix;
            prefix = "";
        } 
    }
    if(root->leftChild){
        getMapCharBinaryCode(root->leftChild, prefix += "0",m);
    }
    if(root->rightChild){
        getMapCharBinaryCode(root->rightChild, prefix += "1",m);
    }
   
}

string encode(string text, Node tree){
    // text is "abracadabra"
    // create map for each char -> binary code
    map<char, string> m;
    string prefix = "";
    getMapCharBinaryCode(tree, prefix, m);
    
    // iterate on text and assign each letter with binary code from map
    for(int i = 0; i < text.size(); i++) {
        codes += m[text[i]];
    }
    return codes;
}

Answer 1

当您使用prefix = "" 保存叶子时，您正在破坏prefix 中的代码，当您从树中退回并转到下一个分支时需要该代码。

您可以为prefix 维护一个单独的存储区域，通过引用传递它。但是，您需要在上下树时管理prefix 的长度，并且您不需要添加0，然后将1 添加到that 以供两者使用分支，为右分支添加 01 而不是 1。

作为起点，您应该只通过值传递prefix，这会产生副本，但在管理中不需要小心。去掉 & 并将 prefix += 替换为 prefix +。摆脱prefix = ""，然后它什么都不做。