C++指针数组无法从指针函数取值触发段错误问题求助

你遇到的崩溃是由三处指针类型错配、内存生命周期管理错误共同触发的，逐点拆解如下：

• Map构造函数中的成员指针赋值完全错误
  你在构造函数内定义的Land *map[map_X][map_Y]是栈上的临时局部数组，构造函数执行完成后这块栈内存会被立刻回收。同时你写的this->map = **&map属于完全的类型逻辑错配：
  二维栈数组Land *map[map_X][map_Y]的本质类型是Land* (*)[map_Y]，连续两层解引用根本拿不到符合Land**类型要求的、可按二维索引访问的指针，最终导致类成员map成为指向非法回收内存的野指针。
• BuildMap函数中的赋值操作类型不匹配
  GetRandomLand()的返回值是指向堆内存的Land*指针，你在代码中写的map[k][i] = *GetRandomLand()相当于把指针指向的Land对象解引用后，把对象值硬存到本来用于存放指针的内存位置。这种操作会直接打乱指针数组的内存布局，后续调用map[k][i].GetSymbol()时，实际是把一段无意义的内存值当Land对象来解析，自然触发段错误。
• 可优化的潜在问题
  你在GetRandomLand()函数内部每次调用都执行srand(time(NULL))，由于循环执行速度远快于time()的秒级精度，会导致短时间内生成的随机地块类型完全一致。这部分不是段错误的直接原因，但会严重影响地图生成效果，建议把srand(time(NULL))移到程序初始化阶段只执行一次。

1. 替换Map构造函数中原有的栈数组逻辑，为Land** map成员正确分配堆内存：

   Map(Player &player) {
       // 先分配横向索引对应的指针数组空间
       this->map = new Land*[map_X];
       // 再为每个横向索引分配纵向的指针数组空间
       for(size_t k = 0; k < map_X; k++) {
           this->map[k] = new Land*[map_Y];
       }
       BuildMap(player);
   }
   

2. 修正BuildMap中的赋值逻辑，去掉多余的解引用操作：

   // 原错误写法：map[k][i] = *GetRandomLand();
   // 正确写法：直接存储函数返回的堆对象指针
   map[k][i] = GetRandomLand();
   

3. 补写Map类的析构函数，遍历二维数组依次释放动态生成的Land对象、两层指针数组空间，避免内存泄漏。

内容的提问来源于stack exchange，提问作者Herodegon