MVC3 Data Context Best Practice

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

согласен с Давидом, в том, что это зависит от размера вашего приложения.

Однако я бы предложил использовать зависимую инъекцию для управления соединением для вас.

Используя что-то вроде StructureMap, вы могли бы иметь что-то вроде этого:

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

что переводится:

Когда что-то просит PhotoGalleryContext, придать ему новый PhotoGalleryContext что HTTP-область (созданная при начале запроса, расположена в конце).

Таким образом, StructureMap автоматически откроет/закроет соединение для вас.

Тогда ваш контроллер может выглядеть следующим образом:

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

И в своих методах действий, _db будет готов пойти на вас.

Несколько строк кода, чтобы сэкономить много повторяющихся с использованием утверждений в вашем приложении.

Answer 2

Возможно, вы рассмотрели размер своей заявки, решив это. Я бы сказал, отделяя модель является наилучшей практикой, но для небольших приложений, может быть, вы просто должны обернуть PhotoGalleryContext в использовании блока:

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; 
}