Не выводит сумму чисел?

Question

[FreeCam]

есть 4 input в 3 из них вводятся числа в 1 выводится ответ. У меня сумма чисел всегда равна 3. Как исправить? И как сделать вывод в input ответа.

var f = document.getElementById('first').value;
var s = document.getElementById('second').value;
var t = document.getElementById('third').value;
var ev = document.getElementById('e');
ev.addEventListener("click", function () {
var rest = Number(f) + Number(s) + Number(t);
document.getElementById("result").innerHTML = rest;
});

Answer 1

[AngReload]

У тебя f, s и t проверяются только один раз. Нужно сделать чтобы их значения обновлялись при каждом нажатии на кнопку.

var ev = document.getElementById('e');
ev.addEventListener("click", function () {
  var f = document.getElementById('first').value;
  var s = document.getElementById('second').value;
  var t = document.getElementById('third').value;
  var rest = Number(f) + Number(s) + Number(t);
  document.getElementById("result").value = rest;
});

Comments

[FreeCam]

Спасибо.

Answer 2

[sloupok123]

Присоединяюсь к предыдущему ответу и добавлю, у вас есть элемент в id = e. Как оказывается id должен содержать больше чем 1 символ, иначе в консоле вываливается ошибка.
Ваш скорректированный пример: https://jsfiddle.net/wbedmtp8/4/

Comments

[AngReload]

А вы заморочились :)

[sloupok123]

AngReload, привычка все проверять на практике :D

[AngReload]

sloupok123, правда, код ваш работает скорее вопреки.

[AngReload]

вы объявляете ev внутри функции, но он доступен снаружи, так как все элементы страницы с id можно получить из глобального объекта.

[sloupok123]

AngReload, спасибо за замечание, я просто проглядел, когда копировал объявления)

[AngReload]

sloupok123, ну не намеренно же. Собственно, это и была причина почему к селектору из одной буквы не подключались события.