使用cv::forEach()实现图像转ASCII算法遇乱序问题，如何修复？

1. cv::forEach()的并行执行特性：OpenCV的forEach()默认采用多线程并行处理像素，多个线程同时向std::cout输出字符，导致输出顺序被随机打乱——线程的执行顺序不固定，字符会交错输出。
2. position索引与换行逻辑错误：
   • forEach()的position数组中，position[0]是像素的行号，position[1]是列号，代码中获取像素的逻辑是对的，但换行判断position[1] >= frame.cols永远不会成立（列索引最大为frame.cols-1），这会导致完全没有换行。
   • 原代码的换行逻辑同样错误：c >= frame.cols永远不成立，正确的做法是每处理完一行所有列后输出换行符。

要兼顾并行优化和正确的输出顺序，最佳方式是先在并行处理中收集每行的ASCII字符，最后统一输出所有行。这样既利用了forEach()的并行性，又避免了多线程输出冲突。

修复后的代码：

std::unordered_map<int, char> brightness_map;
for (int i = 0; i <= 255; i++)
{
    if (i < 30)
        brightness_map[i] = ' ';
    else if (i < 55)
        brightness_map[i] = '.';
    else if (i < 85)
        brightness_map[i] = ':';
    else if (i < 115)
        brightness_map[i] = '-';
    else if (i < 140)
        brightness_map[i] = '=';
    else if (i < 165)
        brightness_map[i] = '/';
    else if (i < 180)
        brightness_map[i] = 'o';
    else if (i < 210)
        brightness_map[i] = '0';
    else
        brightness_map[i] = '@';
}

// 预先创建存储每行ASCII的容器，初始化为空字符串
std::vector<std::string> ascii_lines(frame.rows, std::string(frame.cols, ' '));

// 并行处理像素，将字符写入对应行的位置
frame.forEach<uint8_t>([&brightness_map, &ascii_lines](const uint8_t& pixel, const int* position) -> void
{
    int row = position[0];
    int col = position[1];
    ascii_lines[row][col] = brightness_map[pixel];
});

// 统一输出所有行
std::cout << "\n\n\n\n\n";
for (const auto& line : ascii_lines)
{
    std::cout << line << '\n';
}

• 用std::array替代std::unordered_map存储亮度映射：因为像素值是0-255的连续整数，数组的访问速度远快于哈希表，能进一步提升性能。示例：

  std::array<char, 256> brightness_map;
  for (int i = 0; i <= 255; i++)
  {
      if (i < 30)
          brightness_map[i] = ' ';
      else if (i < 55)
          brightness_map[i] = '.';
      // ... 其余映射逻辑相同
  }
  

• 原代码的换行逻辑修正：如果保持串行循环，应该将换行放在内层循环之后：

  for (int r = 0; r < frame.rows; r++)
  {
      for (int c = 0; c < frame.cols; c++)
      {
          gray = frame.at<uint8_t>(r, c); // 等价于Point(c, r)，更直观
          std::cout << brightness_map[gray];
      }
      std::cout << '\n'; // 每处理完一行输出换行
  }
  

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