javascript - 天真的字符串匹配算法出了问题

Question

我正在尝试实现简单搜索，但没有得到预期的结果。任何人都可以在这里指出可能的错误是什么。

function naive(string, str) {
    for (let i = 0; i <= string.length - str.length; i++) {
        if (string[i] == str[0]) {
            let counter = 1

            for (let j = 1; j <= str.length; j++) {
                if (string[i + j] == str[j]) {
                    counter++;
                    console.log(counter)
                } else
                    counter = 0;
            }

            if (counter == str.length) {
                console.log(`${counter} pattern matched at ${i}`)
            }
        } else
            console.log('nothing matched')
    }
}

Answer 1

var match_found = false;
function naive(string, str){
  for(let i =0; i <= string.length - str.length; i++){
      if(string[i] == str[0]){
        let counter= 1
            for(let j = 1; j < str.length; j++){
                if(string[i + j] == str[j]){    
                  counter++;
                }else{
                  break;
                }
            }
        if(counter == str.length){ 
          console.log('Pattern matched at ' + i);
          match_found = true;// can break; here if you wish to, else it will give you all matches present
        }
      }
  }
  if(match_found === false){
    console.log(str + ' not found in ' + string);
  }
}

naive('abcdgggabcdggg','ggg');

• 你 increment counter当有比赛时，但你需要break有 mismatch 的循环.

• 您的内部 for 循环条件需要有 j < str.length而不是 j <= str.length , 因为索引从 0 开始.

• else console.log('nothing matched') .你不能 just instantly决定那个。如果一个字符串索引不匹配，您仍然需要继续寻找其余的索引。最好的方法是为它维护一个 bool 标志，如上面的代码所示。