0

我大约 1-2 天开始学习 C++，我是从 Java 来到这里的。但是我在删除对象时遇到了一个问题。当我在删除旧对象后尝试创建新对象时出现问题。我有“播放器”类，它以 RGB 格式存储其位置、角度、运动和颜色。这是代码：

class Player {
    public:
        float posX, posY, motionX, motionY, angle, r, g, b;
        Player (float, float);
        ~Player() {}; 

        void draw() {
            < GLCode goes here.. >
        }
};

Player::Player(float x, float y) {
    posX = x;
    posY = y;
    cout << "created a player object!\n";
}

int main(int argc, char **argv);
void redraw(void);
void update(void);

//Players List
std::vector<Player*> data;

void keyboard(unsigned char key, int x, int y) {
     switch (key) {
        case VK_ESCAPE:
            if (data.size() > 0) {
                Player *p =  data[data.size()-1];
                delete p;
                p = NULL;
                data.pop_back();
                cout << data.size() << endl;
            }
         break;

        case VK_SPACE:
            Player* p = new Player(200, 200);
            data.push_back(p);
         break;
     }
 }

void redraw(void) {
    glClear(GL_COLOR_BUFFER_BIT | GL_DEPTH_BUFFER_BIT);
    glPushMatrix();
    for (UINT i = 0; i < data.size(); i++) {
        Player* pr = data[i];
        if (pr)
            pr->draw();
    }

    glPopMatrix();
    glutSwapBuffers();
}

void update(int v) {
    glutTimerFunc(20, update, 0);
    glutPostRedisplay();
}

void initGL(int argc, char **argv) {
    < GLCode goes here.. >
}

void initGameLogic(void) {
    // Here I am creating two player objects, one is red color and one is green
    Player* p = new Player(300, 200);
    p->r = 0;
    p->g = 1;
    p->b = 0;
    data.push_back(p);
    p = new Player(400, 200);
    p->r = 1;
    p->g = 0;
    p->b = 0;
    data.push_back(p);
}

void initGameLoop(void) {
    < GLCode goes here.. >
}

int main(int argc, char **argv) {
    initGL(argc, argv);
    initGameLogic();
    initGameLoop();
}

所以问题是，当我按 ESC （并删除最后一个玩家）并在此之后按 SPACE 时，程序为我创建了新的玩家对象，但它以某种方式（虽然我当然删除了旧的玩家对象）得到了绿色（旧玩家的颜色）颜色，但标准颜色是黑色。如果我再次按下它，它会创建新的播放器，现在是黑色。因此，如果我删除所有播放器，然后一个接一个地创建它们——前两个是绿色和红色，这意味着程序不会清理对象变量。我可以在类中存储浮点指针，在初始化时为它们分配内存，然后在析构函数中删除它，但是我的析构函数方法应该是巨大的，因为我需要删除存储在我的对象中的所有变量。那么我的代码有什么问题？

当我应该重置构造函数中的所有变量时，它是内存泄漏还是仅仅是变量初始化问题？

是的，对不起我的英语不好。

编辑：感谢 Ivan Ishchenko、来自莫斯科的 Vlad 和 Mohammad Ali Baydoun 解决了。顺便说一句，正如 Mohammad Ali Baydoun 为我所建议的那样，只需用播放器对象（而不是指针）替换数组的类型。

4

2 回答 2

3

您的构造函数不会初始化变量 r、g、b。因此，如果使用运算符 new 创建的对象是值初始化的，则变量 r、g、b 具有任意值。

于 2013-10-18T20:30:12.293 回答
2

您正在为 2 个第一批玩家设置颜色，而不是为新玩家设置颜色。所以新玩家的颜色变量是未初始化的，可以存储他们想要的任何东西。考虑在 Player 构造函数中用 BLACK 初始化它们。

于 2013-10-18T20:29:36.157 回答