这是一个糟糕的模式吗？ （切换到/ foreach循环内部）

Question

我发现自己编写的代码如下：

foreach($array as $key => $value) {
    switch($key) {
        case 'something':
            doSomething($value);
            break;
        case 'somethingelse':
            doSomethingElse($value);
            break;
    }
}

还有更好的方法吗？对我来说似乎很脏，但我可能只是在思考它。

我能想到的唯一另一种选择是每个键的if语句，这似乎没有任何好处。即：

if($array[0] == 'something') {
    doSomething($array[0]);
}
if($array[1] == 'somethingelse') {
    doSomethingElse($array[1]);
}

（或类似的东西）

如果需要，我可以发布确切的代码，但这是发生的事情的概要。请批评，但请记住我在这里寻求帮助。所以，如果我做了一些非常错误的事情，那就指出来吧。

Answer 1

将函数映射到字典/关联数组中的键是这种情况的常用方法（正如@jldupont所提到的） - 不仅在PHP中，而且在许多具有关联数组的动态语言中。例如，Python和Lua甚至没有拥有一个switch语句 - 这几乎是模拟一个开关的唯一方法。

考虑这种方法：

<?
$arr[] = "bye";
$arr[] = "hi";

function sayHi() { print("Hello.\n"); }
function sayBye() { print("Goodbye.\n"); }

$funcs["hi"] = sayHi;
$funcs["bye"] = sayBye;

foreach($arr as $k){
    $funcs[$k]();
}

?>

输出：

Goodbye.
Hello.

当你只有两个不同的值时，这是过度的，但显然它会变得更有价值，因为你需要覆盖的情况数量会增加。

Answer 2

这不是一个“坏”的解决方案，但一如既往，还有其他选择。例如，您可以删除switch语句并为字符串使用解释的处理程序。这类似于函数指针列表，但您不必使列表保持最新以添加新行为;只需将新函数添加到处理程序就可以处理它。

$array = array(
  "something" => "itsasecret",
  "somethingelse" => "i can't tell you",
);

class Handler {
  static function something($value) {
    printf("something: %s\n", $value);
  }

  static function somethingelse($value) {
    printf("somethingelse: %s\n", $value);
  }
}

$handler = new Handler();
foreach($array as $key => $value) {
  $handler->$key($value);
}

您可能需要一些代码来清理输入字符串并确保该方法存在于您的处理程序中，但这可能会给您一些想法。

Answer 3

我倾向于在foreach循环中使用开关。恕我直言的肮脏比一堆if。

您可以将开关置于其他功能中，例如：

foreach($array as $key => $value) {
   doTransaction($key , $value);
}

...

function doTransaction($key, $value){
     switch($key) {
        case 'something':
            doSomething($value);
            break;
        case 'somethingelse':
            doSomethingElse($value);
           break;
    }
}

Answer 4

还有（至少）另一种可能性：使用字典查找将工作分派给函数。

使用$ key作为“key”查找函数，检索函数引用并将$ value作为参数应用。

请原谅我，但我的PHP-fu有点生疏。

Answer 5

试试这个：

运行每个版本 - 使用交换机和if - 一百万次。每次运行的时间。

让我们知道哪一个跑得更快。

Answer 6

没错。

如果你只有2或3个项目，那么我只是为了代码复杂性而使用if。如果你有超过5个，我肯定会选择开关...