匿名方法的丑陋之处和优点-C#

Question

我被要求解释一下匿名方法的丑陋之处和优点。

我解释了可能

丑陋的事情

anonymous methods turning quickly into spaghetti code.

优点

我们可以使用匿名方法生成线程安全代码：示例

static List<string> Names = new List<string>(
  new string[] {
    "Jon Skeet",
    "Marc Gravell",
    "David",
    "Bill  Gates"
  });

static List<string> FindNamesStartingWith(string startingText)
{
  return Names.FindAll(
    delegate(string name)
    {
      return name.StartsWith(startingText);
    });
}

但实际上我不知道它是否是线程安全的。我被要求证明合理性它。 任何人都可以帮助我理解吗 (1)匿名方法的优点 （2）上面的代码是否线程安全？

Answer 1

嗯，“线程安全”是一个相当广泛的术语。您正在考虑哪种多线程使用？如果没有任何内容写入列表，我期望它是安全的，没有任何数据损坏或异常...

现在，至于匿名方法的“丑陋”...您使用的是 C# 3 吗？如果是这样，请开始使用 lambda 表达式，它通常更简洁：

static List<string> FindNamesStartingWith(string startingText)
{
    return Names.FindAll(name => name.StartsWith(startingText));
}

或者，使用 LINQ：

static List<string> FindNamesStartingWith(string startingText)
{
    return Names.Where(name => name.StartsWith(startingText)).ToList();
}

或者，如果您不一定需要列表：

static IEnumerable<string> FindNamesStartingWith(string startingText)
{
    return Names.Where(name => name.StartsWith(startingText));
}

或者，如果您更喜欢查询表达式：

static IEnumerable<string> FindNamesStartingWith(string startingText)
{
    return from name in names
           where name.StartsWith(startingText)
           select name;
}

对我来说，这些都不像意大利面条代码。然而，如果你想问是否应该使用这个或其他东西，你真的应该提出一个替代方案。这是一个简单的问题：

static List<string> FindNamesStartingWith(string startingText)
{
    List<string> ret = new List<string>();
    foreach (string name in Names)
    {
        if (name.StartsWith(startingText))
        {
            ret.Add(name);
        }
    }
    return ret;
}

你觉得这样更清楚了吗？如果是这样，那很好 - 但我怀疑只是你不太熟悉匿名函数、LINQ等。正如你所看到的，代码明显多了 - 我肯定需要更长的时间来检查与任何早期的样本相比，它做了正确的事情。

Answer 2

好吧，这完全取决于线程安全的含义。

如果在执行 FindAll 期间，其他某个线程更改了底层 Names 集合，那么您可能会得到奇怪的结果。

在多个线程上多次调用 FindAll 本身是否安全（即仅执行它们）？是的，这是肯定的。

至于匿名方法的优点，请考虑替代方案。

您想要：

1. 查找集合中的某些项目
2. 知道您想要查找哪个项目的唯一方法是计算每个项目的某个表达式

您可以这样做：

List<string> result = new List<string>();
foreach (string name in Names)
    if (name.StartsWith(startingText))
        result.Add(name);
return result;

或者，给定新的 lambda 语法，你可以这样做：

return Names.FindAll(name => name.StartsWith(startingText));

我知道我更喜欢哪个，但他们做不同的事情，得到相同的结果（在本例中）。

在第一种情况下，你自己执行所有代码，没有魔法。

在第二种情况下，您为 FindAll 提供了一种方法来确定将哪些项目放入结果中。在这种情况下，没有不可执行的方法可以做到这一点，您需要让 FindAll 为每个项目执行您指定的一些代码。