4

我的 SurveyController 类中有这个方法：

public ActionResult AddProperties(int id, int[] propertyids, int page = 1)
{
    var survey = _uow.SurveyRepository.Find(id);
    if (propertyids == null)
        return GetPropertiesTable(survey, page);

    var repo = _uow.PropertySurveyRepository;

    propertyids.Select(propertyid => new PropertySurvey
                {
                    //Setting the Property rather than the PropertyID 
                    //prevents the error occurring later
                    //Property = _uow.PropertyRepository.Find(propertyid),
                    PropertyID = propertyid,
                    SurveyID = id
                })
                .ForEach(x => repo.InsertOrUpdate(x));
    _uow.Save();

    return GetPropertiesTable(survey, page);
}

GetPropertiesTable 重新显示属性，但 PropertySurvey.Property 被标记为虚拟，我使用 new 运算符创建了实体，因此从未创建支持延迟加载的代理，当我访问它时它为空。当我们可以直接访问 DbContext 时，我们可以使用 Create 方法显式创建代理。但是我在这里有一个工作单元和存储库模式。我想我可以通过 repository.Create 方法公开 context.Create 方法，然后我需要记住在添加 entity 时使用它而不是 new 运算符。但是将问题封装在我的 InsertOrUpdate 方法中不是更好吗？是否有某种方法可以检测到被添加的实体不是代理时应该是代理并替换代理？这是我的基础存储库类中的 InsertOrUpdate 方法：

    protected virtual void InsertOrUpdate(T e, int id)
    {
        if (id == default(int))
        {
            // New entity
            context.Set<T>().Add(e);
        }
        else
        {
            // Existing entity
            context.Entry(e).State = EntityState.Modified;
        }
    }
4

2 回答 2

4

基于qujck提供的答案。以下是无需使用自动映射器的方法：

编辑为始终检查代理 - 不仅仅是在插入期间 - 正如评论中所建议的那样

再次编辑以使用不同的方式检查代理是否已传递给该方法。更改技术的原因是我在引入从另一个继承的实体时遇到了问题。entity.e.GetType().Equals(instance.GetType()在这种情况下，即使继承实体是代理，也可能无法通过检查。我从这个答案中得到了新技术

public virtual T InsertOrUpdate(T e)
{
    DbSet<T> dbSet = Context.Set<T>();

    DbEntityEntry<T> entry;
    if (e.GetType().BaseType != null 
        && e.GetType().Namespace == "System.Data.Entity.DynamicProxies")
    {
        //The entity being added is already a proxy type that supports lazy 
        //loading - just get the context entry
        entry = Context.Entry(e);
    }
    else
    {
        //The entity being added has been created using the "new" operator. 
        //Generate a proxy type to support lazy loading  and attach it
        T instance = dbSet.Create();
        instance.ID = e.ID;
        entry = Context.Entry(instance);
        dbSet.Attach(instance);

        //and set it's values to those of the entity
        entry.CurrentValues.SetValues(e);
        e = instance;
    }

    entry.State = e.ID == default(int) ?
                            EntityState.Added :
                            EntityState.Modified;

    return e;
}

public abstract class ModelBase
{
    public int ID { get; set; }
}
于 2013-05-29T10:53:48.033 回答
1

我同意你的观点，这应该在一个地方处理，最好的地方就是你的存储库。您可以将 的类型T与上下文创建的实例进行比较，如果类型不匹配，则可以使用Automapper之类的工具快速传输所有值。

private bool mapCreated = false;

protected virtual void InsertOrUpdate(T e, int id)
{
    T instance = context.Set<T>().Create();
    if (e.GetType().Equals(instance.GetType()))
        instance = e;
    else
    {
        //this bit should really be managed somewhere else
        if (!mapCreated)
        {
            Mapper.CreateMap(e.GetType(), instance.GetType());
            mapCreated = true;
        }
        instance = Mapper.Map(e, instance);
    }

    if (id == default(int))
        context.Set<T>().Add(instance);
    else
        context.Entry(instance).State = EntityState.Modified;
}
于 2013-05-24T14:04:21.783 回答