为了封装而使用闭包可以吗？

Question

我使用的是这样的代码：

$a = [];
$a['a'] = 1;
$text1 = [];
foreach ($b as $item)
{
    $text1[] = $item['1'];
}
$a['text1'] = implode(',', $text1);

$text2 = [];
foreach ($b as $item)
{
    $text2[] = $item['2'];
}
$a['text2'] = implode(',', $text2);

我的同事这样改写它：

$a = [];
$a['a'] = 1;
$a['text1'] = call_user_func(function() use ($b) {
    $text1 = [];
    foreach ($b as $item)
    {
        $text1[] = $item['1'];
    }
    return implode(',', $text1);
}();

$a['text2'] = call_user_func(function() use ($b) {
    $text2 = [];
    foreach ($b as $item)
    {
        $text2[] = $item['2'];
    }
    return implode(',', $text2);
}();

他的原因：它增加了封装性，在我的第一个示例中，除非我取消设置它们，否则会有“漫游”变量（$text1、$text2）。

Answer 1

是的，我同意你同事的观点——使用闭包来封装代码是有意义的。

但是，您所拥有的所有内容都可以简化为：

<?php

$a = [
    'a' => 1,
    'text1' => implode(',', array_column($b, '1')),
    'text2' => implode(',', array_column($b, '2')),
];

参考见：

• http://php.net/manual/en/function.array-column.php