请高手指点！
_Jack_Han_
2016-05-13 20:20
<!DOCTYPE HTML>
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=gb18030">
<title>Untitled Document</title>
</head>
<body>
<script type="text/javascript">
/*
* param1 Array
* param2 Array
* return true or false
*/
function arraysSimilar(arr1, arr2){
if(!(arr1 instanceof Array)||!(arr2 instanceof Array)){return "判定结果：不通过";}
if(arr1.length!=arr2.length){return "判定结果：不通过"; }
var count=0;
for(var i in arr1){
for(var j in arr2){
if(typeof arr1[i] ==typeof arr2[j]){count++;}
}
}
if(count==arr1.length){return "判定结果：通过";}else{return "判定结果：不通过"}
}
document.write(arraysSimilar([1,true,null],[null,false,100]));
</script>
<script src="testData.js"></script>
</body>
</html>
我的篇幅比课件答案简短很多，是否代表我的代码功能不如课件答案的功能完善呢？我有哪些方面的bug呢？谢谢。
1回答
-
- 不再热情似火
- 2016-05-14 11:10:53
你这bug有点小多，推荐你去看下同学代码第一和第二的，1. typeof 这是不对的，typeof 只能判断基本类型，但是你的数组中可能会有引用类型，而且 typeof null也会返回Object ,所以你要先封装一个判断类型的函数，可用Object.prototype.toString.call()的方法，（在IE6/7/8下遇到null和undefined会失效，返回Object）. 2. 你用的 instanceof 在页面存在框架时，无法进行准确的判断，所以最好 数组的判断用Array.isArray(), 3. for in 对数组进行循环时，顺序是不固定的，你的 if(typeof arr1[i] ==typeof arr2[j]) 几乎不会进行这个if 条件内的语句，你需要先用sort()方法，对数组进行排序，然后再进行比较 4. 你的count == arr1.length 貌似是多余的吧
JavaScript深入浅出
281101 学习 · 1020 问题
相似问题