应将 std::random_shuffle 全部替换为 std::shuffle，并传入正确初始化的随机引擎（如 std::mt19937），使用 std::begin/std::end 统一处理容器，避免多线程共享引擎或对非随机访问容器直接调用。
std::random_shuffle 编译失败直接删掉
遇到 error: 'random_shuffle' is not a member of 'std' 或 warning: 'random_shuffle' is deprecated，说明编译器已按 C++17 标准移除了它。别修、别绕、别加宏兼容——直接全项目搜索替换为 std::shuffle。旧代码里类似 std::random_shuffle(v.begin(), v.end()) 的调用，光改函数名没用，必须补第三个参数。
std::shuffle 必须传 UniformRandomBitGenerator 引擎
不能只写 std::shuffle(v.begin(), v.end())（C++11 起该重载已被移除），也不能传 rand 或 lambda。错误示例：std::shuffle(v.begin(), v.end(), rand) 会编译报错 no matching function；用 std::default_random_engine{} 不显式播种，结果每次运行都一样。
- 推荐写法：
std::mt19937 g{std::random_device{}()}; std::shuffle(v.begin(), v.end(), g); - 测试复现场景用固定种子：
std::mt19937 g{42}; - 别临时构造引擎：
std::shuffle(v.begin(), v.end(), std::mt19937{rd()})是未定义行为，引擎对象在函数调用后立即析构
裸数组和 std::array 的迭代器写法要统一
旧代码用 std::random_shuffle(arr, arr + N) 看似能迁移到 std::shuffle(arr, arr + N, g)，但风险高：若 N 错误或 arr 是函数参数（退化为指针），sizeof(arr) 就失效。更安全的是统一用 std::begin/std::end。
组合式C++代码评审方案，融合静态分析、AI推理、多轮迭代评审和C++专项检查，适用于PR审查、增量代码审查、全项目评审和代码质量评分，触发词包括review cpp、cpp代码评审、C++review、代码审查。
-
std::array<int> a = {1,2,3,4,5}; std::shuffle(a.begin(), a.end(), g);</int>✅ -
int arr[] = {1,2,3,4,5}; std::shuffle(std::begin(arr), std::end(arr), g);✅ -
std::shuffle(arr, arr + 5, g);❌ 容易因长度计算出错导致越界
多线程和小容器的隐性陷阱
std::shuffle 本身不线程安全：多个线程共用同一个 std::mt19937 实例会破坏内部状态，结果不可预测。另外，对 std::list 或自定义非随机访问容器调用 std::shuffle，不会编译失败，但性能可能暴跌到 O(n²)，因为标准库只能靠 std::advance 模拟随机跳转。
立即学习“C++免费学习笔记（深入）”；
- 多线程中每个线程应持有独立的引擎实例，不要全局共享
g - 确认容器迭代器类型是
RandomAccessIterator：优先用std::vector、std::array或原生数组 - 若必须 shuffle
std::list，先转std::vector，shuffle 后重建，比硬扛 O(n²) 更实际
