在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

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

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

更新链接

Answer 2

三元运算符从左到右进行评估.因此,如果您没有正确分组表达式,您将得到意外的结果.

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 3

不要滥用三元运算符来做这类事情.它使调试几乎无法遵循.为什么不做类似的事情

switch($province) {
    case 6: $Myprovince = "city-1"; break;
    case 7: ...
}

或者只是一些链接if/then/else

if ($province == 6) {
     $Myprovince = "city-1";
} elseif ($province = ...) {
   ...
}

Answer 4

有些人建议使用switch语句或if/else语句.但我会改用数组,以便于维护和更容易阅读:

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

// then you can call:

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

为什么？

代码最终可能更容易管理.也许你想要在一天之内从数据库添加那些省到城市的映射......等等.用一堆switch/case语句很难维护.

Answer 5

我知道这是一个关于 PHP 的问题，但由于这只是一个教育练习，我认为您可能有兴趣了解 Ruby 和 Javascript 实际上的行为方式是否符合您的预期。

红宝石：

ree-1.8.7-2012.02 :009 > def foo x
ree-1.8.7-2012.02 :010?>   x == 1 ? "city 1" : x == 2 ? "city 2" : "out of borders"
ree-1.8.7-2012.02 :011?>   end
 => nil
ree-1.8.7-2012.02 :012 > foo 1
 => "city 1"
ree-1.8.7-2012.02 :013 > foo 2
 => "city 2"
ree-1.8.7-2012.02 :014 > foo 3
 => "out of borders"

Javascript：

> function f(x) { return x == 1 ? "city 1" : x == 2 ? "city 2" : "out of borders"; }
undefined
> f(1)
"city 1"
> f(2)
"city 2"
> f(3)
"out of borders"