霍夫曼编码C ++代码引发致命错误

Question

我正在为著名的霍夫曼编码算法编写代码。我收到一个致命错误，该错误将系统变成蓝屏，然后重新启动。在具有递归调用的display_Codes中会发生此错误。错误出现在以下几行：

display_Codes(root->l, s + "0"); 
display_Codes(root->r, s + "1" );

以下为完整代码。

#include <iostream>

#include <bits/stdc++.h> 

using namespace std;




class HeapNode_Min {




public:


char d; 

unsigned f; 

HeapNode_Min *l, *r;
HeapNode_Min(char d, unsigned f) 
{ 

      this->d = d;
      this->f = f;
}




~HeapNode_Min()
 {
   delete l;
   delete r;
 } 
}; 



class Analyze {  

  public:
    bool operator()(HeapNode_Min* l, HeapNode_Min* r) 
    { 
        return (l->f > r->f);
    } 
}; 



void display_Codes(HeapNode_Min* root, string s) 
{   
    if(!root) 
        return; 

    if (root->d != '$') 
        cout << root->d << " : " << s << "\n";

    display_Codes(root->l, s + "0"); 
    display_Codes(root->r, s + "1" ); 

} 

void HCodes(char data[], int freq[], int s) 
{ 
    HeapNode_Min  *t,*r, *l ;  


    priority_queue<HeapNode_Min*, vector<HeapNode_Min*>, Analyze> H_min; 

    int a=0;
    while (a<s){H_min.push(new HeapNode_Min(data[a], freq[a])); ++a;}


    while (H_min.size() != 1) { 

        l = H_min.top(); H_min.pop(); 
        r = H_min.top(); H_min.pop(); 

        t = new HeapNode_Min('$',  r->f + l->f); 

        t->r = r; t->l = l; 


        H_min.push(t); 
    } 
    display_Codes(H_min.top(), ""); 
} 



int main() 
{ 
    try
    {

        int frequency[] = { 3, 6, 11, 14, 18, 25 };  char alphabet[] = { 'A', 'L', 'O', 'R', 'T', 'Y' };  
        int size_of = sizeof(alphabet) / sizeof(alphabet[0]); 

      cout<<"Alphabet"<<":"<<"Huffman Code\n";
      cout<<"--------------------------------\n";

      HCodes(alphabet, frequency, size_of);
    }
    catch(...)
    {
    }
    return 0; 
}

Answer 1

您从未将l或r设置为nullptr，但是您的代码依赖于有效的指针或nullptr：

void display_Codes(HeapNode_Min* root, string s) 
{   
    if(!root) 
        return; 

    if (root->d != '$') 
        cout << root->d << " : " << s << "\n";

    display_Codes(root->l, s + "0"); 
    display_Codes(root->r, s + "1" ); 

}

传递没有左节点和右节点的root，那么root->l和root->r都没有可用于任何东西的值。将它们传递给下一个递归可让您取消引用它们，从而调用未定义的行为。

要解决的问题，例如在构造函数中，您需要初始化指针：

HeapNode_Min(char d, unsigned f) : d(d),f(f),l(nullptr),r(nullptr) { }

而且您的课程也不遵循rule of 3/5/0。