发现问题
大家在码代码的过程中，经常会遇到在循环中移除指定元素的需求。按照常规的思路，直接一个for循环，然后在循环里面来个if判断，在判断中删除掉指定元素即可。但是实际情况往往不会像预想的那样顺利运行。
下面以一段Javascript代码为例演示这一过程。
|
1
2
3
4
5
6
7
8
9
10
11
12
13
|
(function () {
var arr = [1,2,2,3,4,5];
var len = arr.length;
for(var i=0;i<len;i++){
//打印数组中的情况，便于跟踪数组中数据的变化
console.log(i+"="+arr[i]);
//删除掉所有为2的元素
if(arr[i]==2){
arr.splice(i,1);
}
}
console.log(arr);
})(); |
运行结果如下：
从最终的结果可以看到实际上只删除掉了匹配的其中一个元素，而另外一个元素还存在。
从打印出的运行过程不难发现，原因为当删除掉了一个元素后，数组的索引发生的变化，造成了程序的异常。
解决方法
找到了问题的原因，就不难解决问题了。
方法一
|
1
2
3
4
5
6
7
8
9
10
11
12
13
14
|
(function () {
var arr = [1,2,2,3,4,5];
var len = arr.length;
for(var i=0;i<len;i++){
//打印数组中的情况，便于跟踪数组中数据的变化
console.log(i+"="+arr[i]);
//删除掉所有为2的元素
if(arr[i]==2){
//注意对比这行代码：删除元素后调整i的值
arr.splice(i--,1);
}
}
console.log(arr);
})(); |
上面的代码看起来不大好理解，有没有看起来更易于理解的代码呢？请看下面
方法二
|
1
2
3
4
5
6
7
8
9
10
11
12
|
(function () {
var arr = [1,2,2,3,4,5];
var len = arr.length-1;
//start from the top
for(var i=len;i>=0;i--){
console.log(i+"="+arr[i]);
if(arr[i]==2){
arr.splice(i,1);
}
}
console.log(arr);
})(); |
从后往前遍历可以有效解决问题，也容易理解，那么还有没有跟简洁的实现呢？接着看下面代码
方法三
|
1
2
3
4
5
6
7
8
9
10
11
|
(function () {
var arr = [1,2,2,3,4,5];
var i = arr.length;
while(i--){
console.log(i+"="+arr[i]);
if(arr[i]==2){
arr.splice(i,1);
}
}
console.log(arr);
})(); |
使用while(i--) ,i为数组下标，个人觉得这是最简洁、高效的代码实现了。