虽然每个都已弃用，但 Foreach 替换需要更多时间答案

【问题标题】：While each is deprecated, Foreach replacement takes a lot more time虽然每个都已弃用，但 Foreach 替换需要更多时间
【发布时间】：2018-12-25 16:17:58
【问题描述】：

在 PHP 7.2 中，each 已弃用。 The documentation 说：

警告自 PHP 7.2.0 起，该功能已被弃用。强烈建议不要依赖此函数。

我正在调整电子商务应用程序并将所有while-each 循环转换为（假定）等效的foreach。

如下所示，我已经将所有 reset 和 while 循环替换为等效的 foreach。

它大部分工作正常。但是，我们有一个客户，她的购物车中有很长的商品清单，试图结账并抱怨她从服务器收到错误 502。我试图重现这一点，发现只有她的购物车失败，结帐页面加载需要 2 多分钟，然后是 502。然后我开始调试我最近修改的很多文件，反复试验，直到我发现问题出在这个特定的文件和特定的功能上。每当我将第一个foreach 循环切换回while 循环时，客户可以在不到一秒的时间内加载结帐页面。切换回foreach - 又需要几分钟，但 php 在结束执行之前会超时。

我确实对foreach 与while 循环（例如var_dump $products_id 和$this->contents）的输出进行了测试，它们看起来都相同。我已经重写了代码以使其顺利运行并保持 PHP 7.2 兼容，但我仍然无法弄清楚为什么会发生这种情况。

这是完整的功能：

function get_content_type() {
  $this->content_type = false;

  if ( (DOWNLOAD_ENABLED == 'true') && ($this->count_contents() > 0) ) {

    // reset($this->contents);
    // while (list($products_id, ) = each($this->contents)) {
    foreach(array_keys($this->contents) as $products_id) {

      if (isset($this->contents[$products_id]['attributes'])) {
        // reset($this->contents[$products_id]['attributes']);
        // while (list(, $value) = each($this->contents[$products_id]['attributes'])) {
        foreach ($this->contents[$products_id]['attributes'] as $value) {
          $virtual_check_query = tep_db_query("select count(*) as total from " . TABLE_PRODUCTS_ATTRIBUTES . " pa, " . TABLE_PRODUCTS_ATTRIBUTES_DOWNLOAD . " pad where pa.products_id = '" . (int)$products_id . "' and pa.options_values_id = '" . (int)$value . "' and pa.products_attributes_id = pad.products_attributes_id");
          $virtual_check = tep_db_fetch_array($virtual_check_query);

          if ($virtual_check['total'] > 0) {
            switch ($this->content_type) {
              case 'physical':
                $this->content_type = 'mixed';

                return $this->content_type;
                break;
              default:
                $this->content_type = 'virtual';
                break;
            }
          } else {
            switch ($this->content_type) {
              case 'virtual':
                $this->content_type = 'mixed';

                return $this->content_type;
                break;
              default:
                $this->content_type = 'physical';
                break;
            }
          }
        }

      } elseif ($this->show_weight() == 0) {
      // reset($this->contents);  
      //  while (list($products_id, ) = each($this->contents)) { 
        foreach (array_keys($this->contents) as $products_id) {
          $virtual_check_query = tep_db_query("select products_weight from " . TABLE_PRODUCTS . " where products_id = '" . $products_id . "'");
          $virtual_check = tep_db_fetch_array($virtual_check_query);
          if ($virtual_check['products_weight'] == 0) {
            switch ($this->content_type) {
              case 'physical':
                $this->content_type = 'mixed';

                return $this->content_type;
                break;
              default:
                $this->content_type = 'virtual';
                break;
            }
          } else {
            switch ($this->content_type) {
              case 'virtual':
                $this->content_type = 'mixed';

                return $this->content_type;
                break;
              default:
                $this->content_type = 'physical';
                break;
            }
          }
        }

      } else {
        switch ($this->content_type) {
          case 'virtual':
            $this->content_type = 'mixed';

            return $this->content_type;
            break;
          default:
            $this->content_type = 'physical';
            break;
        }
      }
    }
  } else {
    $this->content_type = 'physical';
  }

  return $this->content_type;
}

谢谢

编辑： 这里是数组： https://pastebin.com/VawX3XpW

该问题已在我尝试过的所有配置上进行了测试和重现：

1) 高端 windows 10 pc + WAMP (Apache 2.4 + MariaDB 10.2 + PHP 5.6+/7+/7.1+/7.2+)

2) 高端 CentOS/cPanel 服务器 + Litespeed + MariaDB 10.1 + PHP 5.6+

强调一下，我不打算重写代码或模仿each 然后重写代码，因为我们不会从中学到很多东西。我只是想找到一个合乎逻辑的解释或解决/调试这个谜团的方法。也许有人在某个地方遇到过这样的问题，并且可以对此有所了解。

