常见做法？

Question

我一直想知道这种简写方式（如果我们可以这么称呼的话）是否是专业 PHP 开发人员中可接受的编码实践：

foo() && bar();

虽然在

if( foo() ) {
    bar();
}

我看来，单行代码更加简洁，但我还没有看到它在任何地方使用。

Answer 1

这是多种语言开发人员的常见做法。从技术上讲它没有任何问题，但从它在 StackOverflow 上出现的次数来看，我会继续说它是“仅当它是唯一有意义的事情时才使用”。大多数人都没有想到这一点。

也就是说，这是完全有效的：

foo()?bar()?bat():zonk():baz();

正如

foo() && bar() && baz();

只需记住：

function foo(){return true;} 
function bar(){echo "bar";} 
function baz(){echo "baz";  return 2;} 
echo foo()||foo()?baz():baz(); // baz2

Answer 2

就我个人而言，我认为这没有什么问题。它相当普遍，因此大多数 PHP 程序员都会像理解扩展版本一样理解它，从而使其完全可以接受。

Answer 3

最重要的部分是编写易于阅读的代码。例如，使用诸如 foo() 和 bar() 之类的函数名称是完全没有用的，因此很难说您要比较的两者中哪一个更好。

除此之外，if 示例也有一些相当大的缺陷：

• if 是一种语言构造，但您可以像函数一样使用它。
• 您可以在 if 条件中添加垂直空间。这可能会使内容难以阅读，因为空间会影响视觉焦点。

另一个建议是：

if (foo()) {

但是您想比较两者：代码始终在其上下文中。可读的代码使用它的上下文。你的两个建议都是有效的，关键是你已经可以从代码中读到含义：

   conditionMet() && gotForIt();

   if (conditionMet()) goForIt();

自己决定。只是不要从一行到另一行混合，因此在整个代码中保持一种样式。

Answer 4

foo() && bar(); 是一个布尔表达式，因此如果不将其放入布尔上下文中（如 if、while 等条件），则会出错）。

代码必须讲述正在解决的问题的“故事”，使用此快捷方式并不是在讲述故事。这是一个丑陋的黑客，它不优雅。

如果您认为它“简洁”，那么您还没有达到代码意味着想法的地步，而不是代码。