一个应该忏悔的代码片段

当然不一定是cvs，但终究要用一种版本控制系统吧，每次提交时写些comment标明本次修改的意图，
然后出现楼主这种情况时只要查看资源历史记录就一目了然了，eclipse、fisheye都支持得很好，可以精确定位到代码行。例如：
https://fisheye.springsource.org/browse/springframework/trunk/src/org/springframework/aop/support/JdkRegexpMethodPointcut.java?r=17120
https://fisheye.springsource.org/browse/springframework/trunk/src/org/springframework/aop/support/JdkRegexpMethodPointcut.java?r1=13709&r2=17120

很可能是
当时msisdn里面没数据
又懒的加测试用数据.
还想要把这个程序页面展现出来

这一般是不作测试用例常见的问题

如果是msisdn里面没数据。注释不注释都不执行for里面的代码。
我认为是那个if里面永远是false，不会执行if里面的代码。也就是每天的文件里都会有msisdn数据，因此加这个if就没什么必要。所以注释了，不知道我的推理对不对。

this.addMsisdnToCache(msisdnArray[i]);  我不知道你这是不是一个缓存, 如果是,说实话我很害怕别人写这种代码.因为我无法保证msisdnArray这个东西 外部调用的时候会不会进行重新赋值,如果会 那么我加到缓存里面的都是最后一次的外部赋值.所以建议msisdnArray里面的东西要实现克隆接口.

一般不加comment的，提交的内容应该被打回。
如果提交的东西导致编译失败的，应该被打回。

楼主的这段代码，代码中没有明显的说明注释原因，注释者，注释时间。
提交的时候也不加任何说明，当属被打回之列。

原来在js里写了很多注释，现在要求全部删除，说是为了减小文件大小。