遇到二维结构数组 C++ 的问题

Question

我已经搜索了这个网站和谷歌，并没有真正找到任何可以解决我问题的东西。我现在正在尝试编写游戏，该游戏包含玩家可以移动的地形图块地图。我想将图块存储在 10x10 数组中，但在初始化数组时遇到了问题。

我可以初始化数组的第一个维度，但错误在于在第一个 for 循环中初始化第二个维度。

我的代码如下所示：

//tile on the "map"
struct tile
{
    char type;
    bool isWall;
};

void initializeMap(tile * map)
{
  int index1, index2;

  for(index1 = 0; index1 < 10; index1++)
  {
    map[index1] = new tile[10];

    for(index2 = 0; index2 < 10; index2++)
    {

    }
  }
}

int main()
{
    tile * tileMap = new tile[10];
    initializeMap(tileMap);

    return 0;
}

我收到此错误：

C:\Users\----\Desktop\TextGame.cpp||In function 'void initializeMap(tile*)':|
C:\Users\----\Desktop\TextGame.cpp|39|error: no match for 'operator=' in '*(map + ((unsigned int)(((unsigned int)index1) * 2u))) = (tile*)operator new [](20u)'|
C:\Users\----\Desktop\TextGame.cpp|9|note: candidates are: tile& tile::operator=(const tile&)|
||=== Build finished: 1 errors, 0 warnings ===|

Answer 1

我建议使用std::vector 共享指向磁贴的指针。这将允许您拥有不同的图块并将它们存储到向量中：

struct Tile; // As you declared it.

struct Water_Tile : public Tile;
struct Mountain_Tile : public Tile;
struct Desert_Tile : public Tile;

// ...
typedef std::vector< boost::shared_ptr<Tile> > Tile_Container;

//...
Tile_Container    my_world(10 * 10);

// ...
my_world[20] = new Water_Tile;

共享指针的一个很好的优点是它可以为您处理内存管理（删除）。

Answer 2

注意：从技术上讲，这不是二维数组，而是结构数组的数组。

也就是说，那个注释反映了你的代码的问题：你有一个数组数组，所以第一次分配应该分配一个数组，其内容是指向其他数组的指针：

tile** tilemap = new tile*[10];

for( std::size_t i = 0 ; i < 10 ; ++i )
    tilemap[i] = new tile[10];

....

//Dealocation at program finish:

for( std::size_t i = 0 ; i < 10 ; ++i )
    delete tilemap[i];

delete tilemap;

但是，如果您使用标准容器（例如 std::vector）而不是容易出错的手动内存管理，您的代码可能会得到改进：

std::vector<std::vector<tile>> tilemap( 10 );

for( std::size_t i = 0 ; i < 10 ; ++i )
    tilemap.emplace_back( 10 );

Answer 3

您正在尝试使用以下命令将实际对象设置为指针：

map[index1] = new tile[10];

map 是 tile*。然而，map[index1] 是一个被尊重的tile*，使其实际上是一个tile，它不能等于new tile[10] 给你的tile*。

因此，您的代码将更好地工作为：

struct tile {
   char type;
   bool isWall;
};

/**
 * Initialize the map
 * @param map The array of tile pointers
 */
void initializeMap(tile** map) {
   int index1, index2;
   for (index1 = 0; index1 < 10; index1++) {

      // Set each element of the tile* array 
      // to another array of tile pointers
      map[index1] = new tile[10];

      for (index2 = 0; index2 < 10; index2++) {
         // Do Something
      }
   }
}

int main() {
   // Create a pointer to a set of tile pointers
   tile** tileMap = new tile*[10];
   // Pass it to the initializer
   initializeMap(tileMap);
   return 0;
}