функция не определена - PullRequest
Новая
       1

функция не определена

0 голосов
/ 29 ноября 2011 
function calcage(secs, num1, num2) {
  s = ((Math.floor(secs/num1))%num2).toString();
  if (LeadingZero && s.length < 2)
    s = "0" + s;
  return s;
}
function CountBack(secs)
{
  if (secs < 0) {
    document.getElementById("cntdwn").innerHTML = FinishMessage;
    return;
  }
  DisplayStr = DisplayFormat.replace(/%%D%%/g, calcage(secs,86400,100000));
  DisplayStr = DisplayStr.replace(/%%H%%/g, calcage(secs,3600,24));
  DisplayStr = DisplayStr.replace(/%%M%%/g, calcage(secs,60,60));
  DisplayStr = DisplayStr.replace(/%%S%%/g, calcage(secs,1,60));
  document.getElementById("cntdwn").innerHTML = DisplayStr;
  if (CountActive)
    setTimeout("CountBack(" + (secs+CountStepper) + ")", SetTimeOutPeriod);
}
function putspan() {
    document.write("<span id='cntdwn'></span>");
}
if (typeof(TargetDate)=="undefined")
  TargetDate = "12/31/2020 5:00 AM";
if (typeof(DisplayFormat)=="undefined")
  DisplayFormat = "%%D%% days, %%H%% hours, %%M%% minutes, %%S%% seconds";
if (typeof(CountActive)=="undefined")
  CountActive = true;
if (typeof(FinishMessage)=="undefined")
  FinishMessage = "Finished.";
if (typeof(CountStepper)!="number")
  CountStepper = -1;
if (typeof(LeadingZero)=="undefined")
  LeadingZero = true;

CountStepper = Math.ceil(CountStepper);
if (CountStepper == 0)
  CountActive = false;
var SetTimeOutPeriod = (Math.abs(CountStepper)-1)*1000 + 990;
putspan();
var dthen = new Date(TargetDate);
var dnow = new Date();
if(CountStepper>0)
  ddiff = new Date(dnow-dthen);
else
  ddiff = new Date(dthen-dnow);

gsecs = Math.floor(ddiff.valueOf()/1000);

CountBack(gsecs);

Если я вставлю этот код внутрь jQuery(function(){});, я получу: 

ReferenceError: CountBack is not defined

Почему?

Ответы [ 2 ]

3 голосов
/ 29 ноября 2011

Если в замыкании определено CountBack, то строка JavaScript, которую setTimeout будет проверять, не будет иметь к ней доступа.setTimeout будет eval() строковым аргументом в глобальной области видимости.

Никогда не передавать строку в setTimeout! Сделайте это вместо: 

if (CountActive)
    setTimeout(function(){CountBack(secs+CountStepper);}, SetTimeOutPeriod);
0 голосов
/ 29 ноября 2011

Передача строки в setTimeout нарушает область (она оценивается в глобальной области вместо текущей области).Вместо этого передайте функцию.

...