Очередь приоритетов Неправильный порядок - PullRequest
3 голосов
/ 03 марта 2010

Я программирую кодировку Хаффмана. Это начало моей программы:

using namespace std;

//Counting methods
int *CountCharOccurence(string text)
{
    int *charOccurrence = new int[127];
    for(int i = 0; i < text.length(); i++)
    {
        charOccurrence[text[i]]++;
    }
    return charOccurrence;
}

void DisplayCharOccurence(int *charOccurrence)
{
    for(int i = 0; i < 127; i++)
    {
        if(charOccurrence[i] > 0)
        {
            cout << (char)i << ": " << charOccurrence[i] << endl;
        }
    }
}

//Node struct
struct Node
{
    public:
        char character;
        int occurrence;

        Node(char c, int occ) {
            character = c;
            occurrence = occ;
        }

        bool operator < (const Node* node)
        {
            return (occurrence < node->occurrence);
        }
};

void CreateHuffmanTree(int *charOccurrence)
{
    priority_queue<Node*, vector<Node*> > pq;
    for(int i = 0; i < 127; i++)
    {
        if(charOccurrence[i])
        {
            Node* node = new Node((char)i, charOccurrence[i]);
            pq.push(node);
        }
    }

    //Test
    while(!pq.empty())
    {
        cout << "peek: " << pq.top()->character <<  pq.top()->occurrence << endl;
        pq.pop();
    }
}

int main(int argc, char** argv) {

    int *occurrenceArray;
    occurrenceArray = CountCharOccurence("SUSIE SAYS IT IS EASY");
    DisplayCharOccurence(occurrenceArray);
    CreateHuffmanTree(occurrenceArray);

    return (EXIT_SUCCESS);
}

Программа сначала выводит символы с их номером вхождения. Это выглядит хорошо:

 : 4
A: 2
E: 2
I: 3
S: 6
T: 1
U: 1
Y: 2

но тестовый цикл, который должен отображать содержимое узла в порядке приоритета, выдает следующее:

peek: Y2
peek: U1
peek: S6
peek: T1
peek: I3
peek: E2
peek:  4
peek: A2

Это не ожидаемый заказ. Почему?

Ответы [ 3 ]

5 голосов
/ 03 марта 2010

Элементы в вашей очереди приоритетов являются указателями. Поскольку вы не предоставляете функцию, которая принимает 2 указателя на объекты Node, функция сравнения по умолчанию сравнивает 2 указателя.

bool compareNodes(Node* val1, Node* val2)
{
   return val1->occurence < val2->occurence;
}
priority_queue<Node*, vector<Node*>,compareNodes > pq;

Ваш оператор <используется, когда узел сравнивается с узлом *</p>

1 голос
/ 03 марта 2010

Вы храните указатели на узлы в очереди, но не предоставили подходящую функцию сравнения, поэтому они сортируются путем сравнения указателей. Предоставленный вами operator< будет сравнивать узел с указателем, а это не то, что вам нужно.

Есть два варианта:

  • Предоставить функцию для сравнения двух указателей узлов в соответствии с их значениями и передать эту функцию в очередь, или
  • Сохранение объектов узлов в очереди и предоставление operator< для сравнения двух узлов.

Второй вариант также исправит утечку памяти в вашем коде и удалит целую кучу ненужных выделений памяти, поэтому я бы предложил это.

1 голос
/ 03 марта 2010

Вы должны указать своей приоритетной очереди, по какой она должна быть отсортирована. В вашем случае вы должны указать сортировку по Node::occurence.

...