Question

我正在尝试在另一个class Tile内创建一个自定义class Board的网络，该网络正在存储std::array<std::shared_ptr<Tile>, 34>。我正在std::shared_ptr<Tile>中使用class Tile个对象来显示连接了哪些节点。但是，当我初始化34个Tile对象中的每一个时，列表中包括的尚未初始化的对象都不会从nullptr更新为它们自己的引用。

#include <memory>
#include <array>
#include <cstddef>

struct Tile;
template <const std::size_t N> using TileLink  = std::array<const std::shared_ptr<const Tile>, N>;
template <const std::size_t N> using TileArray = std::array<std::shared_ptr<const Tile>, N>;

struct Tile {
    /* surrounding tiles */
    const std::size_t ID;
    const std::size_t num_adj;
    const TileLink<6> adjacent;
    Tile (std::size_t id, std::size_t n, const TileLink<6> & arr) : ID (id), num_adj (n), adjacent (arr) { }
    ~Tile (void) = default;
};

class Board {
private:
    TileArray<34> tiles;
public:
    Board (void);
    ~Board (void) = default;
};

Board :: Board (void) {
    /* Column 1 */
    tiles[0]  = std::make_shared<Tile> ( 0, 3, TileLink<6>{ tiles[1],  tiles[4],  tiles[5],  nullptr,   nullptr,   nullptr   });
    tiles[1]  = std::make_shared<Tile> ( 1, 4, TileLink<6>{ tiles[0],  tiles[2],  tiles[5],  tiles[6],  nullptr,   nullptr   });
    tiles[2]  = std::make_shared<Tile> ( 2, 3, TileLink<6>{ tiles[1],  tiles[6],  tiles[7],  nullptr,   nullptr,   nullptr   });
    /* Column 2 */
    tiles[3]  = std::make_shared<Tile> ( 3, 2, TileLink<6>{ tiles[4],  tiles[9],  nullptr,   nullptr,   nullptr,   nullptr   });
    tiles[4]  = std::make_shared<Tile> ( 4, 5, TileLink<6>{ tiles[0],  tiles[3],  tiles[5],  tiles[9],  tiles[10], nullptr   });
    tiles[5]  = std::make_shared<Tile> ( 5, 6, TileLink<6>{ tiles[0],  tiles[1],  tiles[4],  tiles[6],  tiles[10], tiles[11] });
    tiles[6]  = std::make_shared<Tile> ( 6, 6, TileLink<6>{ tiles[1],  tiles[2],  tiles[5],  tiles[7],  tiles[11], tiles[12] });
    tiles[7]  = std::make_shared<Tile> ( 7, 5, TileLink<6>{ tiles[2],  tiles[6],  tiles[8],  tiles[12], tiles[13], nullptr   });
    tiles[8]  = std::make_shared<Tile> ( 8, 2, TileLink<6>{ tiles[7],  tiles[13], nullptr,   nullptr,   nullptr,   nullptr   });
    /* Columns 4 - 6 */
    ...
    ...
    ...
    /* Column 7 */
    tiles[31] = std::make_shared<Tile> (31, 3, TileLink<6>{ tiles[26], tiles[27], tiles[32], nullptr,   nullptr,   nullptr   });
    tiles[32] = std::make_shared<Tile> (32, 4, TileLink<6>{ tiles[27], tiles[28], tiles[31], tiles[33], nullptr,   nullptr   });
    tiles[33] = std::make_shared<Tile> (33, 3, TileLink<6>{ tiles[28], tiles[29], tiles[32], nullptr,   nullptr,   nullptr   });
}

我传递给std::array构造函数的Tile不变。我将如何改变它？

任何帮助将不胜感激。很抱歉，这么长时间提出这个问题，我真的不知道该如何做一个较小的例子。

Answer 1

这里：

tiles[0]  = std::make_shared<Tile> ( 0, 3, TileLink<6>{ tiles[1],  tiles[4],  tiles[5],  nullptr,   nullptr,   nullptr   });

tiles[1]，tiles[4]等尚未初始化，并且被shared_ptr设置为nullptr。稍后在初始化tiles[1]时，它不会更改tiles[0]内部的内容。

您完全滥用shared_ptr。由于tile数组的大小是固定的，因此只需分配所有所需的内存一次，然后使用非所有者指针即可。因为有个内存所有者，所以它是Board。

Answer 2

我已经解决了我的问题-感谢@ t.niese和@fjardon的帮助。

我不得不使Tile更加复杂，以解决这里的实际问题：我无法从类中的std :: array创建唯一的Tile对象。因此，我添加了一个复制构造函数和一个赋值运算符，删除了智能指针和const-ness，并制作了一个类型为Tile的数组。

class Tile;
template <const std::size_t N> using TileLink  = std::array<const Tile *, N>;
template <const std::size_t N> using TileArray = std::array<Tile, N>;

enum class OWNERSHIP { PLAYER_1, PLAYER_2, CONTESTED };

class Tile {
private:
    OWNERSHIP territory;
    /* surrounding tiles */
    std::size_t ID;
    std::size_t num_adj;
    TileLink<6> adjacent = { nullptr };
public:
    Tile (void) { }
    Tile (OWNERSHIP own, std::size_t id, std::size_t num, const TileLink<6> & arr);
    Tile (const Tile & rhs);
    Tile & operator = (Tile other);
    ~Tile (void) = default;
};

class Board {
private:
    TileArray<34> tiles;
    void create_col_1_and_7 (void);
    void create_col_2 (void);
    ...
    void create_col_6 (void);
public:
    Board (void);
    ~Board (void) = default;
};

每个初始化现在看起来像：

tiles[0]  = Tile (OWNERSHIP::PLAYER_2,   0, 3, TileLink<6>{ &tiles[1],  &tiles[4],  &tiles[5],  nullptr,    nullptr,    nullptr    });
tiles[1]  = Tile (OWNERSHIP::CONTESTED,  1, 4, TileLink<6>{ &tiles[0],  &tiles[2],  &tiles[5],  &tiles[6],  nullptr,    nullptr    });
...
tiles[32] = Tile (OWNERSHIP::CONTESTED, 32, 4, TileLink<6>{ &tiles[27], &tiles[28], &tiles[31], &tiles[33], nullptr,    nullptr    });
tiles[33] = Tile (OWNERSHIP::PLAYER_1,  33, 3, TileLink<6>{ &tiles[28], &tiles[29], &tiles[32], nullptr,    nullptr,    nullptr    });

std :: shared_ptr不更新其计数

2 个答案: