这个静态方法是否是线程安全的?

时间:2016-03-29 09:23:29

标签: c# multithreading methods static

我有以下课程:

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和User2同时调用静态方法,是否存在并发问题?

FieldDataValidationManager.Validate(111) 

或 User1正在执行 FieldDataValidationManager.Validate(field1)和User2正在执行FieldDataValidationManager.Validate(field2)

2 个答案:

答案 0 :(得分:8)

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

如果List<T>上有写操作,则可能存在并发问题。然后,您应该使用ConcurrentBag<T>或其他线程安全的集合类型。

答案 1 :(得分:3)

只要您不更改beacuse there is a timeout of 10 seconds that the system allows before considering the receiver to be blocked and a candidate to be killed-列表的内容,就不存在并发问题。
但是你没有显示你填写该列表的位置。因此,如果您的实际代码确实在该列表中插入或删除了条目,而其他线程正在调用FieldRegistry则会出现问题(Patrick {{{{}}建议的ConcurrentBag<T>可能会很好替代方案)。

但你到底想要做什么呢?

Validate

如果已经有public static bool Validate(int fieldID) { return MetadataManager.FieldRegistry.FirstOrDefault(f => f.ID == fieldID).ID > 1; } fieldID大于ID的条目,那么您的ID似乎有效吗?
好的,但如果您的列表中尚未包含1,则您的方法会抛出NullReferenceException。因此,您最好将该方法更改为以下内容:

fieldID