«Правильный» MVC - Должен ли я использовать конструктор или нет?

Question

Если у меня есть модель, которая работает непосредственно с моей базой данных (Лично с использованием ASP.NET w \ Entity Framework), следует ли использовать конструктор для установки переменной?

Пример:

public class songs 
{ 
    public IEnumerable<songs> allSongs {get; } 

    public songs() 
    { 
     using (var context = new entities()) 
     { 
      allSongs = context.songs.orderBy(n => n.name).toList(); 
     } 
    } 
} 

В.С.

public class songs 
{ 
    public IEnumerable<songs> allSongs 
    { 
     get 
     { 
      using (var context = new entities()) 
      { 
       allSongs = context.songs.orderBy(n => n.name).toList(); 
      } 
     } 
    } 

    public songs() {} 
} 

Технически оба являются правильными, но более правильно? Единственный раз, когда я могу думать о том, где есть только один правильный ответ, - если переменная, которую я устанавливаю, должна всегда обновляться или оставаться неизменной во время операции.

Answer 1

Вы не должны получать доступ к данным из класса домена.

Вы должны создать SongRepository, который взаимодействует с вашей базой данных. Вы вводите контроллер в этот репозиторий, и когда вам нужен список песен, вы просто ссылаетесь на этот репозиторий.

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

Второй подход лучше всего подходит для работы с репозиториями.

Пример компоновки:

public class SongController : Controller { 
private SongRepository _songRepository; 

public SongController(SongRepository repo) { 
    _songRepository = repo; 
} 

public ActionResult ShowSongs(){ 
    return View(_songRepository.GetAllSongs()); 
} 
} 

public class SongRepository { 
public IEnumerable<Song> GetAllSongs(){ 
    using (var context = new entities()) 
     { 
      allSongs = context.songs.orderBy(n => n.name).toList(); 
     } 
} 
} 

Answer 2

Что делать, если вы только хотите, чтобы получить 1 песню? Я уверен, что вы не хотите загружать все песни тогда.

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

Я рекомендую просмотр лучшие вопросы в entity-framework, entity-framework-4, entity-framework-5 и entity-framework-6 до принятия решения о вашей стратегии доступа к данным.