0

我遇到了麻烦，每当我尝试编译这个程序时，我都会在包含 q.pop 的包含函数中得到一个错误。如果该行被注释掉，它将编译得很好，但它仍然给出错误的值。由于某种原因，它总是错误的。

错误是：
priority_queue_demo.cpp: In function 'int main()': priority_queue_demo.cpp:54:12: error: invalid conversion from 'int' to 'const char*' [-fpermissive] /usr/lib/gcc/x86_64-redhat-linux/4.6.1/../../../../include/c++/4.6.1/bits/basic_string.tcc:214:5: error: initializing argument 1 of 'std::basic_string<_CharT, _Traits, _Alloc>::basic_string(const _CharT*, const _Alloc&) [with _CharT = char, _Traits = std::char_traits<char>, _Alloc = std::allocator<char>]' [-fpermissive] priority_queue_demo.cpp: In function 'bool contains(const std::priority_queue<T>&, T) [with T = int, typename std::vector<T, std::allocator<_Tp1> >::value_type = int]': priority_queue_demo.cpp:43:47: instantiated from here priority_queue_demo.cpp:27:3: error: passing 'const std::priority_queue<int>' as 'this' argument of 'void std::priority_queue<_Tp, _Sequence, _Compare>::pop() [with _Tp = int, _Sequence = std::vector<int, std::allocator<int> >, _Compare = std::less<int>]' discards qualifiers [-fpermissive] 

#include <iostream>
#include <string>
#include <sstream>

#include <queue>

using namespace std;

template <typename T>
ostream &operator <<(ostream &os, priority_queue<T> &q) {
        os << "{";
        while (!q.empty()) {
               os << q.top() << (!q.empty() ? ", " : "");
               q.pop();
        }

        os << "}";

        return os;
}

template <typename T>
bool contains(const priority_queue<T> &q, T val) {
        for (int i = 0; i < q.size(); i++) {
                if (q.top() == val) return true;
                q.pop();
        }

        return false;
}

int main() {
        cout << boolalpha;

        priority_queue<int> qi;

        for (int i = 0; i < 20; i++)
                qi.push(i);

        cout << qi << endl;

        cout << "qi contians 15: " << contains(qi, 15) << endl;
        cout << "qi contians 23: " << contains(qi, 23) << endl;

        const int ARR_SIZE = 4;
        string arr[ARR_SIZE] = {"cat", "dog", "cow", "elephant"};

        cout << endl;

        priority_queue<string> qs;

        for(int i = 0; i < ARR_SIZE; i++)
                qs.push(i);

        cout << qs << endl;

        while(!qs.empty()) {
                qs.pop();
                cout << qs << endl;
        }


        return 0;
 }
4

3 回答 3

1

问题是您正在传递const对队列的引用，但pop()显然不是一个const操作，因为它会改变队列。

您应该传递一个非常量引用：

bool contains(priority_queue<T>& q, T val) { .... }

如您所见，如果不更改队列，就无法真正检查队列的内容。

下一个错误是您将整数推入队列std::string：

qi.push(i);

你大概是说

qi.push(arr[i]);

请注意，您的队列重载会清空队列，因此在打印队列后ostream& operator<<使用的任何尝试都将产生错误，无论队列的原始内容如何。containsstd::cout

在此处查看演示。

于 2013-03-27T22:21:12.597 回答
1

您在代码中有几个问题：

priority_queue<string> qs;

    for(int i = 0; i < ARR_SIZE; i++)
            qs.push(i);  //<<<<Error, should be arr[i]

arr[i]你真的不应该推，i因为i它期望的是整数string.

您将priority_queueby 引用传递给重载的<<运算符函数，该函数将删除 中的所有元素priority_queue，因此您的所有contains查询都将变为false.

您也可以const priority_queue<T>&在contains函数中使用，但是，pop()函数不是const，这会导致编译错误，正如 juanchopanza 所指出的那样。

于 2013-03-27T22:29:03.293 回答
0

我之前发表了评论，但您的代码还有其他问题不适合评论。

1) qs是 apriority_queue<string>但您将ints 推向它（在行中qs.push(i)）；我猜你想做的是 push arr[i]。因此，更换

qs.push(i);

和

qs.push(arr[i]);

2)模板函数contains采用priority_queue<T>by const 引用。这意味着您不能const在q. 特别是，您不能调用q.pop()，因为pop()它是一个非常量方法。（确实，这种方法有望改变队列。）因此，替换

template <typename T>
bool contains(const priority_queue<T> &q, T val) {

和

template <typename T>
bool contains(priority_queue<T> &q, T val) {
于 2013-03-27T22:35:42.723 回答