foreachでIndirect modification of overloaded propertyが発生する

マジックメソッドの__get()ですが、配列を返すとforeachでNoticeが発生する場合があります。

< ?php
error_reporting(E_ALL);

class A {
  private $array = array(1, 2, 3);

  function __get($name) {
    if ($name == 'hoge') {
      return $this->array;
    }
  }
}

$obj = new A();
foreach ($obj->hoge as $elem) {
  var_dump($elem);
}
?>

PHP Notice:  Indirect modification of overloaded property A::$hoge has
 no effect in __get.php on line 15

The reason for this is that __get() only returns variables in read mode, while foreach() wants a variable in read/write mode as it tries to modify the internal array pointer. As it can’t do this PHP 5.2 will now throw a warning on this.

Overloaded properties (__get) – Derick Rethans

理由はforeachではread/writeモードが要求されるのに__get()ではreadモードの値が返されるためのようです。__get()がreadモードで返すのは当然のような気がしますがちと不便ですね。

対策としては__get()内で(array)でキャストするか

< ?php
  function __get($name) {
    if ($name == 'hoge') {
      return (array)$this->array;
    }
  }
?>

__get()の値を別の変数に代入すれば良いようです。

< ?php
$array = $obj->hoge
foreach ($array as $elem) {
?>

この現象は5.2.0で発生し、5.1.6ではNoticeはでませんでした。__get()は便利で何だか使いたくなる機能ですが、配列を返す場合は注意しましょう。

Tweet

Pocket