foreach写失效的问题

本文由做者张远道受权网易云社区发布。

坦白讲身为程序员，bug在所不免。有人讲，bug越多，说明程序员越伟大。这句话有它必定的道理。

由于从某方面讲，bug多了说明他的代码量也多。

言归正传，这里我记录了我曾经犯过的几个错误。但愿看到的同侪可以见而避之。

经常使用的一个场景，遍历一个集合，对符合某种条件的元素作修改。习惯性地会写出以下代码:

List testInt = new ArrayList();
 testInt.add(1);
 testInt.add(2);
 testInt.add(3);     for(Integer temp :testInt ){      if(temp==1)
     temp=temp*2;
 }   
 for(Integer a:testInt ){
  System.err.println(a);
 }

期待的结果是:

2

2

3

但实际输出为:

1

2

3

这是很容易掉进去的陷阱。即经过foreach遍历对集合元素进行修改。在觉得变动已发生的时候，

其实变动没有发生。形成数据写入失败。

由于

for(Integer temp:testInt){     if(temp==1)
    temp=temp*2;
}

将被翻译成

for(int i=0,length=testStr.size();i<length;i++){

    Integer temp = testStr.get(i).clone();

    if(temp==1)

    temp=temp*2;

}

根据oracle的官方文档,正式翻译应该以下

for (Iteratori = testInt.iterator(); i.hasNext(); ) {            float i0 = (Integer)i.next();            if(i0 == 1)
            i0 = i0*2;
    }

即，foreach里头的的 temp变量只是一个局部变量，并且仍是集合中元素的一个副本，并非元素自己。

想到以前还遇到的一个问题，代码简化以下:

Integer integer1 = 3;
    Integer integer2 = 3;        if (integer1 == integer2)
        System.out.println("integer1 == integer2");        else
        System.out.println("integer1 != integer2");

    Integer integer3 = 300;
    Integer integer4 = 300;        
    if (integer3 == integer4)
        System.out.println("integer3 == integer4");        else
        System.out.println("integer3 != integer4");

即在判断整数相等时,使用了封装类(由数据库映射过来，用封装类防止反射异常)。实际的输出结果以下:

integer1 == integer2

integer3 != integer4

明眼人很容易看出来，这里掉入了两个坑.一个坑是用等号判断相等,除非是为了比较同一个对象，等值比较不该该直接用等号。 另外一个坑是

java的整数缓存。

查看jdk的源码以下:

private static class IntegerCache { static final int low = -128; static final int high; static final Integer cache[]; static { // high value may be configured by property

int h = 127;
        String integerCacheHighPropValue =
            sun.misc.VM.getSavedProperty("java.lang.Integer.IntegerCache.high");            if (integerCacheHighPropValue != null) {                int i = parseInt(integerCacheHighPropValue);
            i = Math.max(i, 127);                // Maximum array size is Integer.MAX_VALUE
            h = Math.min(i, Integer.MAX_VALUE - (-low));
        }
        high = h;

        cache = new Integer[(high - low) + 1];            int j = low;            for(int k = 0; k < cache.length; k++)
            cache[k] = new Integer(j++);
    }        private IntegerCache() {}
}

即整数缓存缓存了前127个整数，没有从新生成。

固然，还遇到其它各类各样的坑。可怕的不是掉入坑中，而是掉入坑里了不正视问题也不查找问题所在，一而再再而三地掉进坑里。

免费领取验证码、内容安全、短信发送、直播点播体验包及云服务器等套餐

更多网易技术、产品、运营经验分享请访问网易云社区。

文章来源： 网易云社区