5

Есть такой скрипт

function timer() {
var seconds = 5;
var seconds_timer_id = setInterval(function() {
            if (seconds > 0) {
                seconds --;
                if (seconds < 10) {
                    seconds = "0" + seconds;
                }
                $(".videotime").text(seconds);
            } else {
                clearInterval(seconds_timer_id);
                $("#allcount").val((+$("#allcount").val() + 1));

            }
        }, 1000);

    }

Который я запускаю так

<a href="index.php?id=test1" rel="prettyPopin" onclick="timer(); this.style.display='none';">Старт</a>

Когда скрипт запускается первый раз то он увеличивает значение в поле input на единицу, а когда я питаюсь запустить его второй раз ничего не преисходит. Вернее тймер идет но увеличения нет.

Форма в которой должно увеличеватся число

function timer() {
  var seconds = 5;
  var seconds_timer_id = setInterval(function() {
    if (seconds > 0) {
      seconds --;
      if (seconds < 10) {
        seconds = "0" + seconds;
      }
      $(".videotime").text(seconds);
    } else {
      clearInterval(seconds_timer_id);
      $("#allcount").val((+$("#allcount").val() + 1));
    }
  }, 1000);
}
<script src="https://ajax.googleapis.com/ajax/libs/jquery/1.4.2/jquery.min.js"></script>
<div class="videotime">0</div>

<form id="formMain" name="formMain">
<input type="text" value="0" id="allcount" name="counttest"  />
</form>

<a href="#" rel="prettyPopin" onclick="timer(); this.style.display='none';">Смотреть видео</a><br>

<a href="#" rel="prettyPopin" onclick="timer(); this.style.display='none';">Смотреть видео</a><br>

<a href="#" rel="prettyPopin" onclick="timer(); this.style.display='none';">Смотреть видео</a><br>

<a href="#" rel="prettyPopin" onclick="timer(); this.style.display='none';">Смотреть видео</a><br>

<a href="#" rel="prettyPopin" onclick="timer(); this.style.display='none';">Смотреть видео</a><br>

<a href="#" rel="prettyPopin" onclick="timer(); this.style.display='none';">Смотреть видео</a><br>

9
  • $("#allcount").val((+$("#allcount").val() + 1)); Непонятно, какой смысл в проверке (seconds < 10), если seconds начинается с числа 5 и уменьшется.
    – user176262
    6 мая 2016 в 21:09
  • @Igor работает только раз, вставил Ваш код при первом запуске скрипта сработало только раз, показало 1 а когда опять запускаю скрипт не работает. Скрипт запускаю так 'onclick="timer();' 6 мая 2016 в 21:16
  • @Igor Чет не понятно мне если пишу так то все работает <a onclick="timer();">asdas</a> а если вот так то нет <a href="#" rel="prettyPopin" onclick="timer(); this.style.display='none';" class="template_button">Смотреть видео</a> то не работает. 6 мая 2016 в 21:20
  • предлагаю Вам отредактировать вопрос. Когда перейдете в режим редактирования - обратите внимание на кнопку, позволяющую создать работающий фрагмент javascript'a и html'a, - легче будет понять, в чем дело.
    – user176262
    6 мая 2016 в 21:22
  • @Igor тут все работает, а у меня чет не хочет, добавляет единице тока раз и все. 6 мая 2016 в 21:32

1 ответ 1

1

Добрый день, переписал твой код.

http://codepen.io/anon/pen/MbgxzL?editors=1111

Сделал 2 разных таймера, когда первый заканчивает свою работу, мы его удаляем с помощью clearInterbal и запускаем новый, который уже увеличивает значение инпута каждый раз на 1

<!-- HTML -->
<div class="videotime">0</div>
<form id="formMain" name="formMain">
<input type="text" value="0" id="allcount" name="counttest"  />
</form>
<a href="#" rel="prettyPopin" class="seeVideo">Смотреть видео</a><br>

<!-- JS -->
$(document).on('ready', function(){

  var secAdvt = 6;
  var timeStart = 0;

  function timer(obj) {
    timeStart++;
    $("input[name=counttest]").val(timeStart);
  }

  function countdown(obj){
    secAdvt--;

    if(secAdvt == 0){
      clearInterval(startCountDown);
      startVideo = setInterval(function(){
        timer();
      }, 1000);
    }

    $(".videotime").text(secAdvt);
  }

  $(".seeVideo").on('click', function(){
    $(".videotime").text();
    startCountDown = setInterval(function(){
      countdown();
    }, 1000);
  });

});
1
  • А вообще в вашем коде логическая ошибка в том, что для счетчика используется одна переменная, можно переписать на 1 таймер, просто будет 2 счетчика. Один для рекламы, а второй для видео. Если нужно будет, пишите. Поправлю код.
    – lazyproger
    3 ноя 2016 в 13:02

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.