最近在浏览同事之前写的代码的时候,发现大量的变量引用使用在 foreach 中:
1 <?php
2 $a = array( ‘a‘, ‘b‘, ‘c‘, ‘d‘);
3 foreach($a as &$v) {
4 $v = $v . ‘_bar‘;
5 }
6 var_dump($a);
这种写法有个好处,就是可以直接通过改变引用变量 $v 的值,来直接改变数组的值。但是显然,这里会带来一个问题:
1 <?php
2 $a = array( ‘a‘, ‘b‘, ‘c‘, ‘d‘);
3 foreach($a as &$v) {
4 $v = $v . ‘_bar‘;
5 }
6 $v = ‘foobar‘; 7 var_dump($a);
通常 foreach 变量都只希望是一个作用域在 foreach 内部的局部临时变量。但是当不慎在后面用了同名的临时变量时,数组的内容会不预期的被改变。就如同上面的代码那样,最终结果 $a[3] = ‘foobar’ 而不是预期的 $a[3] = ‘d_bar’。这对于程序的正确执行时一个隐患,不能让程序的正确性维系在程序员足够细心的不使用重名临时变量,这一线上。
解决方法有两种。
第一种,直接使用数组的键值来修改数组内容:
1 <?php
2 $a = array( ‘a‘, ‘b‘, ‘c‘, ‘d‘);
3 foreach($a as $k => $v) {
4 $a[$k] = $v . ‘_bar‘;
5 }
6 $v = ‘foobar‘;
7 var_dump($a);
第二种,注销掉 foreach 中使用的临时变量:
1 <?php
2 $a = array( ‘a‘, ‘b‘, ‘c‘, ‘d‘);
3 foreach($a as &$v) {
4 $v = $v . ‘_bar‘;
5 }
6 unset($v);
7 $v = ‘foobar‘;
8 var_dump($a);
November 25th, 2009 at 10:36
[...] PHP —— 为什么不要在 foreach 中使用变量引用 [...]