Mapperly 中关于空值映射的优化解析

问题背景

在对象映射工具 Mapperly 的使用过程中，开发者遇到了一个关于空值处理的特殊情况。当从可空类型映射到非空类型时，如果开发者指定了自定义映射函数，Mapperly 生成的代码会先进行空值检查，然后再调用映射函数。然而，这种处理方式在某些场景下并不理想。

典型场景分析

假设我们有以下两个类需要进行映射：

public class Source
{
    public string? Content { get; set; }
}

public class Target
{
    public string Content { get; set; }
}

当使用 Mapperly 默认映射时，生成的代码会包含空值检查逻辑：

if (source.Content != null)
{
    target.Content = source.Content;
}

自定义映射函数的使用

开发者可能希望提供一个自定义的映射函数来处理空值情况：

private static string MapMyString(string? content) => content ?? "";

当全局使用这个映射函数时，Mapperly 能正确生成代码：

target.Content = MapMyString(source.Content);

问题出现

然而，当开发者尝试通过 MapProperty 特性仅对特定属性应用这个映射函数时：

[MapProperty(nameof(Source.Content), nameof(Target.Content), Use = nameof(MapMyString))]

Mapperly 生成的代码却变成了：

if (source.Content != null)
{
    target.Content = MapMyString(source.Content);
}

技术分析

这种双重空值检查的行为实际上是不必要的，因为：

1. 自定义映射函数 MapMyString 已经明确声明可以接受空值参数（string?）
2. 函数内部已经包含了空值处理逻辑（?? ""）
3. 外部的空值检查会导致空值根本不会进入映射函数

解决方案

Mapperly 团队确认这是一个 bug，并在 3.5.1 版本中修复了这个问题。修复后的行为是：

当满足以下条件时，Mapperly 将直接调用映射函数而不进行前置空值检查：

1. 目标属性是非空类型
2. 显式指定了自定义映射函数
3. 映射函数的参数类型可以接受空值

最佳实践建议

1. 当处理可空到非空的映射时，优先考虑在映射函数内部处理空值情况
2. 确保映射函数的参数类型与实际需要处理的类型匹配
3. 对于简单的空值替换场景，可以考虑使用 Mapperly 的内置空值处理特性

这个改进使得 Mapperly 在自定义映射场景下的行为更加符合开发者预期，减少了不必要的代码生成，提高了映射效率。