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

Question

在发现 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，但我没有找到任何针对这是个好主意吗？这个简单问题的解决方案p>

以这种方式使用 lambdas 是否被认为是好的形式，还是更有经验的程序员看不起这一点？它是把事件处理程序隐藏在难以找到的地方，还是通过减少琐碎的事件处理程序的数量来实现代码服务？

Answer 1

这是一个完全合理的想法 - 但在这种特殊情况下，我会改用匿名方法：

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

好处是您不必指定参数 - 这让您更清楚地知道您没有使用它们。这是匿名方法优于 lambda 表达式的唯一优势。

对性能的影响几乎总是可以忽略不计。如果您确实需要能够删除处理程序，那么之后无法删除它们是一个非常现实的问题，但我发现我经常不需要。 （Reactive Extensions 有一个很好的方法——当你订阅一个可观察的序列时，你会得到一个 IDisposable，如果你调用它，它将删除订阅。非常整洁。）

Answer 2

实际上，它考虑将事件处理程序放在容易找到的位置，即紧挨着分配给它的事件名称。

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

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

附加到 txtFirstName 的 KeyUp 事件，因为在使用 Intellisense 创建处理程序后，有人决定重命名文本框，并且 KeyUp 效果更好。使用 Lambda，对象、事件和函数都在一起。

Answer 3

这是一个棘手的问题。我记得在Code Complete 中读到过一些（聪明的）人说你应该让控制流尽可能简单，许多人争论一个方法的单一入口和出口点，因为不这样做会使程序更难遵循.

Lambda 离这一点越来越远，在某些情况下很难跟踪正在发生的事情，控制从一个地方跳到另一个地方。

基本上，我认为这可能是个坏主意，但它也很强大，让生活更轻松。我当然会大量使用它们。总之，慎用！