我有几个线程共享的singleton类。为了防止多个访问问题,我在访问类的一个或另一个属性时使用lock方法。问题是,当在代码中访问类属性时,是否有可能改进代码并将lock方法放在singleton类中,而不是每次都将其放在singleton类中?
/* Class code*/
public class ServerStatus
{
private static ServerStatus _instance;
public static ServerStatus Instance
{
get { return _instance ?? (_instance = new ServerStatus()); }
set { _instance = value; }
}
ServerStatus()
{
PistonCount = 0;
PistonQueue = new List<string>();
ErrorList = new List<string>();
}
public int PistonCount { get; set; }
public List<string> PistonQueue { get; set; }
public List<string> ErrorList { get; set; }
}
/*Code for accessing class properties*/
private static readonly object Locker = new object();
/*Skip*/
lock (Locker)
{
ServerStatus.Instance.PistonQueue.Add(e.FullPath);
}
/*Skip*/
lock (Locker)
{
ServerStatus.Instance.PistonCount++;
}
最佳答案
ServerStatus
应该保持自己的同步,而不是此类的外部客户端。也就是说,您需要重构ServerStatus
并创建一些线程安全(带锁)方法:
删除这些属性:public List<string> PistonQueue { get; set; }
,因为即使您可以锁定这些属性,也无法控制客户端在获得实际的PistonQueue
后执行的操作。
…并替换为方法,例如(抱歉的伪代码,我今天懒得想):
public PistonQueueAdd(string fullPath)
{
lock(_serverStatusSyncRoot)
{
// ...
}
}
关于c# - 多线程应用中的Singleton类,锁定建议,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/6439743/