这是一个糟糕的模式吗？(切换到/ 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

我倾向于在 foreach 循环中使用 switch。恕我直言，不如一堆 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 3

这不是一个"坏"的解决方案,但一如既往,还有其他选择.例如,您可以删除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);
}

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