on keypress jquery issues

Question

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

$(document).ready(function() { 

    $("#inputt").keypress(

    function(event) { 

     if (event.which == melee || MELEE) { 

      alert("You typed MELEE"); 

     } else if (event.which == knuckle || KNUCKLE) { 

      alert("You typed KNUCKLE"); 

     } else { 

      alert("Neither!!!"); 
      $('.knuckle').hide(); 
      $('.melee').hide(); 

     } 

    }); 

}); 

Вот пример. https://jsfiddle.net/galnova/snp1hqr7/1/

Answer 1

Таким образом, у вас есть ошибка в коде, поскольку такого события нет. Если вы видите консоль режима разработчика, вы можете увидеть ошибку. Таким образом, он должен либо скомбинировать значение текстового поля, используя $("#inputt").val() со значением «melee». Это будет делать трюк

if (event.which == melee || MELEE) { 

должен быть заменен

if ($("#inputt").val() == 'melee'|| $("#inputt").val() =='MELEE') { 

и впоследствии для

else if (event.which == knuckle || KNUCKLE) 

с

else if ($("#inputt").val() == 'knuckle' || $("#inputt").val() ==KNUCKLE) 

https://jsfiddle.net/aadi/2odkq3t8/

Редактировать: - Также вы используете keypress, поэтому он будет запущен, если текст, который вы написали, больше, чем в ближнем бою, и введите другой символ, тогда он будет запущен. Было бы лучше, если вы используете событие keyup, которое запускается, когда после нажатия клавиши отпускается какая-либо клавиша.

Answer 2

Есть несколько проблем с кодом.

• Ваш код выдает ошибку. melee is not defined. Вы ищете строковое значение, поэтому ближний бой должен быть в кавычках.
• event.which возвращает целое число, а не строку.
• Я не совсем уверен, что вы хотите связать свое событие нажатием ... , потому что это означает, что он работает с каждым нажатием клавиши. Если вы ищете для определенного слова, вместо этого вы можете привязаться к событию изменения . Или, как минимум, избавиться от предупреждения на каждом нажатии клавиши, потому что это раздражает. В моем примере я использую change, который требует, чтобы пользователь нажимал клавишу ввода после ввода слова.
• Что делать, если кто-то типа Melee? Ваш код ищет только два экземпляра слова, но есть много других, которые вы, возможно, захотите учесть. Один из способов справиться с этим - положить значение в нижнем регистре, а затем вам нужно сравнить только с melee.

Here is an updated fiddle

$("#inputt").change(

    function() { 

     if ($(this).val().toLowerCase() === "melee") { 

      alert("You typed MELEE"); 

     } else if ($(this).val().toLowerCase() === "knuckle") { 

      alert("You typed KNUCKLE"); 

     } else { 

      alert("Neither!!!"); 
      $('.knuckle').hide(); 
      $('.melee').hide(); 

     } 

    });