问单行if语句或没有大括号的if语句是不好的做法吗？

EN

最佳实践是编写其他人可以轻松阅读和更新的代码。

您的第一个表单是有问题的，因为它不遵循大多数PHP开发人员习惯的表单：

if (condition) {
  // code
} else {
  // code
}

// ... or ...

if (condition)
{
  // code
}
else
{
  // code
}

// ... or ...

if (condition) { /* short code */ } else { /* short code */ }

// ... or ...

condition ? /* short code */ : /* short code */;

请注意，这完全是关于标准实践的，并不一定有意义-它只是关于其他开发人员所习惯看到的。

更重要的是，你的第二个表单不是很好，因为它很容易让另一个程序员犯这个错误：

if (condition)
  // code A
else
  // code B
  // code C (added by another programmer)

在本例中，另一个程序员添加了code C，但忘记了将整个else代码块用大括号括起来。这会带来问题。您可以通过简单地将if和else块包装在大括号中来防止这种情况。

如果是为了一致性我更喜欢...所以：

if(...)
{
   statement 1;
   statement 2;
}
else
{
   statement 1;
   statement 2;
}

与以下内容没有区别：

if(...)
{
   statement 1;
}
else
{
   statement 1;
}

所以我总是使用它们，因为它是一致的，而且它避免了后来忘记添加它们的问题。

然而，其他人会看着我的代码，认为把{和}放进去是愚蠢的。他们有他们的理由，我也有我的理由。我碰巧更喜欢我的理由而不是他们的理由:-)

一般来说，不可读的代码是一种糟糕的做法。单行输入效率更高，并且可以节省行号，但是在一年后或在扫描bug时再使用它，会使它变得更加困难。

在我看来，是的，只有一行if语句是不好的做法。

计算机实际上并不关心(据我所知)，但你应该总是像一个知道你住在哪里的连环杀手来维护你的代码一样编写代码。

可读性！很容易自我辨别。