PHP 变量作用域和“foreach”

Question

我的应用程序正在构建 PDF 文档。它使用脚本来生成每个页面的 HTML。 PDF 生成类是“Production”，页面类是“Page”。

class Production
{
  private $_pages; // an array of "Page" objects that the document is composed of

  public getPages()
  {
    return $this->_pages; 
  }

  public render()
  {
    foreach($this->_pages as $page) {
      $pageHtml = $page->getHtml($this); // Page takes a pointer to production to access some of its data.        
    }
  }
}

以下是 Page 类摘要：

class Page 
{
  private $scriptPath; // Path to Script File (PHP)

  public function getHtml(Production &$production)
  {
    $view = new Zend_View();
    $view->production = $production; 
    return $view->render($this->scriptPath); 
  }

}

我在对目录进行编码时遇到了问题。它访问 Production，获取所有页面，查询它们，并根据页面标题构建 TOC：

// TableOfContents.php 
// "$this" refers to Zend_View from Pages->getHtml();
$pages = $this->production->getPages();
foreach($pages as $page) {
  // Populate TOC
  // ...
  // ...
}

发生的情况是 TableOfContents.php 中的 foreach 干扰 Production 中的 foreach。生产 foreach 循环在索引页（实际上是文档中的第二页，在封面页之后）终止。

文档布局如下：

1）封面

2）目录

3）页面 A

4）页面 B

5）页面 C

TableOfContents.php 在其 foreach 循环中，根据需要遍历页面并构建整个页面的索引文档，但 Production 中的循环在目录处终止，并且不会继续渲染页面 A、B 和 C。

如果我从 TableOfContents.php 中删除 foreach，则所有连续页面都会正确渲染。

我觉得这是指针和变量作用域的问题，那么我该怎么解决呢？

Answer 1

诊断

我怀疑问题是 $_pages 不是一个普通的 PHP 数组，而是一个恰好实现 迭代器接口。因此，foreach 循环的“状态”存储在对象本身上，这意味着两个循环是冲突的。

如果 $_pages 是一个普通数组，那么就不会有问题，因为 $pages = $this->product->getPages(); 行会使复制，因为 PHP 数组是在赋值时复制的（与对象不同），而且还因为普通数组上的嵌套 foreach 循环不存在这个问题。 （大概来自一些内部数组复制/分配逻辑。）

解决方案

“快速而肮脏”的修复是为了避免 foreach 循环，但我认为这既令人烦恼，又会成为未来错误的原因，因为很容易忘记 $_pages 需要超级特殊的处理。

对于真正的修复，我建议查看 $_pages 中对象后面的类，看看是否可以更改该类。不要让 $_pages 成为迭代器，而是更改 $_pages 以便它提供 > 通过 IteratorAggregate 的迭代器界面。

这样，每个 foreach 循环都需要一个单独的迭代器对象并维护单独的状态。

下面是一个示例脚本来说明该问题，部分抄袭自 PHP 参考页：

<?php
class MyIterator implements Iterator
{
    private $var = array();
    public function __construct($array)
    {
        if (is_array($array)) {
            $this->var = $array;
        }
    }
    public function rewind()
    {
        reset($this->var);
    }
  
    public function current()
    {
        $var = current($this->var);
        return $var;
    }
  
    public function key() 
    {
        $var = key($this->var);
        return $var;
    }
  
    public function next() 
    {
        $var = next($this->var);
        return $var;
    }
    public function valid()
    {
        $key = key($this->var);
        $var = ($key !== NULL && $key !== FALSE);
        return $var;
    }
}

// END BOILERPLATE DEFINITION OF ITERATOR, START OF INTERESTING PART

function getMyArrayThingy(){
    /* 
     * Hey, let's conveniently give them an object that
     * behaves like an array. It'll be convenient! 
     * Nothing could possibly go wrong, right?
     */
    return new MyIterator(array("a","b","c"));  
}


// $arr = array("a,b,c"); // This is old code. It worked fine. Now we'll use the new convenient thing!
$arr = getMyArrayThingy();

// We expect this code to output nine lines, showing all combinations of a,b,c 
foreach($arr as $item){
        foreach($arr as $item2){
                echo("$item, $item2\n");
        }
}
/* 
 * Oh no! It printed only a,a and a,b and a,c! 
 * The outer loop exited too early because the counter
 * was set to C from the inner loop.
 */

Answer 2

我不确定你的问题是什么，但你可以看看 PHP 函数 reset< /a> =)

Answer 3

解决方案是避免使用 foreach 并使用传统循环，如下所示：
PHP 中的嵌套 foreach 问题