누구든지 내 자바 스크립트가 잘못된 것을 볼 수 있습니까?

Question

내가 쓴 다음

var pages=["[www.google.co.uk] This is the WWW. ","[www.yahoo.co.uk] This is also the WWW. "]; 
function findScoresC(s){ 
var scores=[]; 
var words=[]; 
var wordScore; 
var indexScore=[]; 
s=s.toLowerCase(); 
for(i=0;i<pages.length; i++){ 
    var lowerCaseContents=(pages[i].substring(pages[i].indexOf("]")+1,pages[i].lastIndexOf(" "))).toLowerCase(); 
    words=lowerCaseContents.split(" "); 
    for(i=0;i<words.length;i++){ 
    if(words[i].match(s)){ 
     wordScore=1; 
     indexScore[i]=indexScore[i]+1}; 
    scores[i] =indexScore[i]}}; 
return scores; 
} 
alert(findScoresC("w")); 

함수는 배열의 각 인덱스는 문자열 s는 "각 페이지의 색인에서 발견되는 횟수입니다 ("점수 ") 배열을 반환하는 것을 목표로

"배열은 대괄호 안의 것을 제외하고 각 단어 안에 한 번만 문자열을 찾습니다. 그래서 이상적인 첫 글자 색인은 1이 될 것입니다. 왜냐하면 저는 w라는 문자로 함수를 호출했기 때문에 페이지의 첫 번째 색인에서 "WWW"의 첫 번째 w를 찾는 것이 좋을 것입니다.

필자는이 점을 상당히 장황하게 혼란스러워했기 때문에 점수의 각 지수에 대한 수치가 아닌 ",,,,"함수가 왜 돌아 오는지 알 수 없다.

감사

Answer 1

var pages=["[www.google.co.uk] This is the WWW. ","[www.yahoo.co.uk] This is also the WWW. "; 

function findScoresC(s){ 
    var scores=[]; 
    var words=[]; 
    s=s.toLowerCase(); 
    for(i=0;i<pages.length; i++) 
    { 
    scores[i]=0; 
    var lowerCaseContents=(pages[i].substring(pages[i].indexOf("]")+1,pages[i].lastIndexOf(" "))).toLowerCase(); 
    words=lowerCaseContents.split(" "); 
    for(j=0;j<words.length;j++) 
    { 
     if(words[j].match(s)) 
     { 
      scores[i] += 1; 
     } 
     } 
    } 
    return scores; 
} 
alert(findScoresC("w")); 

에이 기능을 적용하는 방법입니다. 나는 내부 색인을 위해 "i"를 "j"로 바꿨다. 닫는 괄호 뒤에 세미콜론을 사용하지 않아도됩니다. 지침 (한 쌍이 누락 된 경우) 후에는 세미콜론을 사용해야합니다.

아마도 "i"문제 이후의 주요 문제는 점수 [i]가 내부 루프 바깥에 설정되어 있어야한다는 것입니다. 이것은 "scores[i] =indexScore[i]}};"대신에 별도의 줄로 구분 된 괄호가 분리되어 있으면 더 명확 해졌습니다.

변수 indexScore이 필요하지 않은 것으로 나타났습니다. 그건 내 안쪽 루프에 scores[i]을 가져 와서 단어 히트를 직접 축적 할 수있게 해 줬다.

마지막으로 글로벌 공간에서 사용할 수 있다고 가정하는 것보다 pages 변수를 인수로 함수에 전달하는 것이 좋습니다. 내가 할 수 있으면 전역을 피하는 경향이 있습니다.

var pages = [...]; 
function findScoresC(pages, s) 
{ 
    ... 
} 
alert(findScoresC(pages, "w")); 

Answer 2

같은 인덱스 변수와 루프 중첩을 가지고 있기 때문에 그것은있을 수 있습니다.

Answer 3

for 루프가 종료되면 i은 words.length과 같으며 마지막 인덱스 인 indexScore보다 1입니다. 매번 scores[i]에 아무 것도 지정하지 않습니다.)

// 편집,

Answer 4

여기에 [...]

function substrCount(str, subStr) { 
    var str = str.replace(/\[.+?\]/g, ""); 
    var del = str.toLowerCase().split(subStr.toLowerCase()).join(""); 
    return (str.length - del.length)/subStr.length; 
} 

나머지는 명백하다는 계산하지, "SUBSTR을"부분 문자열 얼마나 많은 시간을 계산 작은 함수의 "STR"발생 : 이것은 당신이 몇 가지가 있었다 배열

var someArray = ["whatever", "something", "else" ]; 
var counter = []; 
for(var i = 0; i < someArray; i++) 
     counter[i] = substrCount(someArray[i], "something"); 
     // or, to count only one match, i.e. just to test if a substring is present 
     counter[i] = substrCount(someArray[i], "something") > 0; 

Answer 5

다음은 수정 된 기능입니다. 그것은 당신이하려고했던 것처럼 보이는 [1,1]을 반환합니다. 내 노트가 코드에 있습니다.

var pages=["[www.google.co.uk] This is the WWW. ","[www.yahoo.co.uk] This is also the WWW. "]; 

function findScoresC(s){ 
    var scores = [], 
     words = [], 
     wordScore; 
     // indexScore = [] <- this doesn't seem necessary 
    s = s.toLowerCase(); 

    // Make sure to use `var i` and not just `i`; otherwise, you are creating a global variable. 
    for (var i=0; i<pages.length; i++) { 
     // Initialize me! 
     scores.push(0); 

     var lowerCaseContents = pages[i].substring(
      pages[i].indexOf("]") + 1, pages[i].lastIndexOf(" ") 
     ).toLowerCase(); 
     words = lowerCaseContents.split(" "); 

     // You were using `i` for this loop as well. No can do. 
     for (var j=0; j<words.length; j++) { 
      if (words[j].match(s)) { 
       // wordScore = 1; <- I don't know what you're using this for 
       scores[i]++; 
      } 
     } 
    }; 

    return scores; 
} 

console.log(findScoresC("w"));