Как сделать вспышку экрана, не работает

Question

var flashStep = 1; 
function flash() { 
if(flashStep==1) { 
document.bgColor="FFFF00"; 
flashStep=2; 
} 
else { 
document.bgColor="FF0000"; 
flashStep=1; 
} 
} 
var task = window.setInterval("flash()",1000); 

Этот код должен сделать экран вспышкой, но не работает вообще. http://jsfiddle.net/phjnM/2/. Почему это не работает, оно выглядит безупречно для меня

Answer 1

Проблема заключается в (глобальный) сфера бытия применяется при передаче строки до setInterval. Если вы передадите ссылку на функцию, то она работает:

var task = window.setInterval (flash, 1000);

См http://jsfiddle.net/phjnM/7/

Другой вариант заключается в jsfiddle выбрать опцию «Nowrap», так что ваш код не определен в области видимости OnLoad/onready обработчика ...

Answer 2

Поскольку ваш код в JSFiddle завернут в функцию onload.

Передача ссылки на функцию будет исправить.

var task = window.setInterval(flash,1000); 

Или альтернативой было бы выбрать (no wrap) вариант из меню слева.

---

Кстати, сделайте это, чтобы цвет фона документа.

var i = 0; 
var task = setInterval(flash, 1000); 

function flash() { 
    document.documentElement.style.backgroundColor=(i = ~i) ? '#ff0' : '#f00'; 
} 

JSFIDDLE DEMO

Answer 3

Вместо:

document.bgColor="FFFF00" 

вам нужен этот вид:

document.body.style.backgroundColor="#FFFF00"; 

Вы отсутствовали в .body.style., вы имели неправильное имя стиля атрибут backgroundColor a и вам нужно иметь значение #.

Кроме того, ваш SetTimeout() должен быть таким:

var task = setInterval(flash,1000); 

В вашей скрипке, вы можете получить фотовспышка быть глобальным, изменяя это определение к:

window.flash = function() { 

Вы можете видеть, что все это работает в пересмотренной версии вашего jsFiddle: http://jsfiddle.net/jfriend00/h4VZm/

Answer 4

Использовать window.setInterval(flash, 1000).

Эта конкретная проблема была, вероятно, проблемой с областью при запуске кода на jsfiddle. В общем, не передавайте код как строку, когда вы действительно хотите передать обратный вызов, который в Javascript вы можете сделать, передав эту функцию.

Answer 5

Несколько вопросов. Прежде всего, bgColor - свойство элементов - оно не относится к document. Возможно, вы имели в виду document.body? Во-вторых, вы - , используяbgColor. Пожалуйста, не надо. Вместо этого вы должны использовать style.backgroundColor. В-третьих, вы, вероятно, захотите использовать логическое значение для двух состояний. В-четвертых, никогда не передавайте строку setInterval или setTimeout.Таким образом, ваш код, переписано, было бы что-то вдоль линий:

var flash = false; 
var task = setInterval(function() { 
    if(flash = !flash) { 
     document.body.style.backgroundColor = '#ff0'; 
    } else { 
     document.body.style.backgroundColor = '#f00'; 
    } 
}, 1000); 

Answer 6

setInterval(flash,1000); 

http://jsfiddle.net/phjnM/5/

Answer 7

Смотрите сейчас его работы, http://jsfiddle.net/phjnM/8/