C# 这个静态方法是线程安全的吗？

我有以下课程：

public static class MetadataManager
{
    // assume that it is thread safe
    public static List<Field> FieldRegistry { get; set; }
}

public class Field
{
    public int ID { get; set; }
    public string Name { get; set; }
}



public static class FieldDataValidationManager
{
    public static bool Validate(int fieldID)
    {
        return MetadataManager.FieldRegistry.FirstOrDefault(f => f.ID == fieldID).ID > 1;
    }

    public static bool Validate(Field field)
    {
        return fieldID.ID > 1;
    }
}

或 User1正在执行

---

FieldDataValidationManager.Validate（field1）

用户2正在执行

FieldDataValidationManager.Validate（field2）

是的，您的代码是线程安全的，因为您的代码只是从列表中读取。静止与否根本不重要

---

如果

列表

上有写操作，则可能存在并发问题。然后，您应该使用

ConcurrentBag

或其他线程安全收集类型。

只要不更改

FieldRegistry

列表的内容，就不会出现并发问题。
但你没有说明你在哪里填的。因此，如果您的实际代码在其他线程调用

Validate

时插入或删除该列表中的条目，则会出现问题（Patrick Hofman建议的方法可能是一个很好的替代方法）

---

但你到底想在这里做什么：

public static bool Validate(int fieldID)
{
    return MetadataManager.FieldRegistry.FirstOrDefault(f => f.ID == fieldID).ID > 1;
}

因此，如果已经存在一个具有该

ID

的条目，并且

ID

大于

1

，则您的

fieldID

似乎是有效的
好的，但是如果列表中尚未包含

fieldID

，则您的方法将抛出

NullReferenceException

。所以你最好把这个方法改成这样：

public static bool Validate(int fieldID)
{        
    return 
        MetadataManager.FieldRegistry.Any(f => f.ID == fieldID) &&
        fieldID > 1;
}

---

亲爱的雷内。我同意你的看法，但这只是一个例子。

public static bool Validate(int fieldID)
{        
    return 
        MetadataManager.FieldRegistry.Any(f => f.ID == fieldID) &&
        fieldID > 1;
}