1

В функции organizer() есть событие клик (она увеличивает месяц на один), хочу написать для нее отдельную функцию.

Проблема в том, что как только вывожу из функции она увеличивает только на одни дальше не увеличивает. Функция клик которую я написал находится под функцией organizer().

function organizer() {
  
  var date = new Date();
  var year = date.getFullYear();
  var month = date.getMonth();

  forward.addEventListener("click", function() {
    var dateElem = new Date(year, month += 1);
    if (dateElem.getMonth == 11) {
      dateElem.getFullYear += 1;
    };
    alert(dateElem);
  });


};

function forwardClick(monthElem, yearElem) {

  var newDate = new Date(yearElem, monthElem += 1);
  if (newDate.getMonth() == 11) {
    newDate.getFullYear() += 1;
  };

  alert(newDate);
};
<button id="forward">&#9658;</button>

18
  • mvp предоставьте, хотя бы
    – InDevX
    Commented 21 окт. 2019 в 15:59
  • 1
    для понимания тут не хватает воспроизводимого примера - вот что я имел ввиду, а не "лишний" код
    – InDevX
    Commented 21 окт. 2019 в 16:08
  • 1
    @КамранРаджабов что делает ваша функция forwardClick и зачем она вам? Второй вопрос, что вы понимаете под "хочу написать для нее отдельную функцию"?
    – Арсен
    Commented 21 окт. 2019 в 16:21
  • 1
    @КамранРаджабов, нет - я повторюсь, именно твое решение в вопросе демонстрирует подход "лишь бы сделать": так как сложность кода совершенно неадекватна сложности задачи. В реальной работе, такой индусский код недопустим - за подобное могут даже уволить, на полном серьезе. Вместо того чтобы огрызаться на конструктивное замечание, ознакомься с документацией метода setMonth() и принципами эффективного программирования. Это поможет тебе и данную проблему решить, и не совершать таких же ошибок в будущем.
    – yar85
    Commented 22 окт. 2019 в 8:47
  • 1
    Процитирую с MDN: «If a parameter you specify is outside of the expected range, setMonth() attempts to update the date information in the Date object accordingly. For example, if you use 15 for monthValue, the year will be incremented by 1, and 3 will be used for month.»
    – yar85
    Commented 22 окт. 2019 в 8:48

1 ответ 1

1

function organizer() {

  var date = new Date();
  var year = date.getFullYear();
  var month = date.getMonth();

  forward.addEventListener("click", function() {
    month = forwardClick(month, year);
  });
}

function forwardClick(monthElem, yearElem) {

  var newDate = new Date(yearElem, monthElem += 1);
  if (newDate.getMonth() == 11) {
    newDate.setFullYear(newDate.getFullYear() + 1);
  }

  alert(newDate);
  return monthElem;
}

organizer();
<button id="forward">&#9658;</button>

2
  • нужно чтобы выводила всю дату Commented 21 окт. 2019 в 16:29
  • разобрался, вечером просто голова не работала не понял что написал, спасибо за помощь Commented 22 окт. 2019 в 8:03

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

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