问题是几乎所有变量createWrapp
都是隐式全局的。连续调用clock()
将覆盖它们,并且只有最后创建的时钟才会滴答作响。使用var
语句!
将其更改为:
function clock() {
var outerDiv = createElementWithClass('div', 'clock-window-wrapper'),
innerDiv = createElementWithClass('div', 'clock-menubar-wrapper');
outerDiv.appendChild(innerDiv);
document.getElementById("page-content-wrapper").appendChild(outerDiv);
var clock_windows_wrapper_close = createElementWithClass('div', 'close');
innerDiv.appendChild(clock_windows_wrapper_close);
var clock_toolbar_wrapper_close = createElementWithClass('div', 'clock-content-wrapper');
outerDiv.appendChild(clock_toolbar_wrapper_close);
var hours = createElementWithClass('ul', 'clock-digit-wrapper hour');
clock_toolbar_wrapper_close.appendChild(hours);
clock_toolbar_wrapper_close.appendChild(document.createTextNode(' ')); // Add this ##WTF
var minutes = createElementWithClass('ul', 'clock-digit-wrapper minute');
clock_toolbar_wrapper_close.appendChild(minutes);
clock_toolbar_wrapper_close.appendChild(document.createTextNode(' ')); // And this ##
var seconds = createElementWithClass('ul', 'clock-digit-wrapper second');
clock_toolbar_wrapper_close.appendChild(seconds);
var hour1 = createElementWithClass('li', "clock-digit-four");
var hour2 = createElementWithClass('li', "clock-digit-one");
hours.appendChild(hour1);
hours.appendChild(hour2);
var minute1 = createElementWithClass('li', "clock-digit-three");
var minute2 = createElementWithClass('li', "clock-digit-three");
minutes.appendChild(minute1);
minutes.appendChild(minute2);
var sec1 = createElementWithClass('li', "clock-digit-three");
var sec2 = createElementWithClass('li', "clock-digit-seven");
seconds.appendChild(sec1);
seconds.appendChild(sec2);
createWrapp
作为一个单独的函数实际上是没有必要的,因为它只被调用一次并且没有变量是它的本地变量。