JS中forEach的坑

发表于 | 分类于
JS中forEach的坑

前言

平常我们都把forEach和for循环作为等效的替代。

但是forEach中的return并不会返回并跳出循环。

forEach里的return其实是充当continue的作用，即本次循环之后的语句不再执行，不会终止循环，更不会结束函数返回。

示例

假如我们有这样一个需求

如果文章的标题和内容包含了所有的关键字，我们就返回true，也就是说只要不包含任意一个关键字我们就返回false。

我们就按如下写了代码，但是却不能实现我们想要的结果，该方法最终会永远返回true。

1
2
3
4
5
6
7
8
9
10
//文章的标题和内容是否包含所有的关键字
function isContainAllKeyWords(titleText, contentText, keywords) {
  let textAll = (titleText + contentText).toLowerCase();
  keywords.forEach(keyword => {
    if (textAll.indexOf(keyword.toLowerCase()) === -1) {
      return false;
    }
  });
  return true;
}

解决方法

替换为fori循环

1
2
3
4
5
6
7
8
9
10
11
//文章的标题和内容是否包含所有的关键字
function isContainAllKeyWords(titleText, contentText, keywords) {
  let textAll = (titleText + contentText).toLowerCase();
  for (let i = 0; i < keywords.length; i++) {
    let keyword = keywords[i];
    if (textAll.indexOf(keyword.toLowerCase()) === -1) {
      return false;
    }
  }
  return true;
}

抛出异常并捕获

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
//文章的标题和内容是否包含所有的关键字
function isContainAllKeyWords(titleText, contentText, keywords) {
  let textAll = (titleText + contentText).toLowerCase();
  try {
    keywords.forEach(keyword => {
      if (textAll.indexOf(keyword.toLowerCase()) === -1) {
        throw new Error('not all contain'); // 抛出异常
      }
    });
  } catch (e) {
    if (e.message === 'not all contain') {
      return false; // 这里捕获异常并返回
    }
  }
  return true;
}