Проверьте, содержит ли строка какой-либо массив строк без regExp [duplicate]

Question

Проверьте, содержит ли строка какой-либо массив строк без regExp [duplicate]

1

Я проверяю строковый ввод, содержит ли он какой-либо массив строк или нет. Это проходит большинство тестов, но не ниже.

Может ли кто-нибудь сломать мой код, почему он работает неправильно?

     function checkInput(input, words) {
      var arr = input.toLowerCase().split(" ");
      var i, j;
      var matches = 0;
      for(i = 0; i < arr.length; i++) {
        for(j = 0; j < words.length; j++) {
          if(arr[i] == words[j]) {
            matches++;
          }
        }
      }
      if(matches > 0) {
        return true;
      } else {
        return false;
      }
    };

checkInput("Visiting new places is fun.", ["aces"]); // returns false // code is passing from this test
checkInput('"Definitely," he said in a matter-of-fact tone.', 
    ["matter", "definitely"])); // returns false; should be returning true;

Спасибо за ваше время!

Tiyor 25 окт. 2017, в 08:56

Источник

1

это не проблема дела
JordanHendrix 25 окт. 2017, в 06:40
0

Почему вы не используете регулярные выражения?
inetphantom 25 окт. 2017, в 06:40
0

Много не-регулярных простых способов сделать это. if (words.some(word => input.includes(word))) {/*do a thing*/} останавливается после первого совпадения.
user1596138 25 окт. 2017, в 06:46

Показать ещё 1 комментарий

Теги:

javascript

string

arrays

2 ответа

Ещё вопросы

Почему вы не используете регулярные выражения?
Много не-регулярных простых способов сделать это. if (words.some(word => input.includes(word))) {/*do a thing*/} останавливается после первого совпадения.

Lucky Kleinschmidt · Answer 1 · 2017-10-25T05-19-00.000Z

Для этого вы можете использовать функциональные методы. Попробуйте Array.some.

const words = ['matters', 'definitely'];
const input = '"Definitely," he said in a matter-of-fact tone.';
console.log(words.some(word => input.includes(word)));

Это с учетом регистра. ОП хочет решение без учета регистра.
Нечувствительность к регистру не имеет отношения к вопросу. Это очень, очень ясно, что вы должны уменьшить строки. Не поощряя вампиров.
Вопрос действительно в том, «как посмотреть, содержит ли строка какой-либо массив строк», это явный дубликат поверх обычного вампа Q. Он будет закрыт, мой ответ пропадет. Я просто хочу, чтобы ОП увидел, что есть лучший способ, ха-ха
Вы правы! Кстати, ваш ответ хороший ответ. +1

Hassan Imam · Answer 2 · 2017-10-25T05-31-00.000Z

Вы можете использовать array#includes чтобы проверить, существует ли слово на вашем входе, и конвертировать как ваш input и words в нижнем регистре, а затем использовать array#includes.

function checkInput(input, words) {
 return words.some(word => input.toLowerCase().includes(word.toLowerCase()));
}

console.log(checkInput('"Definitely," he said in a matter-of-fact tone.', 
["matter", "definitely"]));

Вы можете создать регулярное выражение и использовать флаг i для указания нечувствительности к регистру

function checkInput(input, words) {
 return words.some(word => new RegExp(word, "i").test(input));
}

console.log(checkInput('"Definitely," he said in a matter-of-fact tone.', 
["matter", "definitely"]));