c# - 这个扩展方法线程安全吗？

Question

我已经创建了扩展方法来在对象之间进行映射，但我担心它可能不是线程安全的。方法如下:

public static SavableRecord ToSavableRecordForMongoDB(this Record record)
{
    SavableRecord savableRecord = new SavableRecord();

    if (record.Fields == null)
        throw new ArgumentException("Fields of record cannot be null");

    if (string.IsNullOrWhiteSpace(record.id))
        savableRecord._id = record.id;

    foreach (KeyValuePair<string, Field> item in record.Fields)
        savableRecord.Fields[item.Key] = new Field(item.Value);

    return savableRecord;
}

如果此方法不是线程安全的，我该怎么做才能使其线程安全。

更新

record 对象在 MVC 项目的 Controller 中传递。 record 对象在此 Controller 或其路径中永远不会更改。

Answer 1

这取决于 Record 是否不可变。

如果 Record 的 id 和 Fields 仅在对象创建期间设置，则此方法是纯线程安全的 - 它不会产生副作用(假设 SavableRecord 不会产生副作用)并且总是会为相同的输入返回相同的结果。您可以在十个线程上并行调用它，而不会发生任何冲突、数据损坏或困惑。

但是，如果 Fields(或 id)是可变的，一个线程可以在执行此方法时更改值，从而导致意外结果，从 开始字段 迭代器抛出异常，属性在您的验证检查 之后被修改为无效值，甚至丢失值，或者对该函数的两次调用返回不同的值。