indexOf（）：是否有更好的方法来实现呢？

debugcn 发表于 Dev

内森

编辑

谢谢你们，我对我的问题没有更具体表示歉意。编写此代码是为了检查第二个字符串中的字符是否在第一个字符串中。如果是这样，它将返回true，否则返回false。

所以我的代码行得通，我知道很多，但是我很肯定必须有一种更好的方法来实现这一点。

请记住，这是来自Freecodecamp的Javascript树的编码挑战。

这是我的代码：

function mutation(arr) {

  var stringOne = arr[0].toLowerCase();
  var stringTwo = arr[1].toLowerCase().split("");
  var i = 0;
  var truthyFalsy = true;

  while (i < arr[1].length && truthyFalsy) {

    truthyFalsy = stringOne.indexOf(stringTwo[i]) > -1;
    i++

  }
  console.log(truthyFalsy);
}




mutation(["hello", "hey"]);
//mutation(["hello", "yep"]);

这一定是更好的方法。我最近了解了map函数，但不确定如何使用它来实现此功能，并且最近还了解了Array.prototype.every（）函数，我将在今晚阅读该函数。

有什么建议吗？有什么想法吗？

巴米

这个问题很模糊。但是我从代码中了解的是，您需要检查两个字符串之间的字符串匹配。

既然您知道它的两个字符串，我就将它们作为两个参数传递。另外，我会将while更改为for语句，并添加一个break / continue以避免使用变量get和set。

请注意，在最坏的情况下，它几乎是相同的，但在最好的情况下，它是一半的计算时间。

mutation bestCase 14.84499999999997
mutation worstCase 7.694999999999993
bestCase: 5.595000000000027
worstCase: 7.199999999999989

// your function (to check performance difference)
function mutation(arr) {

  var stringOne = arr[0].toLowerCase();
  var stringTwo = arr[1].toLowerCase().split("");
  var i = 0;
  var truthyFalsy = true;

  while (i < arr[1].length && truthyFalsy) {

    truthyFalsy = stringOne.indexOf(stringTwo[i]) > -1;
    i++

  }
  return truthyFalsy;
}



function hasMatch(base, check) {
  var strOne = base.toLowerCase();
  var strTwo = check.toLowerCase().split("");

  var truthyFalsy = false;

  // define both variables (i and l) before the loop condition in order to avoid getting the length property of the string multiple times.
  for (var i = 0, l = strTwo.length; i < l; i++) {
    var hasChar = strOne.indexOf(strTwo[i]) > -1;
    if (hasChar) {
      //if has Char, set true and break;
      truthyFalsy = true;
      break;
    }
  }
  return truthyFalsy;
}

var baseCase = "hello";
var bestCaseStr = "hey";
var worstCaseStr = "yap";

//bestCase find match in first iteration
var bestCase = hasMatch("hello", bestCaseStr);
console.log(bestCase);

//worstCase loop over all of them.
var worstCase = hasMatch("hello", worstCaseStr);
console.log(worstCase);

// on your function
console.log('mutation bestCase', checkPerf(mutation, [baseCase, bestCaseStr]));

console.log('mutation worstCase', checkPerf(mutation, [baseCase, worstCaseStr]));

// simple performance check
console.log('bestCase:', checkPerf(hasMatch, baseCase, bestCaseStr));

console.log('worstCase:', checkPerf(hasMatch, baseCase, worstCaseStr));

function checkPerf(fn) {
  var t1 = performance.now();
  for (var i = 0; i < 10000; i++) {
    fn(arguments[1], arguments[2]);
  }
  var t2 = performance.now();
  return t2 - t1;
}

本文收集自互联网，转载请注明来源。

如有侵权，请联系[email protected] 删除。