Lambda 函数中的 if 语句？

Question

这可能很简单，但我对 Lambda 还很陌生，所以请耐心等待。

我有一个使用 Lambda 函数进行递归的函数。 main 函数接收一个 bool 值，告诉它是否在 lambda 中包含某些信息。

该函数旨在将自定义类写入 XML - 我认为该代码非常不言自明。

目前我已经使用一个简单的 if 语句克服了这个问题，但感觉很难看，所以想知道是否有人知道更好的方法？

        private XElement ErrorListToXml(ErrorList el, bool outputTagsOnly)
    {
        // Need to declare in advance to call within the lambda.
        Func<ErrorType, XElement> recursiveGenerator = null;

        if (outputTagsOnly)
            recursiveGenerator = error => new XElement
                (error.Name,
                 error.ChildErrors.Select(recursiveGenerator));
        else
            recursiveGenerator = error => new XElement
          (error.Name,
          new XAttribute("Ignore", error.Filter),
           error.ChildErrors.Select(recursiveGenerator));


        var element = new XElement
                   ("ErrorList",
                    ChildErrors.Select(recursiveGenerator));

        Console.WriteLine(element);

        return element;
    }

Answer 1

mquander 的解决方案可以稍微改进以减少重复。 您可以利用以下事实：您可以在 XElement 构造函数内容中传入 null 元素，并且该元素会被忽略。 因此，我们可以进一步将条件移至：

Func<ErrorType, XElement> recursiveGenerator = null;    
recursiveGenerator = (error => new XElement(error.Name,
            outputTagsOnly ? null : new XAttribute("Ignore", error.Filter),
            error.ChildErrors.Select(recursiveGenerator));

var element = new XElement("ErrorList", ChildErrors.Select(recursiveGenerator));

Answer 2

您可以非常轻松地在 lambda 中相同类型的值之间做出决定：

customer => flag ? customer.Name : customer.Address

您可以在 lambda 中使用 if 语句，但需要花费更多的精力：

customer =>
{
  if (flag)
    return customer.Name
  else
    return customer.Address
}

这些都对您的方法没有太大帮助。

Answer 3

您可以尝试将问题分解为两个不同的问题：

1. 如何从错误结构构建树。
2. 将什么放入树节点中。

那么代码将如下所示：

        private XElement ErrorListToXml(ErrorList el, bool outputTagsOnly)
        {
            // Need to declare in advance to call within the lambda.
            Func<ErrorType, XElement> treeGenerator = null;
            Func<ErrorType, object[]> elementParametersGenerator = null;

            treeGenerator = error => new XElement
                (error.Name,
                elementParametersGenerator(error));

            if(outputTagsOnly)
                elementParametersGenerator = error => 
                    new object[] {error.ChildErrors.Select(treeGenerator)};
            else
                elementParametersGenerator = error => 
                    new object[] { new XAttribute("Ignore", error.Filter), error.ChildErrors.Select(treeGenerator) };

            var element = new XElement
                       ("ErrorList",
                        ChildErrors.Select(treeGenerator));

            Console.WriteLine(element);

            return element;
        }

在这种特殊情况下并没有明显更好，但它是一种更通用的方法。

Answer 4

如果您愿意，您可以安全地在 lambda 函数内移动“if”语句：

Func<ErrorType, XElement> recursiveGenerator = null;

recursiveGenerator = (error =>
    outputTagsOnly
        ? new XElement(error.Name,
                       error.ChildErrors.Select(recursiveGenerator));
        : new XElement(error.Name, new XAttribute("Ignore", error.Filter),
                       error.ChildErrors.Select(recursiveGenerator)));

var element = new XElement("ErrorList", ChildErrors.Select(recursiveGenerator));

除此之外，似乎没有任何简单的方法可以简化您所拥有的内容。

（PS：当它看起来很丑的时候，通过漂亮的印刷给那只猪涂上口红；）

Answer 5

我想你可以做到这一点，但最终它仍然是一个如果：

                recursiveGenerator = error => outputTagsOnly ? 
                   new XElement(error.Name,error.ChildErrors.Select(recursiveGenerator)
              : 
              new XElement(error.Name,new XAttribute("Ignore", error.Filter), 
             error.ChildErrors.Select(recursiveGenerator);