反映参数名:滥用c# lambda表达式或语法辉煌?

我看着MvcContrib网格组件，我很着迷，但同时又被网格语法中使用的语法技巧所排斥:

.Attributes(style => "width:100%")

上面的语法将生成的HTML的样式属性设置为width:100%。如果你注意的话，'style'并没有被指定。它是从表达式中参数的名称推导出来的!我必须深入研究这个问题，找到“魔法”发生的地方:

Hash(params Func<object, TValue>[] hash)
{
    foreach (var func in hash)
    {
        Add(func.Method.GetParameters()[0].Name, func(null));
    }
}

因此，代码确实使用了正式的、编译时的参数名称来创建属性名称-值对的字典。生成的语法结构确实非常具有表现力，但同时也非常危险。

lambda表达式的一般用法允许替换所使用的名称而不产生副作用。我在一本书上看到一个例子，上面写着收藏。我知道我可以在我的代码集合中写。ForEach(log => fire。burn (log))它的意思是一样的。但是使用这里的MvcContrib Grid语法，我突然发现代码可以根据我为变量选择的名称主动查找并做出决定!

那么这是c# 3.5/4.0社区和lambda表达式爱好者的普遍做法吗?还是说，我不应该担心一个流氓的诡计特立独行?

当前回答

如果方法(func)名称选择得当，那么这是避免维护头痛的好方法(例如:添加一个新的func，但忘记将其添加到函数参数映射列表中)。当然，你需要对它进行大量的文档记录，你最好从该类函数的文档中自动生成参数的文档……

2009-11-11 21:18:30

其他回答

它的互操作性很差。例如，考虑这个c# - f#的例子

C#:

public class Class1
{
    public static void Foo(Func<object, string> f)
    {
        Console.WriteLine(f.Method.GetParameters()[0].Name);
    }
}

F#:

Class1.Foo(fun yadda -> "hello")

结果:

打印“arg”(不是“yadda”)。

因此，库设计者应该避免这些“滥用”，或者至少提供一个“标准”的重载(例如，将字符串名作为一个额外的参数)，如果他们想在。net语言之间有良好的互操作。

2009-11-11 21:10:27

这是一个有趣的方法。如果你约束右边的表达式为常数，那么你可以实现使用

Expression<Func<object, string>>

我认为这才是你真正想要的，而不是委托(你使用lambda来获得两边的名称) 参见下面的naive实现:

public static IDictionary<string, string> Hash(params Expression<Func<object, string>>[] hash) {
    Dictionary<string, string> values = new Dictionary<string,string>();
    foreach (var func in hash) {
        values[func.Parameters[0].Name] = ((ConstantExpression)func.Body).Value.ToString();
    }
    return values;
}

这甚至可以解决前面提到的跨语言互操作问题。

2009-12-03 11:27:43

我属于“语法卓越”阵营，如果他们清楚地记录它，它看起来非常酷，在我看来几乎没有问题!

2009-11-11 21:07:23

该代码非常聪明，但它可能会导致比它解决的问题更多的问题。

正如您所指出的，现在在参数名称(样式)和HTML属性之间存在模糊的依赖关系。没有编译时检查。如果参数名称输入错误，页面可能不会有运行时错误消息，但很难找到逻辑错误(没有错误，但行为不正确)。

更好的解决方案是拥有一个可以在编译时检查的数据成员。所以不要这样: