PHP - 在 foreach 中使用时函数不起作用

Question

我正在通过 XML 循环并从中收集数据。有些值需要翻译，所以我做了一个 switch 函数，它被包裹在另一个函数中，并在每个循环中调用以检查原始值并根据 switch 中定义的新值更改它。

问题是如果我在 foreach 中包含开关，原始值得到正确更改（原始变量 $foo 的字符串被翻译）。但是，如果我将开关包装在函数中并在循环中调用该函数，那么原始值不会改变。它必须是if ($foo == 'one') $foo == 'ein'...这是我的代码：

function checkTranslation($foo) {
    switch($foo) {
        case('one'): $foo = 'ein'; break;
        case('two'): $foo = 'zwei'; break;
        ...
    }
};

foreach ($XML -> some -> data as $item) {
    $foo = $item['value'];
    checkTranslation($foo);
}

Answer 1

更改您的函数签名以获取值by reference：

function checkTranslation(&$foo) { /* ... */ }

这是为什么呢？

当您将某些内容传递给函数时，该变量的值（可以说是副本）将被传递。无论您在函数内部对它做什么，都不会反映到该函数外部的原始变量。

但是，如果你在函数签名中的参数前面添加&，PHP实际上会通过引用传递它，这意味着函数将对你传入的变量进行操作. 因此，函数内所做的任何更改都将反映在函数外部，即使函数完成后也是如此。

如果您正在为这个概念而苦恼，为您传入的变量和方法签名中的变量选择不同的名称会有所帮助。这在我刚开始时帮助我解决了这个问题：

function checkTranslation(&$input) { /* ... */ } // method
checkTranslation($foo);                          // using the method

或者...

只需返回 $foo 从您的方法中，如下所示：

function checkTranslation($foo) {
    /* ... */
    return $foo;
}

然后，在外面，类似：

$foo = checkTranslation($item['value'];);

PS 1

我建议重命名函数，这样它的作用就更明显了。 checkTranslation() 听起来它会简单地检查 输入是否有效并返回 boolean 或其他一些指示性值。但是，您实际上是（试图）操纵输入，所以像 translate() 这样的东西可能更合适？

PS 2

还请注意 EatPeanutButter 在他的评论中指出的内容：目前，您在循环的每次迭代中都会覆盖 $foo。这似乎没有多大意义。