Баг пхп с clone или я что-то не допонимаю?

[HraKK]

Баг пхп с clone или я что-то не допонимаю?

class a implements Iterator
{
    protected $data = array('a','b','c');

    public function next()
    {
        return next( $this->data );
    }

    public function key()
    {
        return key( $this->data );
    }

    public function current()
    {
        return current( $this->data );
    }

    public function rewind()
    {
        return reset( $this->data );
    }

    public function valid()
	{
		return !is_null( key( $this->data ) );
	}
}
$a = new a();
print $a->current(); //a
print $a->next(); //b
print $a->next(); //c
print $a->next(); //false
print $a->current(); //false
$b = clone $a;
print $a->current(); // a WTF o_O
die();

-~{}~ 29.03.10 18:47:

Вдогонку, это нормальное поведение?

$a = array( 1,2,3);
print next($a); // 2
print next($a); // 3
print next($a); // false
print prev($a); // false O_o

 

[.des.]

Разработчики PHP веселые люди - поэтому считай это нормальным поведением

забавно следующее
этот bug report
http://bugs.php.net/bug.php?id=27782
закрыт со статусом "fixed in cvs"

этот поновее
http://bugs.php.net/bug.php?id=40028
Bogus "not a bug"

 

[boombick]

Насчет первого, может можно решить с помощью magic-метода __clone? Ну принудительно передать в контекст клонируемого объекта все значения?

А насчет второго сам прифигел

 

[grigori]

похоже, clone ресетит указатель
остается только не полагаться на встроенный указатель массива

 

[HraKK]

boombick
Все оно передает, проблема в том что оно ресетит указатели у КЛОНИРУЕМОГО объекта. В случае если он достиг конца - false. Это продолжение бага №2.

-~{}~ 29.03.10 19:51:

grigori
Угу, придется переписывать класс. Капец, я <3 php.

 

[boombick]

Капец, я <3 php.

Мы все <3 php

 

[grigori]

открой тикет, интересно что ответят

 

[HraKK]

Не знаю как, что и где. Смысла мне это все равно не добавит.

 

[boombick]

grigori
ну так выше приводили ссылки на багтрекер Дропнули с комментом, что это не баг Или ты предлагаешь открыть баг про clone?

 

[grigori]

да, я про clone, конечно

 

[Ирокез]

С первым вполне правильно (согласно коду zend engine), так как при копировании HashTable, указатель текущего значения не копируется (т.е. у $b current будет первый элемент). Соответственно чтобы перебрать элементы $a используют внутренние значение итератора HashTable $a и его не восстанавливают после копирования