Question

我正在使用带有Ioc SimpleIoc的MVVM灯进行wpf应用。

我实现了这样的存储库模式：

 public interface ICrud<T> where  T : class 
    {
        IEnumerable<T> GetAll();
        Task<IEnumerable<T>> AsyncGetAll(); 
        void AddNew(params T[] items);
        void Delete(params T[] items);
        void Update(params T[] items);
        void SaveOrUpdate(params T[] items);
    }

 public class Crud<T> : ICrud<T> where T : class 
    {

        public void AddNew(params T[] items)
        {
            using (var context = new DataEntities())
            {
                foreach (T item in items)
                {
                    context.Entry(item).State = System.Data.Entity.EntityState.Added;
                }
                context.SaveChanges();
            }
        }

        public void Delete(params T[] items)
        {
            using (var context = new DataEntities())
            {
                foreach (T item in items)
                {
                    context.Entry(item).State = System.Data.Entity.EntityState.Deleted;
                }
                context.SaveChanges();
            }
        }

        public void Update(params T[] items)
        {
            using (var context = new DataEntities())
            {
                foreach (T item in items)
                {
                    context.Entry(item).State = System.Data.Entity.EntityState.Modified;
                }
                context.SaveChanges ();
            }
        }


        public  void SaveOrUpdate(params T[] items)
        {
            using (var context = new DataEntities())
            {
                foreach (T item in items)
                {
                    try
                    {
                        context.Entry(item).State = System.Data.Entity.EntityState.Modified;
                         context.SaveChanges();
                    }
                    catch (Exception)
                    {
                        context.Entry(item).State = System.Data.Entity.EntityState.Added;
                         context.SaveChanges();
                    }

                }

            }
        }

        public IEnumerable<T> GetAll()
        {
            using (var context = new DataEntities())
            {
                DbSet<T> dbSet = context.Set<T>(); 
                return dbSet.AsEnumerable().ToList();
            }
        }


        public Task<IEnumerable<T>> AsyncGetAll()
        {
            return Task.Factory.StartNew(() =>
            {

                    var context = new DataEntities();
                    DbSet<T> dbSet = context.Set<T>();
                    return dbSet.AsEnumerable();

            });
        }
    }

在viewmodel定位器中，我注入了这样的依赖项：

 static ViewModelLocator()
        {
            ServiceLocator.SetLocatorProvider(() => SimpleIoc.Default);
            SimpleIoc.Default.Register<IDataService, DataService>();
             SimpleIoc.Default.Register<ICrud<student>, Crud<student>>();
            SimpleIoc.Default.Register<ICrud<prof>, Crud<prof>>();
//Add the other EF entities dependencies
        }

我的问题是，当我要执行crud操作时，我必须在调用之前实例化EF实体：

SimpleIoc.Default.GetInstance<ICrud<student>>().AddNew();

我需要知道：

如上所述的EF的实例，是否违反了依赖注入
如果是这样，我该如何解决这个问题？

Answer 1

你不应该注入实体。实体不是服务。使用依赖注入构建的对象图应仅由服务组成。任何包含运行时数据（实体，消息，DTO）的东西都应该使用方法调用通过构建的对象图传递。

请查看this和this回复以及this博客文章。

在单个类中混合数据和行为会使测试DI配置变得更加困难，并且难以应用横切关注点。但除此之外，将运行时数据（例如实体）注入服务的构造函数会导致模糊，因为不清楚要注入构造函数的确切实体。例如，某些ICustomerService取决于其构造函数中的Customer实体。我们应该在这里注入哪个实体，因为我们可能有数千个。虽然这可以通过在我们创建ICustomerService实现的位置（我们的组合根）中实现选择标准来解决，但这使得DI配置非常复杂，使得很难验证配置并导致业务逻辑在应用程序的一部分中，不应包含任何业务逻辑。

依赖注入和实体框架

1 个答案: