Добрый день.
Я тут читаю про синхронизацию между потоками процесса, и на msdn есть такой пример:
http://msdn.microsoft.com/en-us/library/ms686355(VS.85).aspx
Они пишут что так писать не thread-safe:
| Код |
int iValue;
BOOL fValueHasBeenComputed = FALSE;
extern int ComputeValue();
void CacheComputedValue()
{
if (!fValueHasBeenComputed)
{
iValue = ComputeValue();
fValueHasBeenComputed = TRUE;
}
}
BOOL FetchComputedValue(int *piResult)
{
if (fValueHasBeenComputed)
{
*piResult = iValue;
return TRUE;
}
else return FALSE;
}
|
И по этому лучше писать так:
| Код |
int iValue;
BOOL fValueHasBeenComputed = FALSE;
extern int ComputeValue();
void CacheComputedValue()
{
if (InterlockedCompareExchange((LONG*)&fValueHasBeenComputed,
(LONG) fValueHasBeenComputed,
(LONG) fValueHasBeenComputed)==FALSE)
{
InterlockedExchange ((LONG*)&iValue, (LONG)ComputeValue());
InterlockedExchange ((LONG*)&fValueHasBeenComputed, TRUE);
}
}
BOOL FetchComputedValue(int *piResult)
{
if (InterlockedCompareExchange((LONG*)&fValueHasBeenComputed,
(LONG) fValueHasBeenComputed,
(LONG) fValueHasBeenComputed)==TRUE)
{
InterlockedExchange((LONG*)piResult, (LONG)iValue);
return TRUE;
}
else return FALSE;
}
|
Зачем нужны InterlockedExchange мне понятно, но вот насчет InterlockedCompareExchange...
Во первых, по моему это по мухе с пулемета, и сравнивать и писать то же значение только для того, чтобы еще раз сравнить.
Во вторых, нужно ли оно тут вообще?
if(fValueHasBeenComputed) ведь только один раз читает память, что есть, как я понимаю, атомная операция.
|
)