代码之家 › 专栏 › 技术社区 › dlras2

为事件处理程序使用lambda表达式的最佳实践

lambda events c#

dlras2 · 技术社区 · 15 年前

在发现lambda表达式并将其用作匿名函数之后,我发现自己编写了许多更琐碎的事件,例如:

txtLogin.GotFocus += (o, e) =>
{
    txtLogin.Text = string.Empty;
    txtLogin.ForeColor = SystemColors.ControlText;
};
txtLogin.LostFocus += (o, e) =>
{
    txtLogin.Text = "Login...";
    txtLogin.ForeColor = SystemColors.InactiveCaptionText;
};

我也不再使用只调用其他函数的事件处理程序,取而代之的是小lambda,它们也做同样的事情:

backgroundWorker.DoWork += (o, e) => DatabaseLookup.Open(e.Argument as string);

我发现了一些类似的问题 performance concerns can't remove them ,但我还没有找到任何解决这个简单问题的方法 这是个好主意吗?

以这种方式使用lambdas是一种好的形式,还是更多有经验的程序员会看不起这一点?它是将事件处理程序隐藏在难以找到的地方,还是通过减少琐碎事件处理程序的数量来编写服务代码?

3 回复 | 直到 8 年前

Jon Skeet 15 年前

这是一个非常合理的想法-但在这种情况下,我会使用匿名方法:

txtLogin.LostFocus += delegate
{
    txtLogin.Text = "Login...";
    txtLogin.ForeColor = SystemColors.InactiveCaptionText;
};

只有匿名方法相对于lambda表达式的优势。

做 IDisposable 这将删除订阅,如果你调用它。非常整洁。)

James Curran 15 年前

实际上,把事件处理程序放在容易找到的地方,也就是说,就在它被分配到的事件的名称旁边,是一种很好的方法。

很多时候,您会看到如下事件处理程序:

 void Text1_KeyDown(....) {....}

附加到txtFirstName的KeyUp事件,因为在使用Intellisense创建处理程序之后,有人决定重命名textbox,KeyUp工作得更好。通过Lambda,对象、事件和函数都在一起。

-1

Grant Crofton 15 年前

这是个棘手的问题。我记得我在图书馆看书 Code Complete 关于一些(聪明的)人如何说你应该保持控制流尽可能简单,许多人主张一个方法只有一个入口和出口,因为不这样做会使程序更难遵循。

兰博达离这一点越来越远,在某些情况下很难跟踪所发生的事情,控制权从一个地方跳到另一个地方。

基本上,我认为这可能是一个坏主意,因为这一点,但它也强大,使生活更容易。我当然用了不少。总之,小心使用!

推荐文章

SpaceCowboy74 Mohammad Aghazadeh · 使用Fluent Validation比较数组中的值

8 月前

Rico Strydom · Linq to XML:如何基于文件中的其他元素获取元素

8 月前

A B · C#Excel自动调整列避免长文本时出错

8 月前

Megrez7 · C#ToArray转换合并为一行,导致数组元素更改

8 月前

Alireza Noori · 全局配置用于本地化的MudDontext验证消息?

8 月前

Aycon · 在工厂方法中释放部分创建的对象的正确方法是什么?

8 月前

Martyn C · 自定义StringEnumConverter未拾取所有枚举

8 月前

Duck0 · 这个对象在更高的帧率下会更快吗,因为它在Update()中?

8 月前

Nwerx_user · 在C#中,如何在不使用static关键字的情况下使用类外的方法?

9 月前

Sei · Avalonia/WPF将路由器传递到控制模板

9 月前