unset heredando de ArrayIterator

[loluchis]

Muy buenas, os traigo algo realmente raro, o no lo logro entender o es un bug o algo raro, ahi voy:

Código:

<?php

class Coleccion extends ArrayIterator
{
    
    public function clear()
    {
	foreach($this as $key => $value)
	    unset($this[$key]);
    }

}

$c = new Coleccion();
$c[] = 'valor1';
$c[] = 'valor2';
$c[] = 'valor...';

echo '<pre>', print_r($c), '</pre>';

/*
  esto imprime:
   Coleccion Object
   (
    [storage:ArrayIterator:private] => Array
        (
            [0] => valor1
            [1] => valor2
            [2] => valor...
        )

   )
   1
 */

$c->clear();

echo '<pre>', print_r($c), '</pre>';

/*
  esto imprime:
   Coleccion Object
   (
    [storage:ArrayIterator:private] => Array
        (
            [1] => valor2
        )

   )
   1
 */

/*
 * SOLUCION:
 * en el metodo clear(), contar elemetos, si aun hay, volver a limpiar
 * 
    public function clear()
    {
	foreach($this as $key => $value)
	    unset($this[$key]);
	if($this->count())
	    $this->clear();
    }
 
 */

es decir, clear no unsetea todos los elementos, sin embargo pasa por los indices, pero por alguna razon el indice 1, no lo unsetea, sabeis por que?, queda muy sucio volver a preguntar si aun hay elementos a count para volver a ejecutar la misma funcion....

PD: aclaro que heredo de ArrayIterator y no de ArrayObject, ya que el primero permite iterarse a si mismo, y el segundo segun he comprobado y leido en php.net itera sobre una copia de si mismo, por lo tanto en este caso concreto, en esta funcion clear, no podria modificar los valores o unsetearlos como es el caso... si podeis aclarar esto magnifico

saludos amigos y muchas gracias

[andresdzphp]

Ese es un pequeño bug conocido ya, aquí encuentras otra alternativa. Un amigo lo reportó pero no se para cuando lo arreglen :D supongo que php 5.4.0

Código PHP:

To avoid this bug you can call offsetUnset in the for loop
 
<?php
/*** ... ***/
for ( $b->rewind(); $b->valid(); $b->offsetUnset( $b->key() ) )
{
/*** ... ***/
?>
 
Or unset it directly in the ArrayObject
<?php
/*** ... ***/
    $a->offsetUnset( $b->key() );
/*** ... ***/
?>
 
which will produce correct results
 

http://www.php.net/manual/es/arrayit...set.php#104789

Saludos.

[loluchis]

muchisimas gracias, eres un crack, pues bien para los interesados, asi quedaria el metodo clear

NOTA: ya no hereda de ArrayIterator, sino de ArrayObject, ya que con el metodo ArrayObject::getIterator() se puede acceder a el como se puede observar:

Código PHP:

      public function clear()
    {
    $iterator = $this->getIterator();

    for($iterator->rewind(); $iterator->valid(); $iterator->offsetUnset($iterator->key()))
        ;
    }  
   

aunque ahora me entra la duda del segundo metodo que plantea 'olav at fwt dot no' aqui:

Código PHP:

 Or unset it directly in the ArrayObject
<?php
/*** ... ***/
    $a->offsetUnset( $b->key() );
/*** ... ***/
?>   

como lo aplicariamos al metodo ahora que heredamos de ArrayObject? ya que sigo pensando que ese for sin instrucciones queda muy sucio... lo siento soy un poco friki.

PD: otra vez gracias al crack

[andresdzphp]

Lo que propone ese tipo es destruirlo directamente con ArrayObject

Código PHP:

class Coleccion extends ArrayIterator
{
}
 
$c = new Coleccion();
$x = new ArrayObject($c);
 
$c[] = 'valor1';
$c[] = 'valor2';
$c[] = 'valor...';
 
foreach ($c as $k => $v) {
    $x->offsetunset($k);
}
 
echo '<pre>', print_r($c), '</pre>';
 

[loluchis]

si, pero suponiendo que ahora en vez de heredar de ArrayIterator, heredamos de ArrayObject

[loluchis]

Código PHP:

  class Coleccion extends ArrayObject
{
    
    public function clear()
    {
    $iterator = $this->getIterator();

    foreach($iterator as $key => $value)
        $this->offsetUnset($key);
    }

}  
   

resultado:

NOTICE Coleccion::clear(): ArrayIterator::next(): Array was modified outside object and internal position is no longer valid

[andresdzphp]

Usando exchangeArray se soluciona ese error, pero sinceramente no sé si es la mejor forma de hacerlo:

Código PHP:

class Coleccion extends ArrayObject
{
    public function clear() {
        $this->exchangeArray($this);
        foreach ($this as $k => $v) {
            $this->offsetUnset($k);
        }
    }
}
 
$c = new Coleccion();
 
$c[] = 'valor1';
$c[] = 'valor2';
$c[] = 'valor...';
 
$c->clear();
 
echo '<pre>', print_r($c, true), '</pre>';
 

[loluchis]

muy grande, muy bueno, muchas gracias, yo tampoco se si sera la mejor manera, pero desde luego es la mas limpia, aunque cuidado con esta clase y las interfaces que implementa, en concreto este metodo, ArrayObject::exchangeArray() ya que simplemente observarlo:

http://www.php.net/manual/es/arrayobject.exchangearray.php

tenemos en las contribuciones de los usuarios que recibe la referencia del array y al cambiar algo en el array referenciado nos lo cambiará en el mismo, en nuestro caso no hay ningun problema ya que el array es $this, pero para confirmarlo, tenemos esta nota de Dmitri

Código:

It seems that input array is always passed by reference.
For example if you have an existing array
$array with some values
then you have an arrayobject $o
and then you do this:

$o->exchangeArray($array);
$o->offsetSet('somekey', 'some value');

Now if you check your $array array, it will have
a key 'somekey' with value of 'some value'

I totally did not expect that, I am sure it was a mistake to pass array by reference by default.

[andresdzphp]

Si es cierto, hay que tener cuidado con eso. Aunque a mi no se me pasa el valor... Quien sabe si lo solucionaron, estoy usando PHP 5.3.8

Código PHP:

<?php
$array = array(1,2,3);
$o = new ArrayObject();
 
$o->exchangeArray($array);
$o->offsetSet('somekey', 'some value');
 
echo '<pre>';
print_r($array);
print_r($o);
echo '</pre>';
 

[loluchis]

buena idea, voy a intentar actualizar php, no tengo ni idea que version exacta estoy usando, el tema es que llevo 2 meses inmerso en linux, ubuntu concretamente y seguramente en pocos dias debian, asi que a buscar comandos:

-para saber la version de php

Código PHP:

  <?php 
 
phpinfo();   

-para actualizar en caso de no estar a la ultima

en cuanto los encuentre los posteo para los que lean el post posteriormente y por seguir el contexto quieran tambien actualizar php en sistemas basados en debian, como en mi caso, ubuntu