Question

我不是说这个问题过于主观。

我已经谷歌了一段时间但没有得到解决这个问题的具体答案。问题是，我认为我有点沉迷于 LINQ 。我已经使用LINQ来查询列表，比如使用Linq到Sql，Xml等等。但后来有些事情让我感到震惊：“如果我用它来查询单个对象怎么办？”我做了。试图用手榴弹发射器杀死一只苍蝇似乎有点不对劲。虽然我们都同意，看到这将是艺术上的愉快。

我认为它非常可读，我认为这方面没有任何性能问题，但让我向您展示一个例子。

在Web应用程序中，我需要从配置文件（web.config）中检索设置。但是如果密钥不存在，这应该有一个默认值。此外，我需要的值是十进制，而不是字符串，这是ConfigurationManager.AppSettings["myKey"]的默认返回。此外，我的电话号码不应超过10，也不应该是负数。我知道我可以这样写：

string cfg = ConfigurationManager.AppSettings["myKey"];
decimal bla;
if (!decimal.TryParse(cfg,out bla))
{
    bla = 0; // 0 is the default value
}
else
{
    if (bla<0 || bla>10)
    {
        bla = 0;
    }
}

这不复杂，不复杂，易于阅读。但是，这就是我喜欢它的方式：

// initialize it so the compiler doesn't complain when you select it after
decimal awesome = 0; 
// use Enumerable.Repeat to grab a "singleton" IEnumerable<string>
// which is feed with the value got from app settings
awesome = Enumerable.Repeat(ConfigurationManager.AppSettings["myKey"], 1)
    // Is it parseable? grab it 
    .Where(value => decimal.TryParse(value, out awesome))
    // This is a little trick: select the own variable since it has been assigned by TryParse
    // Also, from now on I'm working with an IEnumerable<decimal>
    .Select(value => awesome)
    // Check the other constraints
    .Where(number => number >= 0 && number <= 10)
    // If the previous "Where"s weren't matched, the IEnumerable is empty, so get the default value
    .DefaultIfEmpty(0)
    // Return the value from the IEnumerable
    .Single();

没有评论，它看起来像这样：

decimal awesome = 0; 
awesome = Enumerable.Repeat(ConfigurationManager.AppSettings["myKey"], 1)
    .Where(value => decimal.TryParse(value, out awesome))
    .Select(value => awesome)
    .Where(number => number >= 0 && number <= 10)
    .DefaultIfEmpty(0)
    .Single();

我不知道我是否是这里唯一的一个，但我觉得第二种方法比第一种方法更“有机”。由于 LINQ ，它不容易调试，但我认为它非常不合适。至少我写的这个。无论如何，如果你需要调试，你可以在linq方法中添加花括号和返回语句，并对它感到高兴。

我现在已经做了一段时间了，感觉比做一些事情“每行一行，一步一步”更自然。另外，我只指定了一次默认值。它写在一行DefaultIfEmpty，所以它很简单。

另外，如果我注意到查询会比我在那里写的要大得多，我绝对不会这样做。相反，我闯入了较小的linq荣耀块，因此更容易理解和调试。

我发现更容易看到变量分配并自动思考：这是您必须要设置此值，而不是查看ifs，elses，switch等，并尝试弄清楚它们是否属于公式的一部分。

我认为它可以防止开发人员在错误的地方写下不受欢迎的副作用。

但最后，有些人可能会说它看起来非常<强烈> hackish ，或者太强烈。

所以我接到了手头的问题：

对一个被认为是不良做法的对象使用LINQ吗？

Answer 1

我说是的，但这取决于偏好。它肯定有缺点，但我会把它留给你。您的原始代码可以变得更加简单。

string cfg = ConfigurationManager.AppSettings["myKey"];
decimal bla;
if (!decimal.TryParse(cfg,out bla) || bla < 0 || bla > 10)
    bla = 0; // 0 is the default value

这是因为“短路”评估，这意味着一旦找到第一个真实条件，程序将停止检查其他条件。

使用LINQ对单个对象被认为是一种不好的做法？

1 个答案: