返回泄漏实现细节的委托

Question

我有一个名为 MapBuilder 的类，它内部使用 Dictionary 该类用于快速构建将被代理的属性的映射。 该类看起来像这样::

public class MapBuilder<T>{
    private Dictionary<PropertyInfo, string> m_Map = new Dictionary<PropertyInfo,string>();

    public MapBuilder<T> Add<TProperty>(Expression<Func<T, TProperty>> property){
        ArgumentValidator.AssertIsNotNull(()=>property);
        var propertyInfo = Reflect.Property<T>.InfoOf(property);
        m_Map.Add(propertyInfo, propertyInfo.Name);
        return this;
    }

    public MapBuilder<T> Add<TProperty>(Expression<Func<T, TProperty>> property,string columnName){
        ArgumentValidator.AssertIsNotNull(() => property);
        ArgumentValidator.AssertIsNotNull(() => columnName);
        var propertyInfo = Reflect.Property<T>.InfoOf(property);
        m_Map.Add(propertyInfo, columnName);
        return this;
    }

    public Map Compile(){
        return m_Map.TryGetValue;
    }

所以用户会像这样使用它::

 var map= new MapBuilder<MyClass>()
.Add(x => x.Name)
.Add(x => x.Id)
.Add(x => x.Active)
.Compile()

这将构建一个封装了 Name、Id、Active 3 个属性的映射。问题是 Map 委托现在可以向最终用户泄漏实现细节，因为他们可以观察到该方法是 Dictionary并且目标将是私有字典。您会认为这是代码味道吗？

我可以将其包装在匿名方法中，但当可以进行方法组转换时，我倾向于认为这种形式很糟糕。

Answer 1

查看 Map 委托的目标和方法所涉及的工作量与反映 MapBuilder 类本身的字段的工作量大致相同；无论哪种方式，调用者都可以发现私有 Dictionary 实例。

从 MapBuilder 类的角度来看，我不会担心它。反思私有字段肯定会产生代码味道：但这不是你的责任，而是你的类的用户的责任。