每种情况下使用多个值进行PHP切换的最佳方法?

问题描述 投票:65回答:14

你会怎么做这个PHP switch语句?

另请注意,这些是更小的版本,我需要创建的1将添加更多的值。

版本1:

switch ($p) { 
    case 'home': 
    case '': 
        $current_home = 'current';
    break; 

    case 'users.online': 
    case 'users.location': 
    case 'users.featured': 
    case 'users.new': 
    case 'users.browse': 
    case 'users.search': 
    case 'users.staff': 
        $current_users = 'current';
    break;

    case 'forum': 
        $current_forum = 'current';
    break; 
} 

版本2:

switch ($p) { 
    case 'home': 
        $current_home = 'current';
    break; 

    case 'users.online' || 'users.location' || 'users.featured' || 'users.browse' || 'users.search' || 'users.staff': 
        $current_users = 'current';
    break;

    case 'forum': 
        $current_forum = 'current';
    break; 
} 

更新 - 测试结果

我在10,000次迭代中运行了一些速度测试,

Time1:0.0199389457703 //如果声明 Time2:0.0389049446106 //切换语句 时间3:0.106977939606 //数组

php switch-statement
14个回答
50
投票

对于你有一个未知字符串的任何情况,你需要弄清楚它匹配的其他字符串中的哪一个,唯一的解决方案是在你添加更多项目时不会变慢是使用数组,但是所有可能的字符串作为键。因此,您的交换机可以替换为以下内容:

// used for $current_home = 'current';
$group1 = array(
        'home'  => True,
        );

// used for $current_users = 'current';
$group2 = array(
        'users.online'      => True,
        'users.location'    => True,
        'users.featured'    => True,
        'users.new'         => True,
        'users.browse'      => True,
        'users.search'      => True,
        'users.staff'       => True,
        );

// used for $current_forum = 'current';
$group3 = array(
        'forum'     => True,
        );

if(isset($group1[$p]))
    $current_home = 'current';
else if(isset($group2[$p]))
    $current_users = 'current';
else if(isset($group3[$p]))
    $current_forum = 'current';
else
    user_error("\$p is invalid", E_USER_ERROR);

这看起来不像switch()那样干净,但它是唯一一个不包括编写一个小函数和类库以保持其整洁的快速解决方案。将项添加到数组中仍然非常容易。


0
投票

我认为版本1是要走的路。阅读和理解起来要容易得多。


0
投票
if( in_array( $test, $array1 ) )
{
    // do this
}
else if( stristr( $test, 'commonpart' ) )
{
    // do this
}
else
{
    switch( $test )
    {
        case 1:
            // do this
            break;
        case 2:
            // do this
            break;
        default:
            // do this
            break;
    }
}

0
投票

切换与variable variables结合将为您提供更多灵活性:

<?php
$p = 'home'; //For testing

$p = ( strpos($p, 'users') !== false? 'users': $p);
switch ($p) { 
    default:
        $varContainer = 'current_' . $p; //Stores the variable [$current_"xyORz"] into $varContainer
        ${$varContainer} = 'current'; //Sets the VALUE of [$current_"xyORz"] to 'current'
    break;

}
//For testing
echo $current_home;
?>

要了解更多信息,请查看variable variables以及我提交给php手册的示例: 例1:http://www.php.net/manual/en/language.variables.variable.php#105293 例2:http://www.php.net/manual/en/language.variables.variable.php#105282

PS:这个示例代码很简单,就像我喜欢的那样。它经过测试和运作


0
投票

现在你可以......

switch ([$group1, $group2]){
    case ["users", "location"]:
    case ["users", "online"]:
        Ju_le_do_the_thing();
        break;
    case ["forum", $group2]:
        Foo_the_bar();
        break;
}

0
投票

也许

        switch ($variable) {
        case 0:
            exit;
            break;
        case (1 || 3 || 4 || 5 || 6):
            die(var_dump('expression'));
        default:
            die(var_dump('default'));
            # code...
            break;
    }

25
投票

Version 2 does not work!!

case 'users.online' || 'users.location' || ...

与以下内容完全相同:

case True:

并且case将被选为$p的任何值,除非$p是空字符串。

||case声明中没有任何特殊含义,你不是将$p与每个字符串进行比较,你只是检查它是否不是False


8
投票

将这些值放入数组并查询数组,因为switch-case似乎隐藏了当字符串变量用作条件时你想要实现的基本语义,使得它更难以阅读和理解,例如:

$current_home = null;
$current_users = null;
$current_forum = null;

$lotsOfStrings = array('users.online', 'users.location', 'users.featured', 'users.new');

if(empty($p)) {
    $current_home = 'current';
}

if(in_array($p,$lotsOfStrings)) {
    $current_users = 'current';
}

if(0 === strcmp('forum',$p)) {
    $current_forum = 'current';
}

4
投票

如果有其他人维护你的代码,他们几乎肯定会对版本2进行双重考虑 - 这是非常不标准的。

我会坚持使用版本1.虽然那个案例陈述没有自己的陈述块,但是他们旁边应该有一个明确的// fall through评论,以表明它确实是你的意图通过,从而消除任何歧义你是否会以不同的方式处理案件,忘记什么。


4
投票

为了完整起见,我将指出破坏的“版本2”逻辑可以替换为有效的switch语句,并且还可以使用数组来提高速度和清晰度,如下所示:

// used for $current_home = 'current';
$home_group = array(
    'home'  => True,
);

// used for $current_users = 'current';
$user_group = array(
    'users.online'      => True,
    'users.location'    => True,
    'users.featured'    => True,
    'users.new'         => True,
    'users.browse'      => True,
    'users.search'      => True,
    'users.staff'       => True,
);

// used for $current_forum = 'current';
$forum_group = array(
    'forum'     => True,
);

switch (true) {
    case isset($home_group[$p]):
        $current_home = 'current';
        break;
    case isset($user_group[$p]):
        $current_users = 'current';
        break;
    case isset($forum_group[$p]):
        $current_forum = 'current';
        break;
    default:
        user_error("\$p is invalid", E_USER_ERROR);
}    

4
投票

尚未提及的其他一些想法:

switch(true){ 
  case in_array($p, array('home', '')): 
    $current_home = 'current'; break;

  case preg_match('/^users\.(online|location|featured|new|browse|search|staff)$/', $p):
    $current_users = 'current'; break;

  case 'forum' == $p:
    $current_forum = 'current'; break; 
}

有人可能会抱怨#2的可读性问题,但我继承这样的代码没有问题。


3
投票

版本1在眼睛上更容易,更清楚你的意图,更容易添加案例条件。

我从未尝试过第二个版本。在许多语言中,这甚至不会编译,因为每个案例标签必须评估为常量表达式。


1
投票

我绝对更喜欢版本1.版本2可能需要更少的代码行,但是一旦你有很多值,就像你预测的那样,它将非常难以阅读。

(老实说,我甚至不知道版本2到目前为止是合法的。我以前从未见过这样做过。)


1
投票

没有版本2实际上不起作用,但如果您想要这种方法,您可以执行以下操作(可能不是最快,但可以说更直观):

switch (true) {
case ($var === 'something' || $var === 'something else'):
// do some stuff
break;
}

© www.soinside.com 2019 - 2024. All rights reserved.