2018 年 8 月 1 日更新

我已经尝试调试了好几天，最终发现了一些有趣的东西。我在第一个foreach 循环和while 循环上添加了“回声点”和exit，如下所示：

function get_content_type() {
  $this->content_type = false;

  if ( (DOWNLOAD_ENABLED == 'true') && ($this->count_contents() > 0) ) {

    // reset($this->contents);
    // while (list($products_id, ) = each($this->contents)) { echo '1 ';
    foreach(array_keys($this->contents) as $products_id) { echo '1 ';

      if (isset($this->contents[$products_id]['attributes'])) { echo '2 ';
        // reset($this->contents[$products_id]['attributes']);
        // while (list(, $value) = each($this->contents[$products_id]['attributes'])) {
        foreach ($this->contents[$products_id]['attributes'] as $value) { echo '3 ';
          $virtual_check_query = tep_db_query("select count(*) as total from " . TABLE_PRODUCTS_ATTRIBUTES . " pa, " . TABLE_PRODUCTS_ATTRIBUTES_DOWNLOAD . " pad where pa.products_id = '" . (int)$products_id . "' and pa.options_values_id = '" . (int)$value . "' and pa.products_attributes_id = pad.products_attributes_id");
          $virtual_check = tep_db_fetch_array($virtual_check_query);

          if ($virtual_check['total'] > 0) {
            switch ($this->content_type) {
              case 'physical':
                $this->content_type = 'mixed'; echo '4 ';

                return $this->content_type;
                break;
              default:
                $this->content_type = 'virtual'; echo '5 ';
                break;
            }
          } else {
            switch ($this->content_type) {
              case 'virtual':
                $this->content_type = 'mixed'; echo '6 ';

                return $this->content_type;
                break;
              default:
                $this->content_type = 'physical'; echo '7 ';
                break;
            }
          }
        }

      } elseif ($this->show_weight() == 0) {
      // reset($this->contents);  
      //  while (list($products_id, ) = each($this->contents)) { 
        foreach (array_keys($this->contents) as $products_id) {
          $virtual_check_query = tep_db_query("select products_weight from " . TABLE_PRODUCTS . " where products_id = '" . $products_id . "'");
          $virtual_check = tep_db_fetch_array($virtual_check_query);
          if ($virtual_check['products_weight'] == 0) {
            switch ($this->content_type) {
              case 'physical':
                $this->content_type = 'mixed'; echo '8 ';

                return $this->content_type;
                break;
              default:
                $this->content_type = 'virtual'; echo '9 ';
                break;
            }
          } else {
            switch ($this->content_type) {
              case 'virtual':
                $this->content_type = 'mixed'; echo '10 ';

                return $this->content_type;
                break;
              default:
                $this->content_type = 'physical'; echo '11 ';
                break;
            }
          }
        }

      } else {
        switch ($this->content_type) {
          case 'virtual':
            $this->content_type = 'mixed'; echo '12 ';

            return $this->content_type;
            break;
          default:
            $this->content_type = 'physical'; echo '13 ';
            break;
        }
      }
    } exit; //Exiting from the loop to check output
  } else {
    $this->content_type = 'physical';
  }

  return $this->content_type;
}

当我使用while 运行循环时，我得到的输出只是“1 13”一次，这意味着循环只运行一次并停止。但是，当我将其更改为foreach 时，我得到了一长串“1 13 1 13 1 13...”，这意味着它循环了很多次。我已经去进一步调查breaks 的while 循环和foreach 循环之间是否有任何区别，但我仍然找不到任何支持信息。然后我将最后一个break; 重写为break 2; 并再次测试foreach，这一次它似乎只运行了一次，就像它是带有while 循环的break;（不是@ 987654353@) 编辑： 澄清一下 - while breaks 和 foreach breaks 之间没有区别。它们的工作方式相同。

更新 #2： 我已将} elseif ($this->show_weight() == 0) { 修改为} elseif (2 == 0) {，while 循环现在运行的次数与foreach 循环一样多。 var_dump($this->show_weight()); 结果float 4466.54。这个问题对我来说仍然没有任何意义。

再次感谢

【问题讨论】：

当结帐页面加载需要 2 分钟时，视觉结果是什么？是正常的，还是有很多重复的项目，或者它可能被截断（表明某种无限循环）？
@rlanvin 首先，由于内存耗尽，它在大约 45 秒后崩溃。当我增加该值时，由于 120 秒的限制，它超时，然后我增加了更多，最终它在很长一段时间后加载，页面上的细节似乎正常。
你试过 xdebug 或 microtime 吗？
是的，WAMP 服务器开启了 xdebug。试过什么？
foreach 实际上是在大多数情况下循环遍历数组的一种非常慢的方法，而 for 与预先计算的 count 相比。见phpbench.com。将 foreach 循环重写为 for 循环可能会显着提高性能，但显然您应该对此进行测试。

