我这段javascript代码身噩梦地方有问题？用基本的算法定位子字符串在父字符串中首次出现的位置？

Question

function find(){ var string1=document.getElementById('string').value; var substring1=document.getElementById('substring').value; for (var i=0;i<string1.length;i++) { document.write(string1[i]+' '); document.write(substring1[i]+' '); if (string1[i]==substring1[0]) { for (var j = 0; j < substring1.length; j++) { if (string1[i+j]!= substring1[j]) { break; }; else if (j==substring1.length-1) { var pos = i; document.write(pos); break; }; }; }; }; }

Answer 1

几个问题：
1. javascript中有现成的API(indexOf)可以获取子串在父串首次出现的位置，具体可以参见链接http://www.w3cschool.cc/js/js-strings.html
2.代码中有明显的语法错误{ break; }; else if (j==substring1.length-1)
else前面“；”是多余的
3.代码应该有一些跳出处理：一旦找到了位置或者子串长度大于父串剩余字符的长度就该终止循环
我改好之后的代码如下：
function find()
{
var string1=document.getElementById('string').value;
var substring1=document.getElementById('substring').value;
var finded = false;
for (var i=0;i if(i > (string1.length - substring1.length)){
break;
}
document.write(string1[i]+' ');
document.write(substring1[i]+' ');
if (string1[i]==substring1[0]) {
for (var j = 0; j < substring1.length; j++) {
if (string1[i+j]!= substring1[j]) {
break;
}else if (j == substring1.length - 1) {
var pos = i;
finded = true;
document.write(pos);
break;
};
};
};
if(finded){
break;
}
};
}