net c# lock 数据访问层中的语句

Question

我看到一个代码，他们有这样的数据访问层：

public class CustomerDA{  

    private static readonly object _sync = new object();  
    private static readonly CustomerDA _mutex = new CustomerDA();  

    private CustomerDA(){  
    }

    public CustomerDA GetInstance(){    

        lock(_sync){      
            return _mutex;        
        }    
    }  

    public DataSet GetCustomers(){  
        //database SELECT
        //return a DataSet
    }  

    public int UpdateCustomer(some parameters){  

        //update some user
    }

}  


public class CustomerBO{  

    public DataSet GetCustomers(){  

        //some bussiness logic  
        return CustomerDA.GetInstance().GetCustomers();
    }
}

我正在使用它，但开始思考......“如果必须构建一个类似 facebook 的应用程序，其中有数十万并发用户？我会阻止每个用户做他的事情，直到前一个用户结束他的数据库东西？对于更新方法，当数据库引擎已经在数据库服务器级别管理并发时，在应用程序中锁定线程是否有用？

然后我开始考虑将锁移至 GetCustomers 和 UpdateCustomer 方法，但再想一想：“它到底有用吗？”

1 月 3 日编辑：

没关系，我错过了“GetInstance”方法中的“static”关键字。

另一件事：我的想法是，如果有另一个线程在同一个数据访问类中工作，则没有线程可以访问 _mutex 变量。我的意思是，我认为由于 _mutex 变量是从 lock 语句内部返回的，所以在“;”之前没有线程可以访问 _mutex在以下句子中达到：

return CustomerDA.GetInstance().GetCustomer();

在进行了一些跟踪之后，我意识到我做出了错误的假设。您能否确认我做出了错误的假设？

所以...我可以肯定地说我的数据访问层不需要任何锁定语句（即使在 INSERT、UPDATE、DELETE 上）并且我的 DataAccess 中的方法是静态方法还是实例方法都没有关系？

再次感谢...您的 cmets 对我非常有用

Answer 1

该代码中的锁定完全没有意义。它锁定了返回一个永远不会改变的值的代码，因此没有理由在那里锁定。代码中加锁的目的是使对象成为单例，但由于它没有使用延迟初始化，所以根本不需要加锁。

将数据访问层设为单例是一个非常糟糕的主意，这意味着一次只有一个线程可以访问数据库。这也意味着类中的方法必须使用锁来确保一次只有一个线程访问数据库，否则代码将无法正常工作。

相反，每个线程都应该获得自己的数据访问层实例，并拥有自己的数据库连接。这样，数据库就可以处理并发问题，而 theads 根本不需要做任何锁定。

Answer 2

在需要的地方设置您的锁，以便发生并发访问。在锁/关键部分中只放入真正需要的代码。

GetInstance 不应该是静态的？

下面的伪代码解释了 GetInstance 是如何运作的：

1. 锁定
2. rval = _mutex
3. 解锁
4. 返回 rval

_mutex 是只读的，指的是非空对象，所以不能更改，为什么要加锁？

如果您的数据库提供并发管理，但在您的程序中，您创建了两个线程在您自己的域中同时写入相同的数据，同时等待数据， 您的数据库如何提供帮助？