在 PHP 中堆叠多个三元运算符

新手上路,请多包涵

这就是我写的:

  $Myprovince = (
($province == 6) ? "city-1" :
($province == 7) ? "city-2" :
($province == 8) ? "city-3" :
($province == 30) ? "city-4" : "out of borders"
);

但是对于每个字段,我都得到了值 city-4 。我想使用三元运算符而不是 switch/if 因为我想试验看看它是如何完成的。

这段代码有什么问题?

原文由 Mac Taylor 发布,翻译遵循 CC BY-SA 4.0 许可协议

阅读 569
2 个回答

其他人已经提出了正确的做法,但如果你真的想使用三元运算符,你需要使用括号作为:

 $province = 7;
 $Myprovince = (
 ($province == 6) ? "city-1" :
  (($province == 7) ? "city-2" :
   (($province == 8) ? "city-3" :
    (($province == 30) ? "city-4" : "out of borders")))
 );

更新链接

原文由 codaddict 发布,翻译遵循 CC BY-SA 4.0 许可协议

三元运算符从左到右计算。所以如果你没有正确地对表达式进行分组,你会得到意想不到的结果。

PHP 的建议是[docs]

建议您避免“堆叠”三元表达式。在单个语句中使用多个三元运算符时,PHP 的行为并不明显。

您的代码实际上被评估为:

 (
    (
        (
            $province == 6 ? "city-1" : $province == 7
        ) ? "city-2" :
        $province == 8
    ) ? "city-3" : $province == 30
) ? "city-4" : "out of borders";

它应该在哪里

$province == 6 ? "city-1" : (
    $province == 7 ? "city-2" : (
        $province == 8 ? "city-3" : (
           $province == 30 ? "city-4" : "out of borders"
        )
    )
);

这段代码可能看起来不错,但有人会阅读它,他们将需要更多的时间来理解这段代码的作用。


像这样的事情你会更好:

 $map = array( 6 = >'city-1',
              7 => 'city-2',
              8 => 'city-3',
             30 => 'city-4');

$Myprovince = "out of borders";

if(array_key_exists($province, $map)) {
    $Myprovince = $map[$province];
}

或者正如 @Jonah 在他的评论中提到的那样:

 $Myprovince = isset($map[$province]) ? $map[$province] : 'out of borders';

原文由 Felix Kling 发布,翻译遵循 CC BY-SA 3.0 许可协议

推荐问题
logo
Stack Overflow 翻译
子站问答
访问
宣传栏