0

我写了以下内容：

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

该函数旨在返回一个数组（“scores”），其中数组的每个索引是在“pages”数组的每个索引中找到字符串 s 的次数，不包括方括号内的内容 - 但是，只找到每个单词中的字符串 s 一次。所以理想情况下，分数的第一个索引应该是 1，因为我已经用字母 w 调用了函数，我只希望它在页面的第一个索引中找到“WWW”的第一个 w - 如果这有意义的话。

走到这一步，我把自己搞糊涂了，所以我不知道为什么函数返回“,,,”，而不是每个分数索引的数值——有什么想法吗？

谢谢

4

5 回答 5

3

当您的for循环退出时，i等于words.length，它比 的最后一个索引大一indexScore。您scores[i]每次都没有分配任何内容。

于 2009-11-08T23:39:57.620 回答
2

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

于 2009-11-08T23:39:29.727 回答
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”问题之后）是 score[i] 应该设置在内循环之外。如果 cosing parens 被分离到单独的行上，而不是像“ scores[i] =indexScore[i]}};”，这会更清楚。

事实证明，该变量indexScore不是必需的。这使我能够scores[i]进入内部循环以直接累积单词命中。

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

var pages = [...];
function findScoresC(pages, s)
{
   ...
}
alert(findScoresC(pages, "w"));
于 2009-11-09T00:13:36.070 回答
0

here's a small function that counts how many times substring "subStr" occurs in "str", not counting [...]

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

the rest is obvious ;)

// edit: this is how you apply this function to an array

 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;
于 2009-11-08T23:50:40.180 回答
0

这是你的功能固定。它返回[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"));
于 2009-11-09T00:24:46.273 回答