ASP.net MVC - dois-je utiliser AutoMapper de ViewModel pour Entity Framework entités?

Je suis actuellement en utilisant AutoMapper à la carte de mon Entity Framework entités à mon avis, Modèle:

public class ProductsController : Controller
{
    private IProductRepository productRepository;

    public ProductsController(IProductRepository productRepository)
    {
         this.productRepository = productRepository;
    }

    public ActionResult Details(int id)
    {
        var product = productRepository.GetProduct(id);

        if( product == null )
            return View("NotFound");

        ProductDetailsViewModel model = Mapper.Map<Product, ProductDetailsViewModel>(product);

        return View(model);
    }
}

Cela fonctionne bien. La question que j'ai, c'est quand j'en ai besoin pour aller de mon point de Vue le Modèle de l'entité, afin de mettre à jour la base de données. Dois-je utiliser AutoMapper pour cela? Est-ce une mauvaise/pratique dangereuse?

Il semble que AutoMapper est bon pour l'aplatissement d'un type complexe pour un simple (plat) type, mais jusqu'à présent, j'ai du mal en essayant de passer d'un plat/simple à la plus complexe de type comme mon entité avec les différentes propriétés de navigation.

Si c'est une mauvaise idée d'utiliser AutoMapper pour ce faire, alors ce serait mon code pour ressembler à une action de création?

public ActionResult Create(CreateProductViewModel model)
{
    if( ModelState.IsValid )
    {
        //what do i do here to create my Product entity?
    }
}

Ce sujet d'une action d'Édition?

public ActionResult Edit(int id, EditProductViewModel model)
{
    Product product = productRepository.GetProduct(id);

    //how do i convert my view model to my entity at this point???
}
InformationsquelleAutor Dismissile | 2011-09-28
  1. 28

    Je suis de l'état d'esprit que la mise à jour de vos entités est un assez grosse affaire et qu'aucun outil automatisé devrait jamais être utilisé. Définissez les propriétés manuellement.

    Oui ses une très petite quantité de plus de code mais automapper ou de l'exécution updatemodel sur les entités de base de données peut parfois avoir des conséquences inattendues. Mieux assurez-vous que votre écriture est fait correctement.

    • Je suis à l'aise avec cela, mais ce genre de fait mon contrôleur au-delà de laid. Je suis de l'état d'esprit que le contrôleur devrait être aussi simple que possible (quelques lignes). Dois-je tout simplement le sucer? 🙂
    • +1. Automapper et ses semblables sont grands pour l'aplatissement des entités à viewmodel, mais vous avez besoin de mettre un peu plus de réflexion dans la mise à jour des entités.
    • il ne faut pas mettre le code du contrôleur. Créer une classe responsable de la conversion entre les viewmodels et entités, et de l'appeler à partir du contrôleur. Facilite les tests et adhère à la SRP.
    • Vous savez que vous pouvez personnaliser les mappages de faire exactement ce que vous voulez qu'il fasse pour chaque propriété. Je ne vois pas vraiment la différence entre cela et l'écriture de vos propres instructions d'affectation. Pour la simple propriété à la propriété mappages je pense qu'il fonctionne bien.
    • +1 pour jfar. Dire de ne pas demander. À l'aide de automapper est pour vous demander si vous me demandez. Aussi garder aussi simple que possible et de coup de pied comme de nombreuses dépendances de cadre que possible.
    • C'est plus de travail que les secondes qu'il faut pour écrire Model.Property = ViewModel.Property
    • À l'aide de AutoMapper met extrêmement couche opaque entre votre domaine et des couches de présentation. "Visual Studio: renommer Domain.Entity.Property". Félicitations: votre point de vue les modèles sont désormais rompu à cause de leur utilisation implicite de Property.
    • Utiliser une méthode d'extension pour convertir votre modèle de domaine (entité) pour votre modèle de vue. PRO ASTUCE: si vous êtes en utilisant linq to requête, créer votre méthode d'extension sur le IQueryable<T>

    InformationsquelleAutor John Farrell
  2. 12

    - Je utiliser AutoMapper spécialisés dans la cartographie de la classe qui comprend la façon de faire d'un modèle complexe à partir d'un simple. AutoMapper est utilisé pour traiter le one-to-one mapping et la logique personnalisée dans la classe pour faire des choses plus complexes (comme les relations, etc.). Tous les AutoMapper de la configuration se fait dans le constructeur statique pour le mappage de classe, qui valide également la configuration du mappage de sorte que les erreurs de l'échec précoce.

    public class ModelMapper
{
    static ModelMapper()
    {
        Mapper.CreateMap<FooView,Foo>()
              .ForMember( f => f.Bars, opt => opt.Ignore() );

        Mapper.AssertConfigurationIsValid();
    }

    public Foo CreateFromModel( FooView model, IEnumerable<Bar> bars )
    {
         var foo = Mapper.Map<FooView,Foo>();
         foreach (var barId in model.BarIds)
         {
             foo.Bars.Add( bars.Single( b => b.Id == barId ) );
         }
         return foo;
    }
}
    InformationsquelleAutor tvanfosson
  3. 2

    Vous pouvez également essayer de configuration AutoMapper à la carte uniquement des propriétés Scalaires (au lieu d'avoir à .Ignore() chaque propriété que vous ne voulez pas qu'elle (y compris les propriétés héritées comme .EntityKey et .EntityState).

    AutoMapper.Mapper.CreateMap<EntityType, EntityType>()
    .ForAllMembers(o => {
         o.Condition(ctx =>
             {
                 var members = ctx.Parent.SourceType.GetMember(ctx.MemberName); //get the MemberInfo that we are mapping

                if (!members.Any())
                    return false;
                return members.First().GetCustomAttributes(typeof(EdmScalarPropertyAttribute), false).Any(); //determine if the Member has the EdmScalar attribute set
            });
    });

    Quelques infos à http://www.prosoftnearshore.com/blog/post/2012/03/14/Using-AutoMapper-to-update-Entity-Framework-properties.aspx

    InformationsquelleAutor Mauricio Morales
  4. 1

    Essentiellement mappant automatiquement est mauvais, j'ai écrit un billet de blog sur ce http://blog.gavryli.uk/2015/12/02/why-automapping-is-bad-for-you/

    • Le lien dans la réponse est cassé, le nouveau semble être ivanazure.wordpress.com/2015/12/02/... Pourquoi tous les downvotes? L'article donne beaucoup d'arguments contre mappant automatiquement.
    • excellent point de vue alternatif sur la downsites de mappant automatiquement, ne vous inquiétez pas au sujet de la downvotes. Je suggère d'inclure certains de vos points dans votre réponse.
    InformationsquelleAutor Ivan G.