AutoMapper 给出无效操作异常

Question

在我的编辑页面上，我收到了这个错误：

附加一个类型的实体失败，因为另一个相同类型的实体已经具有相同的主键值。

所以，我对此进行了研究并来到this。

这让我想使用 Auto-Mapper 来简化事情。

这是我的代码：

if (db.TableName.Find(value.ID).stringProperty.Equals(value.stringProperty, StringComparison.CurrentCultureIgnoreCase))
{
    Table inContextVariable = db.TableName.Find(value.ID);

    Mapper.Initialize(config => config.CreateMap<ModelName, ModelName>());

    Mapper.Map<ModelName, ModelName>(value, inContextVariable);

    db.Entry(inContextVariable).State = EntityState.Modified;
    db.SaveChanges();
    return RedirectToAction("Index");
}

这导致我出现此错误：

附加信息：操作失败：无法更改关系，因为一个或多个外键属性不可为空。当对关系进行更改时，相关的外键属性将设置为空值。如果外键不支持空值，则必须定义新关系，必须为外键属性分配另一个非空值，或者必须删除不相关的对象。

这个表有 1 个外键，当我调试时外键的值为 11，所以我很难理解这个错误。

---

更新

---

想象一下，如果我有一个数据库表，其中包含包含某人信息的记录，例如：

|    ID    |    First Name    |    Last Name    |          Email           |
----------------------------------------------------------------------------
     1           John                Doe             john.doe@test.com
     2           Christopher       Columbus     |  chris.columbus@test.com |

等等等等..现在，由于一个人只能有一个电子邮件地址，因此您必须在其他用户创建/编辑记录时采取保护措施..

例如，如果我要使用此应用程序并想创建一个帐户。我应该无法使用数据库表中已存在的电子邮件地址成功填写表单。我应该收到一条错误消息说类似

该电子邮件地址已经存在！请输入另一个电子邮件地址

代码如下：

if (db.TableName.Any(x => x.Email.Equals(value.Email, StringComparison.CurrentCultureIgnoreCase)))
{
    ModelState.AddModelError("Email", "This email already exists!");
    return View(value);
}

现在....当用户成功创建一个帐户..然后去编辑他们的帐户电子邮件地址到一个已经存在的东西时会发生什么？应该会出现相同的错误消息。

但是当该用户创建他们的帐户时，他们的电子邮件地址现在在数据库表中。所以当他们进入编辑页面时，使用他们的原始电子邮件，他们点击“保存”，代码将根据数据库中的所有电子邮件地址检查该电子邮件，它会看到它已经在数据库中，导致显示错误消息（请更改您的电子邮件。..等）..

因此，我需要尝试创建一种有效的方法来检查何时用户在 edit 页面上，并使用他们的原始页面点击“保存”电子邮件，它不会告诉他们更改电子邮件。

所以我有这个代码：

if (db.TableName.Find(value.ID).Email.Equals(value.Email, StringComparison.CurrentCultureIgnoreCase))
{
    var inContextVariable = db.TableName.Find(value.ID);
    MappingMethods.MapModelName(inContextVariable , value);

    \\ without the mapping.. I receive the error saying 'Attaching an entity of type failed because another entity of the same type already has the same primary key value.'

    db.Entry(inContextVariable).State = EntityState.Modified;
    db.SaveChanges();
    return RedirectToAction("Index");
}

Answer 1

这是我将在这里使用的基本逻辑

Table inContextVariable = db.TableName.Find(value.ID);
if(!inContextVariable.Email.Equals(value.Email, StringComparison.CurrentCultureIgnoreCase))
{
    // user input a new email so make sure it doesn't match anyone else
    if(db.TableName.Any(x => x.Email.Equals(value.Email, StringComparison.CurrentCultureIgnoreCase) && x.ID != value.ID))
    {
        // user's new email DOES match someone else's email so handle it
    }
    else
    {
        // no match so do the update

    }

}

无需处理外部 else，因为他们没有更改电子邮件，因此您无需执行任何操作！

注意：您将在此处出现竞态条件！如果两个人几乎同时提交，那么支票可能会通过，但他们最终都会收到相同的电子邮件。就像假设他们都更改为 x@y.com 但第二个人的检查是在第一个人的保存之前完成的。但是，这就是网络应用的生活。

Answer 2

在@nurdyguy 的帮助下，我提出了这个想法。我创建了另一个名为 MappingMethods 的类，它包含此方法：

public static void MapModelName(ModelName inContext, ModelName outOfContext)
{
    inContext.ID = outOfContext.ID;
    inContext.stringProperty= outOfContext.stringProperty;
    inContext.OwnerID = outOfContext.OwnerID; \\ foreign key
    inContext.DateCreated = outOfContext.DateCreated;
}

然后在我的编辑操作中：

if (db.TableName.Find(value.ID).stringProperty.Equals(value.stringProperty, StringComparison.CurrentCultureIgnoreCase))
{
    var inContextVariable = db.TableName.Find(value.ID);
    MappingMethods.MapModelName(inContextVariable , value);

    db.Entry(inContextVariable).State = EntityState.Modified;
    db.SaveChanges();
    return RedirectToAction("Index");
}

这行得通。

Answer 3

automapper 的整个想法是避免在静态类中使用此类方法。想象一下，如果你有 100 个属性？ （仅举例）。你会复制粘贴obj1.prop1 = obj2.prop1?

首先我建议创建一个 view-model - smth。这将代表您的模型的视图。是的，您可以使用自动映射器创建对象的副本，但 它不是为它设计的。

试着像这样做：

1. 创建一个视图模型 - 它可以与您的模型相同或不同 - 这完全取决于您（将数据库模型传递到视图中不是一个好习惯）

2. 将您的模型映射到 view-model 和 virce-verce，如下所示：

   Mapper.CreateMap() .ForMember(d => d.DateCreate, d => d.MapFrom(x => x.DateCreated));