在线程之间共享麻烦，天真的方法有什么陷阱？

Question

我需要一个简单的布尔值来通知长时间运行的进程中止，因为它是循环的一部分，如下所示：

bool Abort = false;
void MyLongRunningMethod()
{
while(true)
{
    // loop code goes here, takes 2 hours ish to run
    if(Abort)
       break;
}
}

我知道如何正确编写此代码，但是我想知道，如果我原样保留它，会有任何风险吗？ 还是唯一的风险（对于我而言，这不是问题），循环可能会运行额外的时间（在编写bool的同时读取bool）。

我是否会遗漏任何怪异的边缘情况，或者如果我对以下语句感到满意，那么这是否真的可以解决：“当中止为真时，该方法应该中断，但可能在没有副作用的情况下再运行一段时间”？ 还是我错过了可以完全打破世界的东西？

Answer 1

首先，此处的bool应标记为volatile以创建内存屏障/防止某些缓存给您带来假阴性。 其次， CancellationTokenSource是一个很好的选择：

private readonly CancellationTokenSource _cancel = new CancellationTokenSource();

void MyLongRunningMethod()
{
    while (!_cancel.IsCancellationRequested)
    {
        // loop code goes here, takes 2 hours ish to run
    }
}

然后停止循环：

_cancel.Cancel();

如果希望保留其最初的编写方式，请执行while (!Abort)代替while (true)