我有几个方法根据它们的成功返回一个布尔值，在 IF() 中调用这些方法有什么问题吗？
//&& makes sure that Method2() will only get called if Method1() returned true, use & to call both methods
if(Method1() && Method2())
{
// do stuff if both methods returned TRUE
}
如果 Method1() 返回 FALSE，则 Method2() 不需要触发。
让我知道上面的代码有任何问题。
谢谢。
编辑：由于代码没有任何问题，我会接受最有用的答案...添加评论以解决“新手&&&”问题
【问题讨论】：
标签： c# language-features short-circuiting
我会说你可以使用& operator（而不是&&）来保证即使左侧是false，如果将来出于某种原因也会调用这两种方法您希望避免短路。
逆向适用于 | operator，即使左侧条件的计算结果为 true，右侧条件也会被计算。
【讨论】：
不，if 条件下的方法调用没有任何问题。实际上，这可能是让您的代码更具可读性的好方法！
例如，写起来更简洁：
private bool AllActive()
{
return x.IsActive && y.IsActive && z.IsActive;
}
if(AllActive())
{
//do stuff
}
比：
if(x.IsActive && y.IsActive && z.IsActive)
{
//do stuff
}
【讨论】：
尽管它们很有用，但序列点可能会令人困惑。除非您真正理解这一点，否则 Method2() 可能根本不会被调用尚不清楚。另一方面，如果您需要调用这两个方法并且它们必须返回 true，您会写什么？你可以去
bool result1 = Method1();
bool result2 = Method2();
if (result1 && result2)
{
}
或者你可以选择
if (Method1())
if (Method2())
{
}
因此，恕我直言，我想您的问题的答案是，不，尽管行为将是您所描述的，但您的意思并不完全清楚。
【讨论】：
如果方法是纯（无副作用）函数，我只会推荐它。
【讨论】：
正如大家所说，以这种方式做事并没有什么“错误”，而且在许多情况下，您所做的正是该语言的设计目标。
但是，请记住，为了可维护性，如果 Method2 有副作用（即，它改变了某些状态），那么这个函数没有被调用可能并不明显（一个好的程序员通常会知道，但即使优秀的程序员有时会放屁）。
如果短路表达式有某种副作用，那么严格从维护的角度来看，将语句分开可能更具可读性。
【讨论】：
在我看来不错，如果之前的条件失败，if() 块中的多个子句会短路。
【讨论】：
应该没有问题。
正常的行为是 Method1() 将执行，如果返回 true Method2() 将执行，根据 Method2() 返回的内容，您可能/可能不输入 if() 语句。
现在，这假设编译器生成的代码以这种方式执行。如果你想绝对确定除非 Method1() 返回 true，否则 Method2() 不会执行，你可以这样写
if( Method1() )
{
if( Method2() )
{
// do stuff if both methods returned TRUE
}
}
但是，我一直观察到您的代码会按预期运行，所以这可能没有必要。
【讨论】：
没有错。
实际上...我不会将它们命名为 Method1 和 Method2。更具描述性的东西。也可能是被动的声音（比如 StuffHasHappened 或 DataHasLoaded）
【讨论】：
对我来说看起来不错，但有一些警告......这不是适用一揽子规则的那种事情。
我的指导方针是：
if 语句中有太多的语句/方法调用，您很可能在比较多个“集合”的事物。打破这些“集合”并引入临时变量。if( Open(host) ) 比if( WeCouldConnectToTheServer ) 短。所有这些项目的总大小就是归结为。【讨论】：
我个人会考虑
if(Method1() && Method2())
{
// do stuff if both methods returned TRUE
}
是一种不好的做法。是的，它适用于当前环境，但也可以
if(Method1())
{
if (Method2())
{
// do stuff if both methods returned TRUE
}
}
但它适用于所有环境吗？未来，可能是非微软的 C# 编译器会以这种方式工作吗？如果您的下一份工作涉及另一种语言，而这两种方法都将被调用，该怎么办？我不会依赖那个特定的构造，不是因为它是错误的，而是因为它不能解决任何严重的问题，而且将来可能会出错
【讨论】：