使用返回时的 PHP 速记 If/Else

Question

在 PHP 中编写速记的好方法很少。

不太常见但最短的例子：

!isset( $search_order ) && $search_order = 'ASC';

更常见但更长一点：

!isset( $search_order ) ? $search_order = 'ASC' : $search_order = NULL;

我们甚至可以将上面的例子组合成一个惊人的速记：

!isset( $_POST['unique_id'] ) && preg_match( '/^[a-zA-Z0-9]{8}$/', $_POST['unique_id'] ) ? $post_unique_id = $_POST['unique_id'] : $post_unique_id = NULL;

但是我们如何将上面的示例与函数和返回一起使用，示例：

function filter_gender_request($data) {  
    preg_match('/(fe)?male/i', $data, $data);
    isset($data[0]) && return $data[0]; // It doesn't work here with return
}

同时，如果我陈述以下内容，而不是 isset($data[0]) && return $data[0];，那么一切都会按预期进行：

if (isset($data[0]) ) {
    return $data[0];
}

我在这里做错了什么？如果第一个也是最短的示例在函数之外完美地工作，那么为什么它不能与 return 一起工作？

是否有可能在 return 中使用速记？

Answer 1

使用您当前的语法，当$data[0] 未设置时，您希望您的函数返回什么？当然，您不希望您的函数不返回任何东西，具体取决于条件。

我看到的唯一替代方法是 ternary operator，如果未设置，则返回 $data[0] 以外的其他内容：

return isset($data[0]) ? $data[0] : null; 

Answer 2

对于未来的谷歌员工。

使用 php 7 null coalesce (??) 运算符

return $data[0] ?? null; 

Answer 3

你惊人的快捷方式实际上是相当可怕的代码。您正在滥用三元运算符，并且该代码实际上远不如您写出来的代码可读性和可维护性差。人们期望三元组执行测试并返回一个非此即彼的值。在其中执行分配不是正常行为。

Answer 4

您的代码的问题是您试图将return 语句作为三元表达式的一部分执行。三元运算符通常会产生如下赋值：

$message = is_error() ? get_error() : 'No Errors';

这会导致基于is_error() 的返回值分配给$message。您的代码正在尝试处理操作中的程序控制语句。 return 不能分配给变量。

因此，其他用户发布的内容更适合您的情况。

Answer 5

好吧，我不知道你在做什么，但这应该可以：

return ( isset($data[0]) ? $data[0] : false);

Answer 6

同意这里的回答，一旦你离开它并回来，或者更糟的是，未来的另一个开发人员，这个速记就更难阅读了。

想象一下，即使是一个 500 行的小脚本文件，还有 40 行的速记 elseif，当你使用它时，你可以尝试添加或更改代码吗？

特别是当主题或内容不是你熟悉的东西时，调试或添加变得很头疼。

这更易于管理，不管它是什么，它只是代码：

if ($var == 'unicorns')
  {
    $this->remove_horn;
  }
elseif ($var == 'horse')
 {
   $this->glue_on_horn;
 }
else
  {
    $this->must_be_a_zebra;
  }

只是说