Как проверить массив запрещенных слов?

Question

Я пытаюсь сделать ярлык красным, если пользователь вводит запрещенное слово на клавиатуре. Я могу сделать это одним словом, но независимо от того, что я пробовал, я потерпел неудачу, когда дело дошло до нескольких.

Это мой код:

username.onkeyup = function() { 

    var forbiddenWords = ["ffff, aaaa, bbbb, hhhh, ggggg"]; 
    var username_value = this.value.split(''); 

    function isForbiddenWord(value) { 
     for (var i = 0; i < forbiddenWords.length; i++) { 
      var rgx = new RegExp(forbiddenWords[i], 'gi'); 
      if (rgx.test(value)) { 
       return true; 
      } 
      else { 
       return false; 
      } 
     } 
    } 

    if (isForbiddenWord(this.value) == true) { 
     console.log('Username contains swearing word. Please, remove it'); 
     document.getElementById('username_label').style.color = "red"; 
    } 
    else { 
     if (username_value[username_value.length - 1] === "-" || username_value[username_value.length - 1] === "_") { 
      console.log('Username cannot end in dash (-) or underscore (_)'); 
      this.value = this.value.slice(0, -1); 
     } 
     else if (this.value.length < 4) { 
      console.log('Username cannot be less than 4 characters'); 
      document.getElementById('username_label').style.color = "red"; 
     } 
     else { 
      document.getElementById('username_label').style.color = "green"; 
     } 
    } 
} 

Answer 1

Ваш массив запрещенных слов в настоящее время состоит из 1 слова: "ffff, aaaa, bbbb, hhhh, ggggg". Ваш код работает, если вы используете это как массив:

var forbiddenWords = ["ffff", "aaaa", "bbbb", "hhhh", "ggggg"]; 

Answer 2

Постарайся только возвращение ложным только после того, как цикл будет завершен, а не для каждого iterationas else. Таким образом, если они верны, истина вернет разрыв цикла, иначе, когда цикл будет выполнен, все будут определены как ложные.

Прямо сейчас, как только первый ложь встречается функция возвращает ложь и будет игнорировать все последующие слова в массиве

function isForbiddenWord(value) { 
    for (var i = 0; i < forbiddenWords.length; i++) { 
     var rgx = new RegExp(forbiddenWords[i], 'gi'); 
     if (rgx.test(value)) { 
      return true; 
     } 
     // move this outside loop 
     //else { 
      // return false; 
     //} 
    } 
    return false; 
} 

И, как было отмечено в других ответ исправить кавычки в массиве