Mapperly 中 MapProperty.Use 默认与非默认用户映射的潜在问题分析

Mapperly 是一个强大的对象映射代码生成工具，它通过属性注解自动生成高性能的映射代码。在实际使用过程中，开发者可能会遇到一些映射配置上的陷阱，特别是在同时使用默认映射和非默认映射时。

问题背景

当我们在 Mapperly 中同时配置默认映射方法和非默认映射方法时，如果错误地指定了映射方向，可能会产生不符合预期的链式映射代码。例如，在长度单位转换场景中：

// 模型类
public class Model
{
    public Length Length { get; set; }
    public Length Thickness { get; set; }
}

// DTO类
public class ModelDto
{
    public double LengthInMeter { get; set; }
    public double ThicknessInMillimeter { get; set; }
}

典型错误配置

开发者可能会这样配置映射器：

[Mapper]
public partial class DemoMapper
{
    // 正确的DTO映射
    [MapProperty("Length", "LengthInMeter", Use = nameof(ToMeters))]
    [MapProperty("Thickness", "ThicknessInMillimeter")]
    public partial ModelDto ToDto(Model source);

    // 错误的反向映射 - 使用了错误的转换方法
    [MapProperty("LengthInMeter", "Length", Use = nameof(ToMeters))]
    [MapProperty("ThicknessInMillimeter", "Thickness")]
    public partial Model ToModelWrong(ModelDto source);
}

问题现象

在上述配置中，ToModelWrong 方法错误地使用了 ToMeters 方法（本应使用 FromMeters）。由于 Mapperly 会自动尝试寻找兼容的映射路径，它会生成以下不符合逻辑的链式调用：

target.Length = FromMillimeters(ToMeters(FromMillimeters(source.LengthInMeter)));

这种映射实际上执行了三次转换：

1. 首先使用默认的 FromMillimeters 将 double 转换为 Length
2. 然后使用指定的 ToMeters 将 Length 转换回 double
3. 最后再次使用 FromMillimeters 将 double 转换回 Length

这显然不是开发者想要的结果，而且会导致性能损失和潜在的数据精度问题。

解决方案

Mapperly 团队已经在新版本中引入了编译时警告机制，当检测到 MapPropertyAttribute.Use 指定的映射方法存在类型不兼容时，会发出警告。开发者可以根据需要：

1. 将警告升级为错误（通过调整 Analyzer 规则严重性）
2. 忽略警告（如果不影响业务逻辑）
3. 修正映射配置（推荐做法）

正确的配置应该是：

[MapProperty("LengthInMeter", "Length", Use = nameof(FromMeters))]

最佳实践

1. 明确映射方向：确保使用的映射方法与实际需要的转换方向一致
2. 启用警告检查：建议将相关警告设置为错误级别，防止错误配置进入生产环境
3. 谨慎使用默认映射：当有特殊映射需求时，考虑禁用默认映射或明确指定所有映射
4. 单元测试验证：为映射逻辑编写单元测试，验证转换结果的正确性

通过遵循这些实践，可以避免因映射配置错误导致的数据转换问题，确保对象映射的正确性和可靠性。