正在更改对象状态/内容适当的迭代器用法（php）

Question

考虑一个用于存储项目集合的对象，但该集合可能会因预定义的上下文而异。

Class Container implements IteratorAggregate (
  protected $contexts; // list of associated contexts, example: array(0=>1,1=>3)
  protected $contents; // array
  public loadContents( $contextId ) { /* populates $this->contents*/ }
  public getContexts() { /* populates $this->contexts */ }
  ...
  public function getIterator() { return new ArrayIterator($this->contents); }
  public getContextIterator() { return new contextIterator($this); }
}

迭代器看起来像：

Class contextIterator {
  protected $container;
  protected $contexts;
  protected $currentContext;

  public function __construct($container) {
    $this->container = $container;
    $this->contexts = $container->getContexts();
    $this->currentContext = 0;
  }

  public current() {
    $this->container->loadContents( $this->key() );
    return $this->contexts[ $this->key() ];
  }

  public function key() { return $this->currentContext; }
  public function next() { $this->currentContext++; }
  public function rewind() { $this->currentContext = 0; }
  public function valid() { return isset( $this->contexts[ $this->key() ] ); }
}

对于需要反复检查每个上下文的少数情况，我执行以下操作：

$myContainer = new Container();
foreach( $myContainer->getContextIterator() as $key => $value ) {
  $myContainer->someMethod();
}

上面的代码很简洁，但我觉得很脏，因为我从来没有真正使用过 $key 或 $value。使用迭代器是否矫枉过正？此外，迭代器是否应该改变它正在迭代的对象的状态/内容？

Answer 1

上面的代码很简洁，但我觉得很脏，因为我从来没有真正使用过 $key 或 $value。

您还没有展示getContextIterator() 的内部结构，因此很难提出具体建议。通常可以通过实现OuterIterator interace 或仅实现Iterator interface 在PHP 中创建可迭代对象。这两个接口都是预定义的，然后您可以将您的对象与next()、foreach 等一起使用。

我假设您已经实现了 OuterIterator 之类的东西。如果您改为实施 OuterIterator，您将获得一些速度优势 AFAIK。

使用迭代器是否过大？

不，不会这么说。正如您所说，迭代器非常适合集合。不过，我只是将其更改为 SPL 迭代器。

此外，迭代器是否应该改变它正在迭代的对象的状态/内容？

实际上每个迭代器都会这样做，至少对于迭代的内部指针是这样。但我认为这不是你关心的问题，但可能已经减轻了。

因此，即使对于您正在迭代的对象内部的“更多”更改，只要清楚它的作用，它就可以更改。反例：如果你迭代一个数组并且每次迭代前一步它都会打乱元素是没有用的。

但在其他情况下，这是完全有效和有用的。所以决定做什么，而不是根据一般规则。