下面的用于多线程的代码有问题吗?我自己没把握.

[捕鲸叉]

bool CHardware::IsReady()
{
	bool isready;
	m_CriticalSection.Lock();
	isready = m_IsReady;
	m_CriticalSection.Unlock();
	return isready;
}

[zhousitiaoda]

假如m_IsReady的值是false，m_CriticalSection.Unlock()后另一个线程把m_IsReady的值更新为true。此时IsReady返回的是个false。。能接受这种结果就没问题，不能接受就有问题。 封装个类，在析构函数里Unlock是不是更好？使用的时候CAutoLock autoLock;autoLock.lock();像这样。

[gg606]

就这个函数来讲，没有任何问题，甚至读取一个值是不需要使用关键段的。 只是这个函数的用处令人生疑，因为它所测试的结果等到使用时可能又变了。

m_CriticalSection.Lock();
if (m_IsReady)
// 应该把要做的事情放在这里
m_CriticalSection.Unlock();