Juntae Juntae - 6 months ago 107
Javascript Question

Check if a string contains any element of an array in JavaScript

How can I check if a string contains any element of an array? I want to filter some array if the element has some string. Please see below code.



var arr = ['banana', 'monkey banana', 'apple', 'kiwi', 'orange'];

function checker(value) {
var prohibited = ['banana', 'apple'];

for (var i = 0; i < prohibited.length; i++) {
if (value.indexOf(prohibited[i]) == -1) {
return true;
} else {
return false;
}
}
}

arr = arr.filter(checker);
console.log(arr);





The result is
[ 'apple', 'kiwi', 'orange' ]
. The
'apple'
should get removed, but it isn't.

Above code only filtered 'banana', not 'apple'. I have many keywords to filter. Is there an easier way?

Answer

Problem lies in the for loop, which only iterates once since return ends the function, cutting off the for loop in the process. So, you can update the code like so to make the function only return once the for loop has been completed .

var arr = ['banana', 'monkey banana', 'apple', 'kiwi', 'orange'];

function checker(value) {
  var prohibited = ['banana', 'apple'];

  for (var i = 0; i < prohibited.length; i++) {
    if (value.indexOf(prohibited[i]) > -1) {
      return false;
    }
  }
  return true;
}

arr = arr.filter(checker);
console.log(arr);


For reducing the function you can use every() and indexOf() methods

The 'every' method executes the provided callback function once for each element present in the array until it finds one where callback returns a falsy value (a value that becomes false when converted to a Boolean). If such an element is found, the every method immediately returns false. Otherwise, if callback returned a true value for all elements, every will return true. callback is invoked only for indexes of the array which have assigned values; it is not invoked for indexes which have been deleted or which have never been assigned values.(Taken from here)

var arr = ['banana', 'monkey banana', 'apple', 'kiwi', 'orange'];

function checker(value) {
  var prohibited = ['banana', 'apple'];
  return prohibited.every(function(v) {
    return value.indexOf(v) == -1;
  });
}

arr = arr.filter(checker);
console.log(arr);


For older browser check polyfill option of every method.


You could even use a regex here. Generate regex using the array and use test() to check match

var arr = ['banana', 'monkey banana', 'apple', 'kiwi', 'orange'];

function checker(value) {
  var prohibited = ['banana', 'apple'];
  var regex = new RegExp(prohibited.map(function(s) {
    return s.replace(/[-/\\^$*+?.()|[\]{}]/g, '\\$&')
  }).join('|'));
  return !regex.test(value);
}

arr = arr.filter(checker);
console.log(arr);

Refer this answer for string to regex conversion : Can you create JavaScript regexes on the fly using string variables?

Comments