MVC3 数据上下文最佳实践

Question

这是我在控制器中执行的操作。在此控制器内部创建和处理数据库连接（即 PhotoGalleryContext - 它是与 MySql 数据库的连接）而不是通过在模型的数据访问层中完成此操作来进行抽象是否被认为是不好的做法。 cs课？

    // GET: /Admin/GetPhoto/id
    public ActionResult GetPhoto(int id)
    {
        PhotoGalleryContext db = new PhotoGalleryContext();

        Models.PhotoGallery.Photo photo = new Models.PhotoGallery.Photo();
        photo = db.Photos.Where(p => p.PhotoId == id).Single();

        string filePath = photo.FileLocation;

        db.Dispose();    

        byte[] byteArray;
        try
        {
            byteArray = System.IO.File.ReadAllBytes(filePath);
            return File(byteArray, "image/jpg");
        }
        catch (Exception)
        {
            //throw;
        }
        return null;
    }

Answer 1

在决定这个时，您可能会考虑您的应用程序的大小。我想说分离模型是一种最佳实践，但对于较小的应用程序，也许您应该将 PhotoGalleryContext 包装在 using 块中：

using(PhotoGalleryContext db = new PhotoGalleryContext())
{

    Models.PhotoGallery.Photo photo = new Models.PhotoGallery.Photo();
    photo = db.Photos.Where(p => p.PhotoId == id).Single();

    string filePath = photo.FileLocation;
    byte[] byteArray;
    try
    {
        byteArray = System.IO.File.ReadAllBytes(filePath);
        return File(byteArray, "image/jpg");
    }
    catch (Exception)
    {
        //throw;
    }
    return null;
}

Answer 2

同意 David，这取决于您的应用程序的大小。

但是，我建议使用依赖注入来为您管理连接。

使用StructureMap 之类的东西，你可以有这样的东西：

For<PhotoGalleryContext>()
   .HybridHttpOrThreadLocalScoped
   .Use<PhotoGalleryContext>();

翻译为：

当有东西请求 PhotoGalleryContext 时，给它一个新的 PhotoGalleryContext，它是 HTTP-scoped（在请求开始时创建，在结束）。

这样，StructureMap 会自动为您打开/关闭连接。

那么你的控制器可以如下所示：

private readonly PhotoGalleryContext _db;
public AdminController(PhotoGalleryContext db)
{
   _db = db;
}

在您的操作方法中，_db 将准备好为您服务。

几行代码可以在您的应用程序中节省大量重复的 using 语句。