Asp.net mvc EF4.1 DbContextとサービスレイヤ

Question

サービスレイヤでDbContextを公開するのは悪い習慣ですか？例については

：

ここ

private readonly IRepository<SkillLevels> _repository; 
private readonly IDatabaseFactory _databaseFactory; 
private readonly IUnitOfWork _unitOfWork; 

public SkillLevelService(IRepository<SkillLevels> repository, IDatabaseFactory databaseFactory, IUnitOfWork unitOfWork) 
{ 
    _repository = repository; 
    _databaseFactory = databaseFactory; 
    _unitOfWork = unitOfWork; 
} 

public void InsertSkillLevel(SkillLevels entity) 
{ 
    _repository.Insert(entity); 
    _unitOfWork.Commit(); 
} 

、IDatabaseFactoryは私のDbContextオブジェクトを返すことができます。 SkillLevels以外のエンティティが必要な場合は、このサービスクラスで使用できるように、私はここに公開しました。あなたが知っている場合は、データをフェッチするために複数のテーブルを選択する必要がある複雑なクエリの場合。

しかし、それが悪い練習をするかどうかはわかりません！

何か助けていただければ幸いです。ありがとう。

Answer 1

一般的なリポジトリを使用しています。単純なケースでは機能しますが、結合を行う必要があるかどうかを言ったように、このアプローチは機能しません。だから、1つの改良は、特定のレポ

public interface ISkillLevelsRepository : IRepository<SkillLevels> 
{ 
    //have specific methods to achieve what you want 
} 

別のアプローチは、あなたのサービスクラス内の複数のリポジトリを使用してFactoryを削除するだろうを作成することです。

private readonly IRepository<SkillLevels> _skillsRepository; 
private readonly IRepository<Student> _studentRepository; 
private readonly IUnitOfWork _unitOfWork; 

public SkillLevelService(IRepository<SkillLevels> skillsRepository, IRepository<Student> studentRepository, IUnitOfWork unitOfWork) 
{ 
    _skillsRepository= skillsRepository; 
    _studentRepository = studentRepository; 
    _unitOfWork = unitOfWork; 
} 

次に、必要な機能を実現する複数のreposにアクセスします。

ナビゲーション項目を参照するときにEFが結合を追加するため、ほとんどすべてのナビゲーション項目でモデルを正しく定義した場合、ジョインを行う必要はほとんどありません。

Answer 2

Webアプリケーションでは、リクエストごとに1つのDBContextを使用してBeginRequestで作成し、リクエスト内のすべての呼び出しに渡してEndRequestに配置するのが一般的です。

この例を参照One DbContext per request in ASP.NET MVC (without IOC container)