如何消除 3 行重复代码？

Question

三行重复两次。 怎样才能做得更好？

 $selectedComponentsNewTmp = $this->getNewSelectedComponents(); $selectedComponentsNew = []; foreach ($selectedComponentsNewTmp as $component) { $selectedComponentsNew[$component['componentGroup']['Count_0']]['name'] = $component['componentGroup']['name']; $selectedComponentsNew[$component['componentGroup']['Count_0']]['value'] = $component['coverage']['value']?? ''; $selectedComponentsNew[$component['componentGroup']['Count_0']]['txt'] = $component['coverage']['txt']?? ''; } $selectedComponentsOldTmp = $this->getOldSelectedComponents(); $selectedComponentsOld = []; foreach ($selectedComponentsOldTmp as $component) { $selectedComponentsOld[$component['componentGroup']['Count_0']]['name'] = $component['componentGroup']['name']; $selectedComponentsOld[$component['componentGroup']['Count_0']]['value'] = $component['coverage']['value']?? ''; $selectedComponentsOld[$component['componentGroup']['Count_0']]['txt'] = $component['coverage']['txt']?? ''; }

Answer 1

您也许可以使用一个数组来指定每行中使用的最终值，并在外循环中迭代该数组。

$args=['name','value','txt'];
$new=$old=[];


foreach( $this->getNewSelectedComponents() as $component) {
    foreach( $args as $arg ) $new[$component['componentGroup']['Count_0']][$arg] = $component['componentGroup'][$arg] ?? '';
}
foreach( $this->getOldSelectedComponents() as $component ) {
    foreach( $args as $arg ) $old[$component['componentGroup']['Count_0']][$arg] = $component['componentGroup'][$arg] ?? '';
}

Answer 2

您可以创建和使用function ：

function

function myFunction($selectedComponentsNew, $component) {
    $selectedComponentsNew[$component['componentGroup']['Count_0']]['name'] = $component['componentGroup']['name'];
    $selectedComponentsNew[$component['componentGroup']['Count_0']]['value'] = $component['coverage']['value'] ?? '';
    $selectedComponentsNew[$component['componentGroup']['Count_0']]['txt'] = $component['coverage']['txt'] ?? '';
}

用法

$selectedComponentsNewTmp = $this->getNewSelectedComponents();
$selectedComponentsNew = [];
foreach ($selectedComponentsNewTmp as $component) {
    myFunction($selectedComponentsNew, $component);
}

$selectedComponentsOldTmp = $this->getOldSelectedComponents();
$selectedComponentsOld = [];
foreach ($selectedComponentsOldTmp as $component) {
    myFunction($selectedComponentsOld, $component);
}