我看着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表达式爱好者的普遍做法吗?还是说,我不应该担心一个流氓的诡计特立独行?


当前回答

在我看来,这是对羔羊的滥用。

至于语法辉煌,我发现style=>"width:100%"平原混乱。特别是因为=>而不是=

其他回答

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

只是想提出我的观点(我是MvcContrib网格组件的作者)。

毫无疑问,这绝对是语言滥用。然而,我不会真的认为这是反直觉的-当你看一个属性的调用(style => "width:100%", @class => "foo") 我认为这是非常明显的(它当然不比匿名类型方法更糟糕)。从智能感知的角度来看,我同意这是相当不透明的。

对于那些感兴趣的人,它在MvcContrib中使用的一些背景信息…

我把它添加到网格作为个人偏好-我不喜欢使用匿名类型作为字典(有一个接受“object”的参数就像一个接受参数Func[]一样不透明)和字典集合初始化器相当冗长(我也不喜欢冗长的流畅接口,例如必须将多个调用链接到一个属性(“style”,“display:none”)。属性("class", "foo")等)

如果c#对字典字面量有一个不那么冗长的语法,那么我就不会在网格组件中包含这个语法了:)

我还想指出,在MvcContrib中使用这个是完全可选的-这些是扩展方法,包装重载,以字典代替。我认为重要的是,如果你提供了这样的方法,你也应该支持更“正常”的方法,例如与其他语言的互操作。

另外,有人提到了“反射开销”,我只是想指出,这种方法真的没有太多的开销——没有运行时反射或表达式编译(参见http://blog.bittercoder.com/PermaLink,guid,206e64d1-29ae-4362-874b-83f5b103727f.aspx)。

下面的问题是什么:

html.Attributes["style"] = "width:100%";

它的互操作性很差。例如,考虑这个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语言之间有良好的互操作。

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