C++中使用RAII对象结合逗号运算符是否安全且符合最佳实践？

首先明确：你写的这个逗号表达式版本完全不安全，也不符合最佳实践，绝对不能这么用。

原因分析 #

在这段代码里：

T get_data() {
  return (m_mutex.lock(), m_data);
}

逗号表达式的执行逻辑是先调用m_mutex.lock()创建一个临时Guard对象，再求值m_data。但关键问题是：这个临时Guard的生命周期只到整个逗号表达式结束为止——也就是在返回m_data之前，临时Guard就会被销毁，锁也就被提前释放了。

这直接导致读取m_data时完全无锁保护，彻底失去了加锁的意义，必然引发线程安全问题（比如数据竞争）。

正确写法的核心逻辑 #

你原本的代码是标准的RAII锁正确用法：

T get_data() {
  auto guard = m_mutex.lock();
  return m_data;
}

这里的guard是函数局部变量，它的生命周期会持续到get_data()函数执行完毕（return m_data之后）。所以读取m_data时锁是持有的，直到函数返回后，guard的析构函数才会自动释放锁，完美保证了线程安全。

额外提醒 #

别抱有临时对象生命周期会延长到返回值之后的侥幸，C++标准明确规定：逗号表达式中的临时对象会在整个表达式求值完成后立即销毁，这种写法从根本上破坏了RAII锁的核心逻辑。

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