我有一些与下面类似的代码,请参阅 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 丑陋得多。还有一个更大的风险是 future 的开发人员会忘记所需的额外调用。
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);
}
有更好的解决方案吗?
最佳答案
问题是 Automapper AfterMap
每个映射运行一次。您的映射配置是:
Mapper.CreateMap<Foo, Bar>();
因此,如果您将 AfterMap 扩展附加到此映射,它将在 Foo 和 Bar 之间的每个映射上运行它。这就是为什么您会看到它运行不止一次。
如果只想运行一次,则应将其附加到 List 到 List 映射配置,而不是项到项配置。但是 AutoMapper 不够灵活,无法轻松与 List to List 配置一起使用。
使其工作的一种方法是使用
ConvertUsing
方法并明确指定要在列表项上使用的映射,并在那里调用 after 映射内容: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;
});
关于c# - AutoMapper AfterMap,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/28278306/