返回值时可以执行代码吗？

Question

我想知道我是否可以缩短这个：

bool Check()
 {
 return textBox1.Text.All(char.IsDigit) ? true : Falsepath();
 }

 bool Falsepath()
 {
 MessageBox.Show("The data you entered is incorrect","Error",MessageBoxButtons.OK);
 return false;
 }    

到这样的事情：

    bool Check()
        { 
        return textBox1.Text.All(char.IsDigit) ? true : (sender, e) => 
                {
                MessageBox.Show("The data you entered is incorrect", "Error", MessageBoxButtons.OK); 
                return false;
                };
        }

当然，我输入的第二个代码不正确，但我以它为例。

那么，我可以在检查某些内容时执行代码还是必须使用单独的函数？

Answer 1

你可以写：

bool Check()
{
    return textBox1.Text.All(char.IsDigit) ? 
           true : 
           ((Func<bool>)(() =>
           {
               MessageBox.Show("The data you entered is incorrect", "Error", MessageBoxButtons.OK);
               return false;
           }))();
}

但是太可怕了，请不要这样做！...

遗憾的是，在 C# 中，您必须明确告诉编译器匿名函数的类型。这使一切变得更加复杂。看到(Func<bool>) 的演员表了吗？在 Javascript 中你不需要它，事实上在那种语言中它是一种常见的模式。在 C# 中，这不是因为它不可读且丑陋。

注意最后的() 来执行匿名方法。

请注意，在这种特殊情况下，您可以编写：

bool Check()
{
    return textBox1.Text.All(char.IsDigit) ? 
           true : 
           MessageBox.Show("The data you entered is incorrect", "Error", MessageBoxButtons.OK) == DialogResult.Abort;
}

因此调用MessageBox.Show() 并以比较结果为false 的方式比较其结果。

Answer 2

你真的需要三元运算符吗？

bool Check()
{
    if (textBox1.Text.All(char.IsDigit))
    {
        return true;
    }
    else
    {
        MessageBox.Show("The data you entered is incorrect","Error",MessageBoxButtons.OK);
        return false;
    }
}

Answer 3

阅读和维护这样的代码很痛苦。在顶层你有三元运算符，它被加载到开发人员的大脑（工作记忆）中。然后添加 lambda 表达式，它也应该被加载。然后添加一些通知用户的功能。

所以我需要记住我在 lambda 中显示错误对话框 作为三元运算符的一部分的函数，它检查是否所有 某处的字符是数字。所有这些工作人员都发生在方法调用中 您检查某事的上下文（您已经处于某些功能的中间）。

Average person can keep about 7 things in the working memory.如果你添加更多信息，你就会开始忘记以前的数据。如果可以保持简单，为什么要让方法变得如此复杂？简单的方法可以让您在大脑中保留更高级的上下文。

另一个问题是混淆方法名称，它应该只检查一些东西。它不应该通知用户或执行其他操作。并为方法和控件赋予有意义的名称。

bool IsSocialSecurityNumberValid(string ssn)
{
    return ssn.All(char.IsDigit);
}

并调用此方法：

if (!IsSocialSecurityNumberValid(ssnTextBox.Text))
   MessageBox.Show("SSN should contain only digits", "Error", MessageBoxButtons.OK);