Почему этот простой секундомер JavaScript не работает?

Question

Я написал очень простой код, который должен представлять секундомер с двумя кнопками, где нажатие «Пуск» должно начинаться с отметки вверх с 0 с 1 секундой задержки.

Однако, когда я нажимаю, ничего не меняется. Где моя ошибка?

"use strict"; 
 
var s = 1; 
 

 
function mf() { 
 
    setInterval(function() { 
 
    document.getElementById("#sw").innerText = ++s; 
 
    }, 1000); 
 
}

<div id="sw">0</div> 
 
<input type="button" value="Start" name="start" onlick="mf" /> 
 
<input type="button" value="Stop" name="stop" />

Answer 1

У вас есть две проблемы:

1): Вы используете

onlick="mf" 

Однако, это должно быть:

onclick="mf();" 

2): Вы использовали # в своем 'getElementByID'. Это не jQuery- использование

document.getElementById("sw") 

Работа ответа

"use strict"; 
 
var s = 1; 
 

 
    function mf() { 
 
    setInterval(function() { 
 
    document.getElementById("sw").innerText = s++; 
 
    }, 1000); 
 
}

<div id="sw">0</div> 
 
<input type="button" value="Start" name="start" onclick="mf();" /> 
 
<input type="button" value="Stop" name="stop" />

Ваши ошибки:

document.getElementById("#sw").innerText = ++s; 

должен быть

document.getElementById("sw").innerText = s++; 

Снимите #, и использовать s++, не ++s.

<input type="button" value="Start" name="start" onlick="mf" /> 

Этот код имеет 2 ошибки: onlick и mf. Onlick не поддерживается всеми браузерами (xD). Вместо этого используйте onclick. Кроме того, включите параметры, поэтому mf()

Answer 2

Прежде всего, у вас есть опечатка. Вы набрали onlick вместо onclick. Во-вторых, document.getElementById принимает строку, которая ссылается на идентификатор элемента, который вы пытаетесь получить. Однако ему не нужно #.

document.getElementById("sw")