我有以下课程:
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)
答案 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