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

同意大卫的观点，因为这取决于您的应用程序的大小。

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

使用诸如 StructureMap 之类的东西，您可能会得到如下所示的内容：

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

翻译为：

当某些内容请求 PhotoGalleryContext 时，请为其提供一个 HTTP 范围的新 PhotoGalleryContext（在请求开始时创建，在结束）。

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

然后你的控制器看起来像这样：

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

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

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