如何不使用static int遍历Windows的CHAR_INFO结构体

首先要明确：循环变量用static int能运行只是巧合，它并没有解决真正的问题，只是掩盖了内存非法访问的bug。咱们一步步拆解问题，找到不用static的正确方案：

一、定位崩溃的核心原因 #

你提到最小复现示例在srcWriteRect->Top = 4;触发SIGSEGV段错误，这说明srcWriteRect是无效指针（野指针、指向已释放内存、未初始化的随机地址），和循环变量是static int还是普通int没有直接关系。

为什么用static变量时程序能运行？因为static变量存储在程序的静态存储区，而普通int在栈上。如果你的程序存在栈内存被破坏（比如越界写、野指针覆盖），static变量不会被影响，所以碰巧能正常执行，但这只是权宜之计，不是根本解决方法。

二、不用static int的正确遍历方案 #

只要修复内存访问的bug，普通int循环变量完全可以正常工作，以下是修正后的代码示例：

1. 单循环遍历（替代第一个可行代码） #

const int MAP_SIZE = 45;
CHAR_INFO map[MAP_SIZE];
const int TILE_CENTER = 22;

// 用普通int循环变量，完全没问题
for (int j = 0; j < MAP_SIZE; j++) {
    // 直接用j判断是否是中心索引，逻辑正确
    map[j].Char.UnicodeChar = (j == TILE_CENTER ? tiles[0].getAsciiChar() : L'▓');
    map[j].Attributes = 0x20;
}

2. 嵌套循环遍历（修复第二个崩溃代码的逻辑错误） #

你原来的嵌套循环里有个明显的逻辑错误：j == TILE_CENTER永远为假（j的范围是0~8，而TILE_CENTER是22），应该判断整个数组的索引是否等于中心值：

const int MAP_SIZE = 45;
CHAR_INFO map[MAP_SIZE];
const int TILE_CENTER = 22;

// 普通int循环变量，正常使用
for (int i = 0; i < 5; i++) {
    for (int j = 0; j < 9; j++) {
        int idx = j + (i * 9);
        // 修正判断逻辑：检查当前索引是否是中心
        map[idx].Char.UnicodeChar = (idx == TILE_CENTER ? tiles[0].getAsciiChar() : L' ');
        map[idx].Attributes = 0x20;
    }
}

三、必须修复的srcWriteRect指针问题 #

要彻底解决崩溃，重点要处理srcWriteRect的合法性：

• 如果是通过Windows API（比如GetConsoleScreenBufferInfo）获取的指针，一定要检查API的返回值，确保调用成功：

  SMALL_RECT srcWriteRect;
  CONSOLE_SCREEN_BUFFER_INFO csbi;
  if (!GetConsoleScreenBufferInfo(GetStdHandle(STD_OUTPUT_HANDLE), &csbi)) {
      // 处理错误，比如打印GetLastError()的结果
      return 1;
  }
  // 用csbi中的信息初始化srcWriteRect，或者直接使用csbi.srWindow
  srcWriteRect = csbi.srWindow;
  srcWriteRect.Top = 4;
  

• 如果是自己定义的指针，确保它指向有效的内存：不要使用未初始化的指针，不要指向已经销毁的栈变量，堆分配的指针要确保new/malloc成功。

四、额外的安全建议 #

• 避免硬编码数组长度，用MAP_SIZE代替魔法数字（比如45、9），减少越界风险；
• 开启编译器的警告（比如g++用-Wall -Wextra），能帮你提前发现未初始化变量、逻辑错误等问题；
• 用调试工具（比如gdb）查看崩溃时的调用栈，确认srcWriteRect的具体值，更容易定位问题。

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