C++，(过剩) - 在显示循环中维护 bool 值的全局 vector - 导致程序崩溃

Question

我认为我遇到了一个棘手的问题。我是 C++ 和 OpenGL/glut 的新手，所以要温柔!我正在编写一个独立于平台的简单程序(仅 2D)并且需要一个基本的 UI，因此我有一个生成 Button 对象的 Button 类。希望这能够从 OpenGL/glut 中的 displayFunc() 循环内部动态调用(我认为这是正确的词)。

每次循环开始时，它都会实例化正确的按钮对象，然后在最后销毁它们。当然，按钮需要能够执行某些操作，因此在 glutMouseFunc 函数中，它会检查鼠标是否单击了需要触发按钮的位置。

允许通过公共(public)的 get/set 函数访问按钮成员变量 (xMin.. yMax)。然而，由于按钮是由显示循环实例化的，因此我必须创建一个指向按钮的全局指针 vector ，该 vector 在创建按钮时进行更新，并在显示循环结束时与对象一起进行清理。我相信这一切都很好。

当我真正想要一个按钮来改变状态时，棘手的事情就出现了。我唯一能想到的就是 std::vector<bool> buttonStates; ，在创建按钮时增强，在单击按钮时更改，在单击按钮时销毁。

当然，这种状态变化不会延续到下一个显示循环，所以我尝试了 std::vector<bool> btnStatesCopy; ，它通过使用btnStatesCopy = buttonStates;维护buttonStates vector 的拷贝当它发生变化时，但在显示循环结束时不会被销毁。创建按钮后，按钮状态现在从 btnStatesCopy vector 复制，该 vector 自上次循环以来没有更改。

不幸的是，这会使我的程序崩溃，即使在尝试打印 btnStatesCopy vector 时也是如此。所以我想知道你是否知道它为什么会崩溃，以及我可以做什么来避免它!我可以发布代码，但我不确定它是否有帮助。有没有一种我没有想到的更好、更明显的方法来做这些事情？我很想听听任何想法!多谢，帕特里克

编辑:如果有帮助的话，这是代码的简洁版本(请记住，我对这件事真的很陌生!):

//*in global scope*
// Stores pointers to the ui buttons.
std::vector<Button*> buttons;
// Stores button states, which are copied from buttonStates and
// destroyed every time display() runs.
std::vector<bool> buttonStates;
// Stores a copy of buttonstates which is written to when
// a button is pressed.
std::vector<bool> btnStatesCopy;

// Prototype functions:
void init(void);
void display(void);
void key(unsigned char key, int x, int y);
void processHits(GLint hits, GLuint buffer[]);
void click(int button, int state, int x, int y);
void idle(void);
void resize(int width, int height);
void drawMenu(void);
void circle(float x,float y,float size,int segments);
void initialiseButtons();

// Main loop goes here; calls glutDisplayFunc(display);

void display(void) {
    initialiseButtons();
    drawMenu();
}

bool changeBinaryState(bool state){
    /* Changes 1 to 0 and 0 to 1. */
    if (state == 0) state = 1;
    else state = 0;
    return state;
}

void click(int button, int state, int x, int y) {
/* Called when a mouse button is clicked. */
    if (button == GLUT_LEFT_BUTTON && state != GLUT_DOWN) {
        std::cout<<x<<", "<<y<<std::endl;
        for(int i = 0; i<buttons.size();i++){
            //put a pointer of a button in variable but
            Button* but = buttons.at(i);
            if (x > but->getMinX() && x < but->getMaxX()) {
                if (y > but->getMinY() && y < but->getMaxY()){
                    but->buttonFunc();
                    changeButtonState(buttonStates.at(i));
                    buttonStates.at(i) = 1;
                    btnStatesCopy = buttonStates;
                }
            }
        }
    }
}

void drawMenu(void) {
    // Create a button and its pointer and add pointer to buttons vector.
    Button* p_b = new InfoButton("User Manual",20,spacing+yOffset);
    buttons.push_back(p_b);
    // Add button state of previous button state.
    buttonStates.push_back(btnStatesCopy.back());

    // Turn on buttons if their state is on.
    for (unsigned i = 0; i<buttonStates.size();i++){
        if (buttonStates[i]==1) {
        Button* but = buttons.at(i);
        but->buttonFunc();
    }
}
void initialiseButtons(){
    for (unsigned i = 0; i < buttons.size();i++)
    {
        delete buttons[i];
    }
    buttons.clear();
    buttonStates.clear();
}

Answer 1

您问题的标题为

Blockquote C++, (GLUT) - maintaining a global vector of pointers to booleans in the display loop - crashes program

(强调已添加)。如果我理解正确的话，那就意味着你有

std::vector<bool*> buttonStates;
std::vector<bool*> btnStatesCopy;

而不是 std::vector<bool> 。

如果确实如此，并且如果这些指针指向的 bool 值在显示循环中被删除，那么它们也会在显示循环中被删除。您正在保留指向已释放内存的指针，这是崩溃最可能的原因。

另一方面，如果这是 Java(或任何其他垃圾收集语言)而不是 C++，则存储指向对象的指针可以防止它被垃圾收集。