Почему js выполняет код несколько раз?

Question

[marsel81]

При нажатии на кнопку "Кнопка", он должен внизу вывести ' Result: ' с найденными номерами абонентов. Но он делает это повторно, два раза. Странно, что даже после обнуления он делает это

https://codepen.io/anon/pen/LBpBXQ

Comments

[Павел Корнилов]

У меня вообще не работает. Мозилла.

[marsel81]

KorniloFF, вы могли бы скопировать на пк файлы? у меня тоже что-то не запускается, хотя открываю с пк и работает хорошо

[Павел Корнилов]

marsel81, гуд. Ща заморочусь.

[marsel81]

KorniloFF, перезалил сюда
https://codepen.io/anon/pen/LBpBXQ

[Павел Корнилов]

marsel81, да я уже в один файл все впихнул, пытаюсь понять что к чему.

[Павел Корнилов]

Либо я не понял вопрос, либо просто не вижу этого повторного срабатывания.

[marsel81]

KorniloFF, да, у меня тоже вот пропало. очень странно, но время от времени появляется

Answer 1

[0xD34F]

Вы обходите массив, проверяете совпадение имени - ну, отлично. А вот что не отлично - так это ветка else, где вы пишите в результат поиска, что ничего не найдено. Делать это надо один раз - после цикла, а не внутри. Для этого можете перед циклом объявить переменную, типа var found = false, в случае если совпадение найдено - устанавливаете её в true. А код из блока else выносите за цикл и оборачиваете в if (!found).

Answer 2

[lamer350]

https://codepen.io/anon/pen/KBdxNN
Там еще помимо комментариев выше есть нюансы, в том числе проблема с циклом так как length выводит количество элементов, а цикл начинается с 0, и у вас один лишний запрос в цикле, а в последствии после цикла код не будет выполняться.