Question

在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然后再重写代码，因为我们不会从中学到很多东西。我只是想找到一个合理的解释或一种解决/调试此谜的方法。也许有人在某个地方碰到了这样的问题，可以对此有所了解。

UPDATE 01 / Aug / 2018

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

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的{{1}}循环和while循环之间是否有任何区别，但我仍然找不到任何支持信息。然后，我将最后一个foreach改写为break;，并再次测试了break 2;，这次似乎只运行了一次，就像它是foreach循环时一样while（不是break;）编辑：只是要澄清-break 2;和foreach break之间没有区别。它们的工作方式相同。

更新＃2： 我已经将break修改为} elseif ($this->show_weight() == 0) {，并且} elseif (2 == 0) {循环现在运行的次数与while循环一样多。 foreach个结果var_dump($this->show_weight());。这个问题对我来说仍然毫无意义。

再次感谢

Answer 1

这实际上是一个非常简单的算法问题，它与以下事实有关：show_weight()为0时，您循环相同的数组（编辑：和根据您的评论，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个不共享相同内部指针的不同数组，因此没有理由循环停止。在第二个（内部）return之后使用简单的foreach应该可以解决问题。

Answer 2

作为备份解决方案，执行相同操作的替代品减少了吗？

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;
}

Answer 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 ';
            ...

使用return in switch将破坏整个功能，也退出。如果要中断switch语句的循环，请使用：

break NUMBER_OF_PARENT_STATEMENTS;

休息2；会破坏开关和父foreach

休息3；会中断开关，而第一个和第二个父级foreach

Answer 4

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

$this->show_weight() == 0

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

尽管不推荐使用，但替换Foreach会花费更多时间

4 个答案:

为什么`while`更快？

尽管不推荐使用，但替换Foreach会花费更多时间

4 个答案:

为什么while更快？

为什么`while`更快？