c++ - C++ の最小ヒープのコンパレーター

Question

STL などを使用して C++でs の最小ヒープ¹を作成しようとしていますが、コンパレータが正しく比較されていないようです。以下は私の現在のコンパレータです：longmake_heap

struct greater1{
    bool operator()(const long& a,const long& b) const{
        return a>b;
    }
};

ただし、std::pop_heap(humble.begin(),humble.end(),g);where gis an instance greater1and humbleis a heap who make [9,15,15,25]whensort_heapが呼び出されると、15ポップが発生します。

私のコンパレータは正しいですか？何がうまくいかないのですか？

編集:
コンパレータなしで sort_heap を実行していることに気付きましたが、このコンパレータを実行すると[15,15,9,25]、sort_heap. 今、私はコンパレーターが確実に機能していないと考えていますが、その理由は不明です。

¹ STL はデフォルトで max-heap を作成するため、コンパレータが必要です。

Answer 1

おそらくどこかで何かが足りないので、以下のコードは意図したとおりに機能します。

#include <vector>
#include <algorithm>
#include <iostream>

struct greater1{
  bool operator()(const long& a,const long& b) const{
    return a>b;
  }
};

int main() {
  std::vector<long> humble;
  humble.push_back(15);
  humble.push_back(15);
  humble.push_back(9);
  humble.push_back(25);

  std::make_heap(humble.begin(), humble.end(), greater1());
  while (humble.size()) {
    std::pop_heap(humble.begin(),humble.end(),greater1());
    long min = humble.back();
    humble.pop_back();  
    std::cout << min << std::endl;
  }

  return 0;
}

Answer 2

そのまま使用して greater<int>()ください。std で事前定義されています。

Answer 3

You want to call make_heap on the vector again, not sort_heap. make_heap will rearrange your entire vector into a min heap given the greater-than comparator whereas sort_heap sorts your element into ascending order and is no longer a heap at all!

#include <algorithm>
#include <iostream>
#include <vector>

struct greater1{
    bool operator()(const long& a,const long& b) const{
        return a>b;
    }
};

int main()
{
  unsigned int myints[] = {10,20,30,5,15};
  vector<unsigned int> v(myints, myints+5);

  //creates max heap
  std::make_heap(v.begin(). v.end()); // 30 20 10 5 15

  //converts to min heap
  std::make_heap(v.begin(). v.end(), greater1()); // 5 15 10 20 30

  unsigned int s =  v.size();

  //ALSO NEED TO PASS greater1() to pop()!!!
  for(unsigned int i = 0; i < s; i++)
    std::pop_heap(v.begin(). v.end(), greater1()); // popping order: 5 10 15 20 30

  return 0;
}