使用实体框架更新记录的更好方法答案

【问题标题】：Better way to update a record using Entity Framework使用实体框架更新记录的更好方法
【发布时间】：2012-09-27 07:32:09
【问题描述】：

我正在寻找一种更好的方法来使用实体框架更新记录。

下面是我现在使用的，其中 playerFromModel 是从 View 中获取的数据。

public bool UpdatePlayer(PlayerEntity playerFromModel)
{
    bool updateSuccessfull = false;

    using (var context = new PlayerEntities())
    {
        var player = context.Player.Where(m => m.emailAddress == playerFromModel.EmailAddress).FirstOrDefault();

        if (player != null)
        {
            player.emailAddress = playerFromModel.EmailAddress;
            player.first_name = playerFromModel.FirstName;
            player.last_name = playerFromModel.LastName;
            player.alt_email_id1 = playerFromModel.AlternateEmailAddress;
            player.street = playerFromModel.Street;
            player.city = playerFromModel.City;
            player.state = playerFromModel.State;
            player.zip = playerFromModel.Zip;
            player.country_code = playerFromModel.CountryCode;
            player.phone1 = playerFromModel.Phone;

            try
            {
                context.SaveChanges();
                updateSuccessfull = true;
            }
            catch
            {
                updateSuccessfull = false;
            }
        }
    }

    return updateSuccessfull;
}

如您所见，我必须手动映射每个字段，并采用我在所有地方都遵循的相同方法。

我在想肯定有更好的方法来做到这一点。

请指导我。谢谢。

【问题讨论】：

使用一些映射库，比如 automapper：automapper.codeplex.com
@MiBu 是的，我正在使用 AutoMapper，并使用它将 MVC 模型文件夹中定义的 Viewmodel PlayerViewModel 映射到我的 Repository Proj 中定义的 PlayerEntities
能否请您添加所有层级的示例类（播放器），以便我更好地理解它？！
所有层的所有类都具有相同的属性
@Gromer 太好了，从来不知道你可以那样做。谢谢！

标签： asp.net-mvc asp.net-mvc-3 entity-framework entity-framework-4

【解决方案1】：

您可以使用TryUpdateModel。例如

    bool updateSuccessfull = false;
    using (var context = new PlayerEntities())
    {
        var player = context.Player.Where(m => m.emailAddress == playerFromModel.EmailAddress).FirstOrDefault();

        if (player != null)
        {
             //Beware this will try to map anything it can. This can be dangerous
            if(TryUpdateModel(player)){
            try
            {
                context.SaveChanges();
                updateSuccessfull = true;
            }
            catch
            {
                updateSuccessfull = false;
            }
         }
        }
    }
    return updateSuccessfull;
}

我不会在这里看到Real example of TryUpdateModel, ASP .NET MVC 3。

您可以使用 Automapper 或 MiBu 说的类似的东西。创建 Player 的 ViewModel link is here，然后将其映射到实体

//Get a PlayerUpdate ViewModel     

       using (var context = new PlayerEntities())
    {
   // fetch the domain model that we want to update - BTW I'd use a repository pattern but that is another debate
        var player = context.Player.Where(m => m.emailAddress == playerFromModel.EmailAddress).FirstOrDefault();

        if (player != null)
        {

        // Use AutoMapper to update only the properties of this domain model
        // that are also part of the view model and leave the other properties unchanged
        AutoMapper.Map<UpdatePlayerViewModel , PlayerEntity>(viewModel, player);

           try
            {
                context.SaveChanges();
                updateSuccessfull = true;
            }
            catch
            {
                updateSuccessfull = false;
            }
    }

但是我认为更新实体属性是一件很重要的事情，不应该自动完成。其他人也有同样的感觉。创建 AutoMapper 的 Jimmy Bogard 似乎并不认为它需要 two way mapping（但他说这部分是因为他们为他们的要求构建了 AutoMapper）on Stackoverflow 的类似答案是说自己做。

这取决于您的应用程序的复杂性，但我会考虑使用命令模式将消息发送到包含您的属性的适当处理程序并让处理程序更新它们。如果它成功了，我们就做一件事，如果不成功，我们就做另一件事。 MVC in Action 中描述了相同的模式。看到类似的（这里是链接）[http://www.paulstovell.com/clean-aspnet-mvc-controllers]和这里is the second link

【讨论】：