论坛里有同学问了一个比较基础的js数组处理,就是在一个乱序且有重复值的数组中,找到最大的三个值。在回答这个问题过程中,因为没有实际测试,我连续犯了几个低级错误,现在记录下来,给自己一个警醒.
第一次回答的代码:
function getMax(list, num){
var result = [];
list.sort().reverse().foreach((p)=>{
if(result.lenth < num){
if(result.indexOf(p) != -1){
result.push(p);
}
}
})
return result;
}
提交后,立马发现判断语句写反了,应该用等号,修改之后再提交,又发现forEach敲错了,大写的E敲成了小写的,并且length少敲了一个g。。。
然后想想,做了一个测试,发现返回结果是空数组,立马反应过来,forEach里面封装的是匿名函数,异步执行导致的,唉。。。。
一个小问题,出了几个低级错误。。。。真丢脸。。。。
后来把forEach遍历改成for循环遍历,得到了正确的结果。
不过心里始终有个疙瘩,现在lambda语法很流行,forEach比for要优雅一些,有没有办法用forEach也能得到需要的结果呢?
于是继续改造,给forEach添加await语法,强制同步。虽然代码看起来有些奇怪,但也能解决问题,并且可以看做是学习js基础语法的一个良好例题吧:
function getMax(list, num){
var result = [];
list.sort().reverse().forEach(async (p)=>{
if(result.length < num){
if(result.indexOf(p) == -1){
await result.push(p);
}
}
})
return result;
}