可以在C#构造函数中放入数据库初始化调用吗？

Question

我已经看到这是各种代码库,并想知道这是否通常不赞成.

例如:

public class MyClass
{
   public int Id;

   public MyClass()
   {
      Id = new Database().GetIdFor(typeof(MyClass));
   }
}

Answer 1

嗯..我不会.但是,我的方法通常还涉及类不负责检索自己的数据.

Answer 2

有几个原因通常不被认为是良好的设计,其中一些例如导致难以进行单元测试并且已经提到了处理错误的难度.

我选择不这样做的主要原因是你的对象和数据访问层现在非常紧密耦合,这意味着在原始设计之外对该对象的任何使用都需要大量的返工.举个例子,如果遇到一个需要使用该对象的实例而没有分配任何值来保存该类的新实例？你现在要么重载构造函数,然后确保所有其他逻辑处理这个新案例,或者继承和覆盖.

如果对象和数据访问被解耦,那么您可以创建一个实例,然后不进行水合.或者,如果您的项目使用相同的实体但使用不同的持久层,则对象可以重复使用.

说过去我在项目中采用了更简单的耦合方式:)

Answer 3

它还会使为类编写单元测试变得困难,因为您将无法强制类使用db类的Mock/Stub版本.见这里:http: //en.wikipedia.org/wiki/Dependency_injection

Answer 4

如果您参考数据库连接,则可以使用一次性模式:

public class MyClass : IDisposable
{
    private Database db;
    private int? _id;

    public MyClass()
    {
        db = new Database();
    }

    public int Id
    {
        get
        {
            if (_id == null) _id = db.GetIdFor(typeof(MyClass));
            return _id.Value;
        }
    }

    public void Dispose()
    {
        db.Close();
    }
}

用法:

using (var x = new MyClass()) 
{
    /* ... */

} //closes DB by calling IDisposable.Dispose() when going out of "using" scope