任何人都可以看到我的Javascript有什么问题吗？

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")); 

有几件事情。我用“j”替换了“i”作为内部索引。关闭后，你不需要分号。指示后你应该有一个分号（一对夫妇失踪）。

可能主要问题（在“我”问题之后）是分数[i]应该设置在内部循环之外。如果将接合部分分离成单独的线而不是像“scores[i] =indexScore[i]}};”那样，这将更清楚。

原来变量indexScore不是必需的。这使我可以在内部循环内部带上scores[i]来直接累计单词匹配。

最后，我倾向于将pages变量作为参数传递给函数，而不是假设它在全局空间中可用。如果可以的话，我倾向于避免使用全局变量。

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

Answer 2

这可能是因为你有一个嵌套for循环与相同的索引变量。

Answer 3

当您的for环路退出时，i等于words.length，这比最后一个指数indexScore大1。您每次都没有分配scores[i]。

Answer 4

这里的一个小功能，计数多少次串 “SUBSTR” 发生在 “海峡”，不计[...]

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

其余为明显;）

//编辑：这是你如何运用此功能的阵列

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"));