0

根据我最近在一次工作面试中的建议，我被建议研究 C++11 的 unique_ptr 功能，作为自动垃圾收集的一种手段。所以我正在使用一个较旧的项目，并用 unique_ptrs 替换指向使用“new”关键字创建的对象的原始指针。但是，我认为我已经解决了所有权问题。

在我的 mainclass.cpp（发布在下面）中，请将您的注意力转移到 init 函数和 3 个 unique_ptrs 到我创建的新实例化对象上。命名为“bg”、“bg2”和“theGrid”。（它们下面被注释掉的声明是它们以前是如何完成的，切换回这个方法，程序运行得很好。）

但是，使用 unique_ptrs，函数 void display() 中的行：

theGrid->doGridCalculations();//MODEL

产生访问冲突。这也是序列中第一次取消引用任何指向的对象，这让我相信 unique_ptr 的所有权已经在某个地方丢失了。但是，unique_ptrs 本身永远不会传递到另一个函数或容器中，并且仍然在 mainclass.cpp 的范围内，因此我没有看到任何机会使用 std::move(theGrid) 来将所有权转移到它需要的地方成为。

主类.cpp：

#include <stdio.h>
#include <GL/glut.h> 
#include <math.h>
#include "Block.h"
#include "dStructs.h"
#include "Grid.h"
#include "Texture.h"
#include "freetype.h"
#include <Windows.h>


//////////////////////////////////////////////////////
///Declare a couple of textures - for the background
//////////////////////////////////////////
Texture* bg;
Texture* bg2;
//and theGrid
Grid* theGrid;

/////////////////////////////////////////////////
///Declare our font
/////////////////////////////////////////////////
freetype::font_data scoreFont;
/////////////////////////////////////////////////////////
//Initialize the variables
///////////////////////////////////////////////////////
typedef dStructs::point point;
const int XSize = 755, YSize = 600;


point offset = {333,145};
point mousePos = {0,0};


void init(void)
{
    //printf("\n......Hello Guy. \n....\nInitilising");
    glMatrixMode(GL_PROJECTION);    
    glLoadIdentity();
    gluOrtho2D(0,XSize,0,YSize);

    //////////////////////////
    //initialise the fonts
    /////////////////////////


    try{
    scoreFont.init("Visitor TT2 BRK Regular.ttf", 20);
    } catch (std::exception &e) {
        MessageBox(NULL, e.what(), "EXCEPTION CAUGHT", MB_OK | MB_ICONINFORMATION);

    }
    ///////////////////////////////////////////////////////////////
    ///bg new MEMORY MANAGED EDITION
    //////////////////////////////////////////////////////////////////
    unique_ptr<Texture> bg(new Texture(1024,1024,"BackGround.png"));
    unique_ptr<Texture> bg2(new Texture(1024,1024,"BackGround2.png"));
    unique_ptr<Grid> theGrid(new Grid(offset));
    /////////////////////////////////////////////////
    /// Old bad-memory-management style of pointed objects
    /////////////////////////////////////////////////
    //bg = new Texture(1024,1024,"BackGround.png");
    //bg2 = new Texture(1024,1024,"BackGround2.png");
    //theGrid = new Grid(offset);

    glClearColor(0,0.4,0.7,1);

    glBlendFunc(GL_SRC_ALPHA, GL_ONE_MINUS_SRC_ALPHA);//activate the alpha blending functionality
    glEnable(GL_BLEND);
    glLineWidth(2);         // Width of the drawing line
    glMatrixMode(GL_MODELVIEW); 
    glDisable(GL_DEPTH_TEST);
    //printf("\nInitialisation Complete");

}

void myPassiveMouse(int x, int y)
{
    //Stupid OGL coordinate system
    y = YSize - y;
    mousePos.x = x;
    mousePos.y = y;
    printf("\nthe mouse coordinates are (%f,%f)",mousePos.x, mousePos.y);
}

void displayGameplayHUD()
{
    ///////////////////////////////
    //SCORE
    //////////////////////////////
    glColor4f(0.7f,0.0f,0.0f,7.0f);//set the colour of the text
    freetype::print(scoreFont, 100,400,"SCORE: ");
    glColor4f(1.0f,1.0f,1.0f,1.0f);//Default texture colour. Makes text white, and all other texture's as theyre meant to be.

}

