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
    21 окт 2019 в 15:59
  • 1
    для понимания тут не хватает воспроизводимого примера - вот что я имел ввиду, а не "лишний" код
    – InDevX
    21 окт 2019 в 16:08
  • 1
    @КамранРаджабов что делает ваша функция forwardClick и зачем она вам? Второй вопрос, что вы понимаете под "хочу написать для нее отдельную функцию"?
    – Arsen
    21 окт 2019 в 16:21
  • 1
    @КамранРаджабов, нет - я повторюсь, именно твое решение в вопросе демонстрирует подход "лишь бы сделать": так как сложность кода совершенно неадекватна сложности задачи. В реальной работе, такой индусский код недопустим - за подобное могут даже уволить, на полном серьезе. Вместо того чтобы огрызаться на конструктивное замечание, ознакомься с документацией метода setMonth() и принципами эффективного программирования. Это поможет тебе и данную проблему решить, и не совершать таких же ошибок в будущем.
    – yar85
    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
    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
  • нужно чтобы выводила всю дату 21 окт 2019 в 16:29
  • разобрался, вечером просто голова не работала не понял что написал, спасибо за помощь 22 окт 2019 в 8:03

Ваш ответ

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

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