C#线程安全代码

Question

这段代码线程安全吗？

DoStuff 使用任务在新线程中调用。

Task.Factory.StartNew(() => DoStuff());

private void DoStuff()
{
    List<SalesRecord> salesRecords = new List<SalesRecord>();
    SalesRecord salesRecord1 = new SalesRecord {Amount = 10.0, Sales = 1};
    SalesRecord salesRecord2 = new SalesRecord {Amount = 15.0, Sales = 1};
    SalesRecord salesRecord3 = new SalesRecord {Amount = 1.0, Sales = 2};
    salesRecords.Add(salesRecord1);
    salesRecords.Add(salesRecord2);
    salesRecords.Add(salesRecord3);
    SalesRecord result = Util.SumSales(salesRecords);
}

一个仅用于存储数据的结构：

public struct SalesRecord
{
    public uint Sales;
    public double Amount;
}

东西

public static class Util
{
    public static SalesRecord SumSales(List<SalesRecord> records)
    {
        SalesRecord result = new SalesRecord();

        result.Amount = records.FindAll(record => (record.Sales == 1)).Sum(record => record.Amount);
        result.Sales = 1;
        return result;
    }
}

Answer 1

您的代码看起来不错。即使 DoStuff 是通过 Task 启动的，我也没有在您处理 class 变量的代码中看到任何地方，它们都是局部变量。

参考： C# : What if a static method is called from multiple threads?

Answer 2

Is this code thread safe?

是的，是的。此代码不使用任何共享状态。根据定义，不使用任何共享状态的代码是线程安全的。

您可以在任意数量的线程中并发调用 DoStuff 而不会出现任何问题。也就是说，DoStuff 方法不是很有用。

Answer 3

发布给那些试图弄清楚为什么即使使用静态方法它也是线程安全的

在这种特殊情况下，对 Util.SumSales(salesRecords) 的访问是线程安全的，因为它是线程本地方法（调用此方法的线程将自己的数据副本提供给方法和数据该线程中的访问权限对调用它的线程是独占的。

当您调用以下代码时：

Task.Factory.StartNew(() => DoStuff());

您分配了一个新线程来处理 DoStuff()。 DoStuff 中发生的所有事情（除非您引入可以在其外部操作的变量）都是该新线程独有的。

我假设您将从 DoStuff() 获得的内容做更多的事情。在这种情况下，您应该将结果存储在 Concurrent Collections.

中

C#线程安全代码

C# Thread safe code

c#

multithreading