使双端队列迭代器超出范围

时间:2018-09-18 03:57:18

标签: c++ visual-studio

我正在尝试通过一些类似数组的对象进行改组。我想在前面添加一个值,然后弹出最后一个值。我尝试根据其他帖子上看到的建议使用双端队列,但仍然出现错误。有任何想法吗?我是C ++的新手,虽然我可以尝试解决此问题,但我想知道它在哪里,以便我可以解决它。

我得到一个:无法从Visual Studio中找到超​​出范围调试错误的双端队列迭代器。

引起问题的函数:

(我本来是用向量的)

void vector_pusher(deque <int> &v, int new_val ) { 
v.push_front(new_val);
v.erase(v.begin() + 3);
}

这是我其余的代码(也许是构造问题?):

#include "lock.h"
#include <deque>
#include <iostream>
using namespace std;

lock::lock(int x, int y, int z) {
    is_closed = true;
    current_top = 0;
    comb[0] = x % MAX_VAL;
    comb[1] = y % MAX_VAL;
    comb[2] = z % MAX_VAL;
    deque <int> trycomb = { 1, 1, 1 };
    deque <char> trydir = { 'q', 'q', 'q' };
    deque <int> rot = { 1,1,1 };
}

void lock::turn(int input, char direction, int rotation) {

    vector_pusher(trydir, direction);
    vector_pusher(trycomb, input);
    vector_pusher(rot, rotation);

}

在lock.h中:

 public:
        lock(int, int, int);
        void turn(int, char, int);
        \* these functions are defined elsewhere and doing fine
        void new_comb(int, int, int);
        void open_close();
        bool lock_status() const;
        *\

        //entered combination
        deque <int> trycomb;

        // entered diections
        deque <char> trydir;

        //entered rotations (0 is directly too, 1 is around once)
        deque <int> rot;



    private:
        // current correct combo
        deque <int> comb = {0, 0, 0};

        //top val
        //this is unessesary
        int current_top;

        //open-ness state of lock
        bool is_closed;

        //needed directions
        deque <char> dir = {'r', 'l', 'r'};

感谢您的帮助!

2 个答案:

答案 0 :(得分:0)

似乎您从未在构造函数中初始化trycombtrydirrot。您在构造函数中声明了具有相同名称的局部变量,这些局部变量遮盖了lock类的成员变量。您的构造函数应如下所示:

lock::lock(int x, int y, int z) {
    is_closed = true;
    current_top = 0;
    comb[0] = x % MAX_VAL;
    comb[1] = y % MAX_VAL;
    comb[2] = z % MAX_VAL;
    trycomb = { 1, 1, 1 };
    trydir = { 'q', 'q', 'q' };
    rot = { 1,1,1 };
}

答案 1 :(得分:0)

如前所述,您必须将构造函数更改为

lock::lock(int x, int y, int z) {
    is_closed = true;
    current_top = 0;
    comb[0] = x % MAX_VAL;
    comb[1] = y % MAX_VAL;
    comb[2] = z % MAX_VAL;
    trycomb = { 1, 1, 1 };
    trydir = { 'q', 'q', 'q' };
    rot = { 1,1,1 };
}

否则,将不会初始化类成员。

第二,如果要删除队列中的第一个元素,请使用pop_front,而要删除队列中的最后一个元素,请使用pop_back(请参见双端队列的documentation)。这使代码更具可读性。