我有几个线程共享的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/

10-11 13:42