//////////////////////////////////////////////////////
void display()
{
    ////printf("\nBeginning Display");
    glClear(GL_COLOR_BUFFER_BIT);//clear the colour buffer

    glPushMatrix();
    theGrid->doGridCalculations();//MODEL

    point bgLoc = {XSize/2,YSize/2};
    point bgSize = {XSize,YSize};
    bg2->draw(bgLoc,bgSize);
    theGrid->drawGrid();//DISPLAY
    bg->draw(bgLoc,bgSize);

    if(theGrid->gridState == Grid::STATIC)
    {
        theGrid->hoverOverBlocks(mousePos);//CONTROLLER
    }

    displayGameplayHUD();

    glPopMatrix();

    glFlush();  // Finish the drawing
    glutSwapBuffers();
    ////printf("\nFresh Display Loaded");

    glutPostRedisplay();
}

int main(int argc, char** argv) 
{
  glutInit(&argc, argv);    // GLUT Initialization 
  glutInitDisplayMode(GLUT_RGBA|GLUT_DOUBLE); // Initializing the Display mode
  glutInitWindowSize(755,600);  // Define the window size
  glutCreateWindow("Gem Miners");   // Create the window, with caption.
  init();   // All OpenGL initialization


  //-- Callback functions ---------------------
  glutDisplayFunc(display);
  //glutKeyboardFunc(mykey);
  //glutSpecialFunc(processSpecialKeys);
  //glutSpecialUpFunc(processSpecialUpKeys);
  glutMouseFunc(mymouse);

  glutPassiveMotionFunc(myPassiveMouse);

  glutMainLoop();   // Loop waiting for event 
}

我认为所有权需要在某个时候转移，但我不知道在哪里。

在此先感谢，盖伊

4

3 回答 3

3

这些是全局原始指针：

Texture* bg;
Texture* bg2;
//and theGrid
Grid* theGrid;

这些是完全不相关unique_ptr的，在 init 函数中是局部的。

unique_ptr<Texture> bg(new Texture(1024,1024,"BackGround.png"));
unique_ptr<Texture> bg2(new Texture(1024,1024,"BackGround2.png"));
unique_ptr<Grid> theGrid(new Grid(offset));

当unique_ptrs 超出范围时，它们将被销毁。它们指向的对象也会被销毁，因为unique_ptr它的析构函数就是这样做的。在该过程中，没有任何时候涉及崩溃的全局原始指针。它们被unique_ptr同名的本地 s 隐藏。

您应该将全局原始指针更改为unique_ptrs。然后你可以像这样在 init 函数中设置它们（不要重新声明它们）：

bg.reset(new Texture(1024,1024,"BackGround.png"));
bg2.reset(new Texture(1024,1024,"BackGround2.png"));
theGrid.reset(new Grid(offset));
于 2013-02-05T11:45:51.163 回答
2

您unique_ptr<Grid>的 ininit是该函数的本地函数。意志在unique_ptr<Grid>函数结束时超出范围，摧毁自己和Grid它拥有的东西。似乎您实际上想要一个全局对象unique_ptr<Grid> theGrid;来替换Grid* theGrid;您目前拥有的对象。然后init你可以这样做：

theGrid.reset(new Grid(offset));

theGrid被访问的是display全局theGrid类型Grid*。

unique_ptr对于您尝试创建的其他 s，情况完全相同。

当然，与全局对象相比，传递这些对象会好得多，但是您对 GLUT 的使用使这有点痛苦。

于 2013-02-05T11:40:22.250 回答
2

您在init函数中创建的唯一指针不会修改在文件范围内声明的指针，文件范围内的指针默认初始化为 0 或nullptr（我不太精通 C++11，所以我不确定哪一个）。

您在init函数中所做的是创建三个新对象，其名称会在文件范围内隐藏对象，因此当您在文件范围内使用这些对象时，您会遇到访问冲突，因为它们从未设置为指向任何有效的对象。

于 2013-02-05T11:41:27.023 回答