AutoMapper 后图

Question

我有一些代码看起来很像下面，请参阅 https://dotnetfiddle.net/wuE81t 中的工作示例。

public class Program
{
    public static void Main()
    {
        Mapper.CreateMap<Foo, Bar>()
            .AfterMap((s, d) => {
                var stuff = SomeController.GetStuff(DateTime.Now.Second);
                d.Stuff = stuff.Contains(s.Name);
            });

        var foo = new List<Foo>() {
            new Foo() { Name = "joe", Age = 10 },
            new Foo() { Name = "jane", Age = 20 },
        };

        var bar = Mapper.Map<List<Foo>, List<Bar>>(foo);
    }
}

public class Foo
{
      public string Name { get; set; }
      public int Age { get; set; }
}

public class Bar
{
      public string Name { get; set; }
      public int Age { get; set; }
      public bool Stuff { get; set; }
}

public static class SomeController
{
    public static List<string> GetStuff(int currentUserId)
    {
        return new List<string>() { "jane" };
    }
}

我遇到的问题是为源列表中的每个项目调用 GetStuff，这是一个相当繁重的操作，所以我想通过只调用一次来优化它。在我的实际代码中，GetStuff 使用了 currentUserId 参数。

我目前通过将 GetStuff 移动到 Mapper.Map 之后解决了这个问题，但是由于我们有很多地方调用它，所以它比使用 AfterMap 更难看。还有一个更大的风险，即未来的开发人员会忘记所需的额外调用。

public static void Main()
{
    Mapper.CreateMap<Foo, Bar>();

    var foo = new List<Foo>() {
        new Foo() { Name = "joe", Age = 10 },
        new Foo() { Name = "jane", Age = 20 },
    };

    var bar = Mapper.Map<List<Foo>, List<Bar>>(foo);
    AddStuff(bar); // Required extra call!

    bar.Dump();
}

private static void AddStuff(List<Bar> bar)
{
    var stuff = SomeController.GetStuff(DateTime.Now.Second);
    foreach(var b in bar)
        b.Stuff = stuff.Contains(b.Name);       
}

有更好的解决方案吗？

Answer 1

问题是 Automapper AfterMap 运行每个映射一次。您的映射配置是：

 Mapper.CreateMap<Foo, Bar>();

因此，如果您将 AfterMap 扩展附加到此映射，它将运行在 Foo 和 Bar 之间的每个映射上。这就是为什么您不止一次看到它运行。

如果您只想运行它一次，您应该将它附加到列表到列表映射配置，而不是项目到项目配置。然而，AutoMapper 不够灵活，无法轻松使用列表到列表配置。

使其工作的一种方法是使用 ConvertUsing 方法并明确指定要在列表项上使用的映射并在那里调用映射后的内容：

Mapper.CreateMap<Foo, Bar>();

Mapper.CreateMap<List<Foo>, List<Bar>>()
    .ConvertUsing(source =>
    {
        var mapped = source.Select(Mapper.Map<Foo, Bar>).ToList();

        // After mapping code;
        var stuff = SomeController.GetStuff(DateTime.Now.Second);

        return mapped;
    });

AutoMapper 后图

AutoMapper AfterMap

c#

automapper