ASP.net MVC 控制器 - 构造函数的使用

Question

我正在开发 ASP.net MVC 应用程序，并且有一个关于为控制器使用构造函数的问题。

我正在使用 Entity Framework 和 linq to Entities 来处理所有数据事务。我需要访问我的实体模型来执行几乎所有控制器操作。当我第一次开始编写应用程序时，我在每个 Action 方法的开头创建一个实体对象，执行我需要的任何工作，然后返回结果。

我意识到我正在为每个操作方法一遍又一遍地创建相同的对象，因此我为实体对象创建了一个私有成员变量，并开始在每个控制器的构造函数中实例化它。现在，每个方法仅引用该私有成员变量来完成其工作。

我还在问自己哪条路是对的。我想知道 A.) 哪种方法最合适？ B.) 在构造函数方法中，这些对象的存活时间是多长？ C.) 构造函数方法是否存在性能/完整性问题？

Answer 1

你问的是正确的问题。

答：在每个操作方法中创建这种依赖关系绝对是不合适的。 MVC 的主要功能之一是能够分离关注点。通过加载具有这些依赖项的控制器，您可以使控制器变得更厚。这些应该被注入到控制器中。依赖注入 (DI) 有多种选项。通常，这些类型的对象可以注入构造函数或属性中。我的偏好是构造函数注入。

B. 这些对象的生命周期将由垃圾收集器决定。 GC 不是确定性的。因此，如果您的对象与资源受限服务（数据库连接）有连接，那么您可能需要确保自己关闭这些连接（而不是依赖于 dispose）。很多时候，“生命周期”问题被分离到控制反转 (IOC) 容器中。那里有很多。我最喜欢的是 Ninject。

C. 实例化成本可能是最小的。数据库事务成本是您可能需要关注的地方。您可能需要研究一个称为“工作单元”的概念。本质上，数据库可以处理比一次保存/更新操作更大的事务。增加事务大小可以带来更好的数据库性能。

希望这能让你开始。

Answer 2

RRavens 有一些出色的见解。我想向您展示如何实施他的建议。

最好首先为要实现的数据访问类定义一个接口：

public interface IPostRepository 
{
    IEnumerable<Post> GetMostRecentPosts(int blogId);
}

然后实现一个数据类。实体框架上下文的构建成本很低，并且当您不处理它们时，您可能会得到不一致的行为，因此我发现通常最好将所需的数据提取到内存中，然后处理上下文。

public class PostRepository : IPostRepository
{
    public IEnumerable<Post> GetMostRecentPosts(int blogId)
    {
        // A using statement makes sure the context is disposed quickly.
        using(var context = new BlogContext())
        {
            return context.Posts
                .Where(p => p.UserId == userId)
                .OrderByDescending(p => p.TimeStamp)
                .Take(10)
                // ToList ensures the values are in memory before disposing the context
                .ToList(); 
        }
    }
}

现在您的控制器可以接受这些存储库之一作为构造函数参数：

public class BlogController : Controller
{
    private IPostRepository _postRepository;
    public BlogController(IPostRepository postRepository)
    {
        _postRepository = postRepository;
    }

    public ActionResult Index(int blogId)
    {
        var posts = _postRepository.GetMostRecentPosts(blogId);
        var model = new PostsModel { Posts = posts };
        if(!posts.Any()) {model.Message = "This blog doesn't have any posts yet";}
        return View("Posts", model);
    }

}

MVC 允许您使用自己的控制器工厂代替默认的控制器工厂，因此您可以指定您的 IoC 框架（如 Ninject）决定如何创建控制器。您可以设置注入框架，以了解当您请求 IPostRepository 时，它应该创建一个 PostRepository 对象。

这种方法的一大优点是它使您的控制器可进行单元测试。例如，如果您想确保模型在没有帖子时收到消息，您可以使用 Moq 之类的模拟框架来设置存储库不返回帖子的场景：

var repositoryMock = new Mock<IPostRepository>();
repositoryMock.Setup(r => r.GetMostRecentPosts(1))
    .Returns(Enumerable.Empty<Post>());
var controller = new BlogController(repositoryMock.Object);
var result = (ViewResult)controller.Index(1);
Assert.IsFalse(string.IsNullOrEmpty(result.Model.Message));

这使得测试特定行为变得容易您期望控制器执行操作，而无需设置数据库或任何类似的特殊操作。像这样的单元测试很容易编写、具有确定性（它们的通过/失败状态基于代码，而不是数据库内容）并且速度快（您通常可以在一秒钟内运行一千个这样的测试）。