The docs say "Check whether the doubly linked list contains more nodes". I do not believe this is correct.
Example:
$dlist = new SplDoublyLinkedList;
$data=[1,2,3,4,5];
foreach($data as $d)
$dlist->push($d);
$dlist->rewind();
for($i=0;$i<6;$i++)
{
$currentValue = $dlist->current();
$currentValid = $dlist->valid();
$status = $currentValid ? 'True' : 'False';
echo 'Current value is: ' . $currentValue . ' Valid status is: ' . $status . "\n";
$dlist->next();
}
Output:
Current value is: 1 Valid status is: True
Current value is: 2 Valid status is: True
Current value is: 3 Valid status is: True
Current value is: 4 Valid status is: True
Current value is: 5 Valid status is: True
Current value is: Valid status is: False
Note that when we are on the last node of the list, value = 5, the valid() function returns true. Yet we are on the last node of the list, and there are no more nodes. If the valid() function were checking to see if there were any more nodes on the list, it would return false, not true.
If you look at the docs for Iterator::valid, they say "Checks if current position is valid". I believe that is in fact what valid() does, it checks to see if the *current* position is valid, *not* if there are any more nodes.
Be aware of this or it will bite you. You can happily iterate to the end of the list, run valid(), think there is one more node, do a next(), grab the value, and you get null instead of the last node of the list.
