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

Question

这是我写的：

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

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

这个代码有什么问题？

Answer 1

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

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';

Answer 2

其他人已经建议了正确的方法，但是如果你真的想使用三元运算符，你需要用括号作为：

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

更新的链接