|
php.net | support | documentation | report a bug | advanced search | search howto | statistics | random bug | login |
[2012-09-24 18:44 UTC] maciej dot sz at gmail dot com
Description:
------------
When I unset more then one element while iterating through SplDoublyLinkedList object it throws an OutOfRangeException exception. Im guessing that this is because each time an the offsetUnset() method is called, the keys of the list are reset. This should not be happening until the loop is rewind()'ed.
Test script:
---------------
$List = new SplDoublyLinkedList();
$List->push('a');
$List->push('b');
$List->push('c');
$List->push('d');
foreach ( $List as $key => $value ) {
echo "Current element: key = {$key}, value = '{$value}'. ";
if ( in_array($value, ['b', 'd']) ) {
echo "MATCH! performing unset";
unset($List[$key]);
}
echo "\n";
}
Expected result:
----------------
The elements should be removed from the list, and no exception should be thrown. At the very least it should act as the ArrayObject object: triggers a notice, but gets the job done. But the perfect solution would be the way ArrayIterator does it - clean, no errors:
$Arr = new ArrayIterator(['a', 'b', 'c', 'd']);
foreach ( $Arr as $key => $value ) {
echo "Current element: key = {$key}, value = '{$value}'. ";
if ( in_array($value, ['b', 'd']) ) {
echo "MATCH! performing unset";
unset($Arr[$key]);
}
echo "\n";
}
var_dump($Arr->getArrayCopy());
Actual result:
--------------
OutOfRangeException
PatchesPull RequestsHistoryAllCommentsChangesGit/SVN commits
|
|||||||||||||||||||||||||||
Copyright © 2001-2025 The PHP GroupAll rights reserved. |
Last updated: Fri Nov 07 23:00:01 2025 UTC |
This is not a bug. However, the expected work-around would be: <?php $list = new SplDoublyLinkedList(); $list[] = 0; $list[] = 1; $list[] = 2; $list[] = 3; for ($list->rewind(); $list->valid(); ) { if (in_array($list->current(), array(0, 2))) { $list->offsetUnset($list->key()); } else { $list->next(); } } ?> However, this does not work. I am not entirely sure which function/functions is/are faulty here.