Bug #29992 [Com]: foreach by reference corrupts the array

Edit report at https://bugs.php.net/bug.php?id=29992&edit=1 ID: 29992 Comment by: martijn at twotribes dot com Reported by: fletch at pobox dot com Summary: foreach by reference corrupts the array Status: Bogus Type: Bug Package: Scripting Engine problem Operating System: linux PHP Version: 5.0.1 Block user comment: N Private report: N New Comment: To elaborate of why I strongly feel this is a bug and not a 'feature': $a = array('a', 'b', 'c', 'd'); foreach ($a as &$v) { } var_dump($a); One would expect that every element of $a is a string. Well it was, up until I did that foreach-with-reference. That changed the last element into string reference: array(4) { [0]=> string(1) "a" [1]=> string(1) "b" [2]=> string(1) "c" [3]=> &string(1) "d" } The PHP guys can claim that this is correct behavior all they want, but it is fundamentally wrong from a design perspective that an array changes, without doing anything to its elements. Previous Comments: ------------------------------------------------------------------------ [2011-07-13 06:37:58] martijn at twotribes dot com Can someone please promote this "bogus" status to an actual bug? It completely baffles me why something so obviously wrong, has been present in PHP for almost 7(!) years. Come on guys, fix this! ------------------------------------------------------------------------ [2011-07-01 06:41:47] daniel at milde dot cz I agree with Looris. This behaviour is dangerous and no one expects it. I suggest unsetting the variable BEFORE each loop. This should cause no harm. ------------------------------------------------------------------------ [2010-11-27 14:59:24] johannes@php.net I explained this with some pictures in my blog: http://schlueters.de/blog/archives/141-References-and-foreach.html And no, unsetting after the loop is no option as it is inconsistent with other loops and there are valid reasons for keeping it after the loop. Something along the lines of foreach ($array as &$item) { if ($item == "foo") { break; } } $item = "bar"; ------------------------------------------------------------------------ [2010-11-27 10:28:58] tarama at hotmail dot com I highly agree with looris. I had to patch each use of hundreds of foreach loops in my code due to this weird behaviour. The unset() tips at the end of each loop does the job, but from my point of view it's really not logical. It caused really hard to find bugs in my code and I prefer to not imagine how many PHP scritps are running on the web with the same "bug" silently breaking things. ------------------------------------------------------------------------ [2010-05-24 19:20:22] looris at gmail dot com This *IS* a bug, since no one would EVER expect this kind of behaviour. And no one "might use this for some weird reason", believe me. "foreach" means "do stuff FOR EACH element of an array". It does not mean, to any sane person, "do stuff for each element of an array except the last one, and twice for the one before the last one". Your stubbornness in stating this is intentional is quite frightening. ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at https://bugs.php.net/bug.php?id=29992 -- Edit this bug report at https://bugs.php.net/bug.php?id=29992&edit=1