动态初始化std :: vector <unique_ptr <class t =“”>＆gt; </unique_ptr <class>

Question

我有一个Grid类声明如下：

Grid.h

#ifndef DATATEST_GRID_H
#define DATATEST_GRID_H

#include <memory>
#include <vector>
#include "Position.h"

class Grid
{
public:
    Grid(int length_x, int length_y);
    ~Grid();

    Position *at(int x, int y);
    void printGrid();

private:
    int length_x, length_y;
    std::vector<std::unique_ptr<Position>> grid;
};

#endif

它最重要的成员变量是vector<unique_ptr<Position>>，我用它来模拟一个二维数组，其大小是在运行时确定的。该职位的类声明如下：

Position.h

#ifndef DATATEST_POSITION_H
#define DATATEST_POSITION_H

#include <memory>
#include <string>

class Position {
public:
    Position(int x, int y);
    ~Position();

    std::string toString();
    int getX() { return x; };
    int getY() { return y; };

private:
    int x, y;
};

#endif

在Grid的构造函数中，我想创建所需数量的Positions并将它们添加到vector<unique_ptr<Position>>。

Grid.cpp

#include "Grid.h"
#include <iostream>
#include <memory>
#include <vector>
#include "Position.h"

Grid::Grid(int length_x, int length_y) 
    : length_x(length_x), length_y(length_y)
{
    grid.resize(length_x * length_y);   

    for (int x = 0; x < length_x; x++) {
        for (int y = 0; y < length_y; y++) {
            /* Option 1 */
            std::unique_ptr<Position> temp = std::make_unique<Position>(x, y);
            grid.push_back(std::move(temp));

            /* Option 2 */
            // std::unique_ptr<Position> temp = std::make_unique<Position>(x, y);
            // grid.emplace_back(std::move(temp));

            /* Option 3 */
            // grid.push_back(std::make_unique<Position>(x, y)); 

            /* Option 4 */
            // grid.emplace_back(std::make_unique<Position>(x, y));
        }
    }
}

Grid::~Grid()
{
    grid.clear();
}

Position *Grid::at(int x, int y)
{
    if (x < 0 || x >= length_x || y < 0 || y >= length_y) {
        return nullptr;
    }
    else {
        return grid.at(x * (length_y) + y).get();
    }
}

void Grid::printGrid()
{
    for (int i = 0; i < grid.size(); i++) {
        std::cout << grid.at(i)->toString() << std::endl;
    }
}  

我正在通过为每个unique_ptr<Position>调用Position :: toString来测试访问权限，并将结果打印到控制台。

Position.cpp

#include "Position.h"
#include <string>

Position::Position(int x, int y) 
    : x(x), y(y)
{
}

Position::~Position()
{
}

std::string Position::toString()
{
    return "Position(" + std::to_string(x) + ", " + std::to_string(y) + ")";
}

最后，主要功能：

Main.cpp的

#include "Grid.h"
#include "Position.h"

int main()
{
    Grid g(2, 2);
    g.printGrid();

    return 0;
}

无论我填充vector<unique_ptr<Position>>的哪种方式，我总是会收到以下错误：

CombatSim.exe中0x0087D8A3的第一次机会异常：0xC0000005：访问冲突读取位置0x00000000。 CombatSim.exe中0x0087D8A3处的未处理异常：0xC0000005：访问冲突读取位置0x00000000。

据我所知，我可能有以下四个问题之一：
1）我正在将创建的Position对象的unique_ptr添加到向量中 2）我使用错误的方法动态创建Position对象 3）以上所有。
4）我不知道的事情。

Answer 1

如果您之后要std::vector::reserve(n)这些 n 元素，请使用std::vector::resize(n)代替push_back。

resize将填充带有初始化对象的向量，因此vector.size()之后将 n 。

reserve将为所有 n 元素保留足够的空间，但不会插入任何对象。

Answer 2

似乎调整网格大小将首先插入所有指向length_x * length_y的{​​{1}}个唯一指针。您的0应该没问题，但您的真实元素从向量中的push_back位置开始。我想删除这行

length_x * length_y

应该解决问题。也许其他人可以解释为什么会这样。