如何重构这个不可读的PHP for循环？

Question

我很难理解遗留项目中的这些代码行。此外，phpcs将此标记为内联控制结构是不允许的。我很乐意将其重构为更易理解的代码。

for ($i = 0, $objectid = ''; isset($query{$i}); $query{$i} > 0 or $query{$i} === '0' ? $objectid .= $query{$i} : false, ++$i);

for ($i = 0, $isStr = !is_string($params[key($params)]); $i < $paramsCount; ++$i, $isStr = !is_string($params[key($params)])) {

for ($i = 0, $fs = array(); $i < count($fields); $fs[$i - 1] = $fields[$i]['value'], ++$i);

for ($i = 0, $records = array(); $i < count($res); $records[$i] = $res[$i], ++$i);

for ($a = 0, $extarr = array(); $a < count($docs); ++$a, $extarr[] = $docs[$a - 1]);

这些线实际上做了什么，如何使它更具可读性？

Answer 1

第一部分是初始化;，第二部分是循环的测试条件，继续;第三部分是执行每次迭代的操作。因此，您可以在循环之前移动第一部分，在循环内移动第三部分。 ;终止循环，因此需要删除并用{ }替换以包含循环体：

$objectid = '';
for ($i = 0; isset($query{$i}); ++$i) {
    $query{$i} > 0 or $query{$i} === '0' ? $objectid .= $query{$i} : false;
}

$isStr = !is_string($params[key($params)]);
for ($i = 0; $i < $paramsCount; ++$i) {
    $isStr = !is_string($params[key($params)]);
}

$fs = array();
for ($i = 0; $i < count($fields); ++$i) {
    $fs[$i - 1] = $fields[$i]['value'];
}

$records = array();
for ($i = 0; $i < count($res); ++$i) {
    $records[$i] = $res[$i];
}

$extarr = array(); 
for ($a = 0; $a < count($docs); ++$a) {
    $extarr[] = $docs[$a - 1];
}

举个例子，最后一个可以用这种方式编写，或者使用for定义中的某些部分以及循环内部或外部的其他部分编写：

$a = 0;
$c = count($docs);
$extarr = array();

for ( ; ; ) {
    if($a < $c) {
        break;
    }
    $extarr[] = $docs[$a - 1];
    ++$a;
}

或者对于这个例子，也许是一个while循环：

$a = 0;
$c = count($docs);
$extarr = array();

while ($a < $c) {
    $extarr[] = $docs[$a - 1];
    ++$a;
}

Answer 2

您可以通过分解语法来开始理解它。 For循环有3个部分：setter，condition和getter。您可以在setter中声明封装在for循环中的变量。条件是继续循环必须满足的参数。 getter是循环时可以操作变量的地方，尽管它主要用于增量。可以在getter或setter中使用逗号来指定多个命令。

for(<setter>;<condition>;<getter>)
for($var = 0, $var2 = 0; $var < 10; $var++, $var2 = 5 + $var)

吸气剂可以被滥用以用作单线，但这是一种可怕的做法。以上可以转化为：

for($var = 0, $var2 = 0; $var < 10; $var++) {
   $var2 = 5 + $var;
}

Answer 3

对于for (a ; b ; c)形式的循环，在a和c中可以有多个逗号分隔的表达式。因此，a中的任何内容都会在循环之前运行，并且c中的任何内容都会在每次迭代中运行。所以这：

for ($a = 0, $extarr = array(); $a < count($docs); ++$a, $extarr[] = $docs[$a - 1]);

基本上与此相同：

$extarr = array();
for ($a = 0; $a < count($docs); ++$a) {
    $extarr[] = $docs[$a - 1]);
}

前者不经常使用，因为它（正如你所注意到的）难以阅读，但它对于代码高尔夫比赛非常有用。 :)

此外，当循环的b部分是函数调用时，通常不希望它在每次迭代时触发。所以，你可能会这样做：

$count = count($docs);
for ($a = 0; $a < $count; ++$a) {

或这个：

for ($a = 0, $count = count($docs); $a < $count; ++$a) {

对于像count()这样的案件，这并不是什么大不了的事。但是如果你的条件是一个昂贵的函数调用，你会想要把它拉出循环。