Перебор объекта?

Question

[Ivan Ivanovich]

Здравствуйте.

Есть код - https://jsfiddle.net/a2ns1zbe/

Подскажите пожалуйста, почему если я ввожу слово test, ничего не происходит, хотя должно выводиться ok, но когда я ввожу stop, все работает как надо.

Подскажите пожалуйста, почему это происходит?

Answer 1

[MagicMight]

И можно заменить

inp.addEventListener('keyup',()=>{
for(let key in codes){
	  if(inp.value === key){
	    info.textContent = `${codes[key]}`;
	    info.style.display = 'block';
	  } else {
	    info.style.display = 'none';
	  }
	}; 
});

на

inp.addEventListener('keyup',()=>{

  if(inp.value in codes) {
  	info.textContent = `${codes[inp.value]}`;
    info.style.display = 'block';
  }
  else {
  	info.style.display = 'none';
  }

});

Answer 2

[Салават Салахутдинов]

Потому что в цикле for вы не делаете выход из цикла после того как значение было найдено(16 строчка break):
https://jsfiddle.net/ot5mjuvw/2/
Но вообще проверку ключа можно сделать и без цикла (как в ответе MagicMight)

Comments

[MagicMight]

Я бы поостерегся делать
if(codes[inp.value]) {...}
потому что вот:


Так как он будет проверять не только наличие ключа, но и преобразование к логическому типу самого значения, если ключ найден

[Салават Салахутдинов]

MagicMight Тут я как раз подумал о том, чтобы не выводить логически ложные значения (null, undefined, false и пустую строку ""), а про число 0 забыл ) Так что да, если нужно выводить число 0, ваш вариант правильнее

Answer 3

[Константин]

else блок не нужен. во второй раз он сбрасывает display дива на none

Answer 4

[Арсений Матыцин]

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

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