std::mutex不能保护FSM的process_event()可重入性，因其不可重入，嵌套调用会导致死锁；正确方案是避免共享可变状态，采用线程独占、状态快照、消息队列或std::variant+std::visit值语义实现，配合原子操作或线程局部存储确保安全。
为什么 std::mutex 不能直接保护 FSM 的 process_event()
可重入性不是“加个锁就完事”，而是要求同一对象在被某个线程调用时，能安全响应其他线程（甚至同一线程）的嵌套调用。如果在 process_event() 入口粗暴加 std::mutex::lock()，一旦状态转移中触发回调又调用自身（比如事件处理函数里发新事件），就会死锁——std::mutex 不可重入。
真正可行的路径是：放弃互斥锁保护整个方法，转而确保状态数据结构本身无共享可变状态，或使用线程局部/不可变建模。常见做法包括：
- FSM 实例按线程独占分配（最简单，推荐默认方案）
- 若必须共享实例，用
std::shared_mutex+ 状态快照读取 + 原子切换（仅适用于只读查询频繁、变更稀疏的场景） - 彻底放弃共享，用消息队列 + event loop 将所有事件序列化到单一线程（如
std::thread+std::queue+std::condition_variable）
用 std::variant + std::visit 替代虚函数表实现状态跳转
传统虚函数方式（每个状态一个子类）在多线程下易因 vptr 修改或动态绑定引入间接开销与缓存抖动；更严重的是，虚析构+继承树会让状态生命周期难以对齐线程边界。现代 C++ 更倾向值语义 + 无指针跳转。
示例核心跳转逻辑：
立即学习“C++免费学习笔记（深入）”；
struct Idle {};
struct Running {};
struct Paused {};
using State = std::variant<Idle, Running, Paused>;
State transition(const State& s, const Event& e) {
return std::visit([](const auto& state, const auto& event) -> State {
if constexpr (std::is_same_v<std::decay_t<decltype(state)>, Idle>) {
if (std::holds_alternative<StartEvent>(event)) return Running{};
}
else if constexpr (std::is_same_v<std::decay_t<decltype(state)>, Running>) {
if (std::holds_alternative<PauseEvent>(event)) return Paused{};
}
// ... 其他分支
return state; // 默认不转移
}, s, e);
}
注意：std::visit 要求所有分支返回相同类型，且编译期穷举——这反而成了类型安全优势。但别忘了：每次 transition() 返回新 State，必须配合原子赋值或线程局部存储，否则仍存在竞态。
组合式C++代码评审方案，融合静态分析、AI推理、多轮迭代评审和C++专项检查，适用于PR审查、增量代码审查、全项目评审和代码质量评分，触发词包括review cpp、cpp代码评审、C++review、代码审查。
如何让 process_event() 支持递归调用而不崩
关键不在“能不能调”，而在“调的时候状态怎么管”。典型崩坏场景：A 线程进 process_event() → 进入 Running::handle() → 触发内部事件 → 再次进 process_event() → 此时 m_current_state 可能正被修改中。
解决方案只有两个有效方向：
- 强制单次调用深度：用线程局部静态
bool processing{false}拦截嵌套调用，抛出std::logic_error("reentrant call not allowed")—— 明确拒绝，比静默错乱好 - 允许嵌套但隔离状态：把当前状态拷贝进栈，所有中间转移只作用于该副本；顶层返回前才用 CAS 原子提交（需
std::atomic<State>支持，C++20 起可用std::atomic_ref包装std::variant） - 回调不直接调
process_event()，改走异步通知（std::function<void()>队列 + 循环 dispatch）—— 把重入变成顺序化
std::atomic<State> 在 C++17 下无法直接使用的原因和绕过方式
std::variant 默认不满足 std::atomic 的 TriviallyCopyable + LockFree 要求，C++17 标准库不提供 std::atomic<std::variant<...>> 特化。强行 reinterpret_cast 或 memcpy 会触发未定义行为。
可行替代方案（按推荐度排序）：
- 用
std::atomic<uint8_t>编码状态 ID（0=Idle, 1=Running…），再配查表函数映射到实际状态值——轻量、无锁、完全兼容 - 封装一层
class AtomicState { std::atomic<size_t> m_idx; std::array<State, N> m_states; }，靠索引原子切换，状态值本身不动 - 升级到 C++20 并启用
std::atomic_ref<State>（前提是State是 trivially copyable，即不含虚函数、引用、非平凡构造/析构）
别碰 std::mutex 包裹 std::variant——那不是可重入，那是给自己套枷锁。
