Таймер обратного отсчета работает неправильно - PullRequest
Новая
       5

Таймер обратного отсчета работает неправильно

1 голос
/ 01 октября 2010

Я работаю над таймером обратного отсчета для моего сайта, но не могу заставить скрипт работать. Создатель сценария, PHP F1, не смог помочь с ним.

Вопрос в том, почему переменная года в целевой дате ($targetDate) не попадает в реализацию HTML / JavaScript?

Я использую функцию mktime(). Независимо от того, поместил ли я фактическую дату в последнюю позицию атрибута (для года) или добавил ли я функцию даты (например, как я это сделал в приведенном ниже коде), я просто не могу заставить ее правильно отображаться на передний конец.

Я ни в коем случае не гуру PHP / JavaScript, так что это может показаться проблемой, но если кто-нибудь сможет мне помочь с этим, это будет очень признательно!

Вот живая реализация кода ниже: http://tripdealio.com/countdown/countdown5.php

Вот код: 

<?php
/* Define your target date here
    $targetYear  = 2011;
    $targetMonth = 09;
    $targetDay   = 30;
    $targetHour  = 23;
    $targetMinute= 59;
    $targetSecond= 59;
End target date definition */ 

// Define date format
$dateFormat = "Y-m-d H:i:s";

$targetDate = mktime(23,59,59,09,30,date("Y"));
$actualDate = time();

$secondsDiff = $targetDate - $actualDate;

$remainingDay     = floor($secondsDiff/60/60/24);
$remainingHour    = floor(($secondsDiff-($remainingDay*60*60*24))/60/60);
$remainingMinutes = floor(($secondsDiff-($remainingDay*60*60*24)-($remainingHour*60*60))/60);
$remainingSeconds = floor(($secondsDiff-($remainingDay*60*60*24)-($remainingHour*60*60))-($remainingMinutes*60));

$targetDateDisplay = date($dateFormat,$targetDate);
$actualDateDisplay = date($dateFormat,$actualDate);

?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>Max's Countdown System</title>
<link href="style/style.css" rel="stylesheet" type="text/css" />
<script type="text/javascript">
  var days = <?php echo $remainingDay; ?>  
  var hours = <?php echo $remainingHour; ?>  
  var minutes = <?php echo $remainingMinutes; ?>  
  var seconds = <?php echo $remainingSeconds; ?>  

function setCountDown ()
{
  seconds--;
  if (seconds < 0){
      minutes--;
      seconds = 59
  }
  if (minutes < 0){
      hours--;
      minutes = 59
  }
  if (hours < 0){
      days--;
      hours = 23
  }
  document.getElementById("remain").innerHTML = days+" days, "+hours+" hours, "+minutes+" minutes, "+seconds+" seconds";
  setTimeout ( "setCountDown()", 1000 );
}

</script>
</head>

<body onload="setCountDown();">
<div id="container">
    <div id="header"><div id="header_left"></div>
    <div id="header_main">Max's Countdown System</div><div id="header_right"></div></div>
    <div id="content">
        <table class="countTable">
           <tr><td>Target date:</td><td><?php echo $targetDateDisplay; ?></td></tr>
           <tr><th colspan="2" id="remain"><?php echo "$remainingDay days, $remainingHour hours, $remainingMinutes minutes, $remainingSeconds seconds";?></th></tr>
           <tr><td>Actual date:</td><td><?php echo $actualDateDisplay; ?></td></tr>
       </table>
    </div>
    <div id="footer"><a href="http://www.phpf1.com" target="_blank">Powered by PHP F1</a></div>
</div>
</body>
</html>

Ответы [ 2 ]

1 голос
/ 01 октября 2010

Вы рассчитываете целевое время, используя 

$targetDate = mktime(23,59,59,09,30,date("Y"));
    // be careful:            ↑

, но числа, начинающиеся с 0, обрабатываются как восьмеричные в PHP; кроме того, чтобы процитировать документацию :

Если недопустимая цифра указана в восьмеричном целом числе (т. Е. 8 или 9), остальная часть числа игнорируется.

Таким образом, 09 будет фактически проанализирован как 0 и, как пример № 3 из mktime показывает

Последний день любого месяца может быть выражен как "0" день следующего месяца, а не как -1 день.

Таким образом, месяц 0 приведет к уменьшению номера года, в данный момент до 2009 года.

Как вы уже догадались, решение простое: отбросьте 0

$targetDate = mktime(23,59,59,9,30,2011);

(date("Y") вернет 2010 в данный момент, но согласно вашему комментарию вы хотите 2011).

Кстати, я думаю, вы хотите использовать setInterval вместо setTimeout; см. setTimeout или setInterval?

1 голос
/ 01 октября 2010

Я не вижу ничего особенно плохого в алгоритме, в коде JS есть только синтаксические ошибки (пропущенные точки с запятой).Это фиксированный код: 

var days = <?php echo $remainingDay; ?>;
var hours = <?php echo $remainingHour; ?>;  
var minutes = <?php echo $remainingMinutes; ?>;  
var seconds = <?php echo $remainingSeconds; ?>;

function setCountDown ()
{
  seconds--;
  if (seconds < 0){
      minutes--;
      seconds = 59;
  }
  if (minutes < 0){
      hours--;
      minutes = 59;
  }
  if (hours < 0){
      days--;
      hours = 23;
  }
  document.getElementById("remain").innerHTML = days+" days, "+hours+" hours, "+minutes+" minutes, "+seconds+" seconds";
  setTimeout ( "setCountDown()", 1000 );
}

Вы также можете использовать setInterval вместо setTimeout, но это просто косметическое изменение.

...