Добрый вечер. Коллеги помогите оптимизировать код на обработку времени. Тесты проходят, не проходит скрипт по времени. Ниже сама задача

button.onclick = function() {
   let x = document.getElementById("popup");
   if (x.style.display === "none") {
     x.style.display = "block";
   } else {
     x.style.display = "none";
   }
   let val1 = document.getElementById('firstName').value;
   let val2 = document.getElementById('lastName').value;
   let val3 = document.getElementById('email').value;

   document.getElementById('name').innerHTML="First name: " + val1;
   document.getElementById('surname').innerHTML="Last name: " + val2;
   document.getElementById('email_val').innerHTML="Email: " + val3;
   
}

ага, функция работает ж

у тебя есть индекс элемента, зачем тебе проверять остальные элементы?

так задание не проходит

Добрый вечер. Коллеги помогите оптимизировать код на обработку времени. Тесты проходят, не проходит скрипт по времени. Ниже сама задача

как так?

так и у меня все passed ))

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

Там типо на оптимизацию

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

Мда?

Решение в лоб даёт ~285 мс.

Задача - найти первый элемент, чьё значение совпадает с индексом, на входе массив

Вроде понял, у popup походу нет стиля display none, поэтому сначала выполняется else, а потом при повторном нажатии if