标签： php performance foreach while-loop each

【解决方案1】：

作为一种备份解决方案，执行相同操作的替代品下降会怎样？

function eachLegacy( &$array )
{
  if( current( $array ) !== false )
  {
    $return = array( 1 => current( $array ), 'value' => current( $array ), 0 => key( $array ), 'key' => key( $array ) ); // Get the current values
    next( $array ); // Advance the cursor
    return $return;
  }
  return false;
}

【讨论】：

谢谢，@Scuzzy，我不是在寻找模仿each 的方法或修改我已经做过的代码。另外我已经在这里看到了这个each 仿真答案：stackoverflow.com/questions/46492621/… 我真的很想知道是什么导致了这个谜团和/或如何调试它
合理的方法，如果我能看到任何东西，我会发布另一个答案，但我想不出像你所说的那样会导致 while 循环和 foreach 循环之间的差异。祝你好运。

【解决方案2】：

这实际上是一个非常简单的算法问题，它与当 show_weight() 是 0 时，你循环相同数组（编辑：并基于你的 cmets , show_weight() 本身也循环同一个数组）。

TL;DR 对于while，所有这些循环都共享相同的内部指针并相互影响。使用foreach，每个循环都是独立的，因此它运行的迭代次数更多，因此存在性能问题。

既然一个例子值一千字，希望下面的代码能让事情更清楚：

<?php

$array = ['foo','bar','baz'];

foreach ( array_keys($array) as $key ) {
    echo $array[$key],"\n";
    foreach ( array_keys($array) as $key ) {
        echo "\t",$array[$key],"\n";
    }
}

echo "---------------\n";

while ( list($key,) = each($array) ) {
    echo $array[$key],"\n";
    reset($array);
    while ( list($key,) = each($array) ) {
        echo "\t",$array[$key],"\n";
    }
}

这将输出：

foo
        foo
        bar
        baz
bar
        foo
        bar
        baz
baz
        foo
        bar
        baz
---------------
foo
        foo
        bar
        baz

如您所见，对于大小为 3 的数组，foreach 需要 3² 次迭代，而 while 只需要 3 次。这就是您的性能问题。

为什么`while` 更快？

因为在第二个（内部）while 的末尾，$array 的内部指针将指向数组的末尾，因此第一个（外部）while 将停止。

对于foreach，由于您使用的是对array_keys 的2 次不同调用的结果，因此您正在使用不共享相同内部指针的2 个不同数组，因此没有理由停止循环。在第二个（内部）foreach 之后的一个简单的return 应该可以解决问题。

【讨论】：

谢谢。请注意，我确实尝试将 array_keys 的所有部分更改为 foreach(array_keys($this->contents) as $products_id) { 到 foreach($this->contents as $products_id => $val) { 相同的问题。这会改变你关于while 的理论吗？
是的，请阅读！ show_weight 调用 calculate，后者执行同一数组上的另一个嵌套循环。这正是我在回答中所解释的。对于while，所有这些循环都共享相同的内部指针并相互影响。使用foreach，每个循环都是独立的，因此它会运行方式更多次迭代，因此会出现性能问题。
准确且出色的答案！对于任何将while 循环转换为foreach 循环并认为它是一个相同的插入式替换的人来说，这应该是一个巨大的通知。尤其是现在 php 7.2 出来了。
@rlanvin 在分析中脱颖而出。这有点像嵌套循环： for ($i = 0; $i
上述附加说明：我个人投票反对 each() 弃用，因为您的代码正在运行，我们不需要破坏它。对不起。 -Sara（PHP 7.2 发布管理器）

【解决方案3】：

我不太了解这里的主要问题，但是您可以从使用键值方法优化 foreach 循环开始：

foreach($this->contents as $products_id => $products_value) {
    echo '1 ';
    if (isset($products_value['attributes'])) {
        echo '2 ';
        foreach ($products_value['attributes'] as $value) {
            echo '3 ';
            ...

在 switch 中使用 return 会破坏整个函数，也会退出。如果您想从 switch 语句中中断循环，请使用：

break NUMBER_OF_PARENT_STATEMENTS;

休息 2;将中断 switch 和父 foreach

休息 3;将中断 switch 和第一个和第二个父 foreach

【讨论】：

【解决方案4】：

在您的原始代码中，如果以下在第一次迭代中返回 true：

$this->show_weight() == 0

代码使用each() 循环遍历$this->contents，将$this->contents 上的数组指针设置到末尾。因此，当我们回到第一个 while() 语句时，它假定它已经完成，因为 each($this->contents) 的下一次调用返回 false。

【讨论】：

为什么while 更快？

为什么`while` 更快？