Functions declared within loops referencing an outer scoped variable may lead to conf

[mrG0bliN]

function jqFormSteps() {
    var n = 0;
    for (n = 0; n < 5; n++) {

      var formStep = $('#form_step_one> fieldset');
      var stepNext = $('fieldset[data-step="' + n + '"]').next();
      var stepPrev = $('fieldset[data-step="' + n + '"]').prev();

  тут ошибка->>    var stepFormAddClass = function(parm1, parm2) {
        $(parm1).on('click', function(event) {
          event.preventDefault();
          formStep.removeClass('selector');
          parm2.addClass('selector');
        });
      };
      stepFormAddClass('#next' + n + '', stepNext);
      stepFormAddClass('#prev' + n + '', stepPrev);
    }
  }
-------

ошибка: Functions declared within loops referencing an outer scoped variable may lead to confusing semantic.

как избавится от ошибки? понимаю что функция вошла в цикл и это не самый лучший способ что бы листать блоки туда сюда

[j0hnik]

функцию вынеси за цикл фор, зачем ее в цикл засунул?

[mrG0bliN]

function jqFormSteps() {
      var n = 0;
      var formStep = $('#form_step_one> fieldset');
      var stepNext = $('fieldset[data-step="' + n + '"]').next();
      var stepPrev = $('fieldset[data-step="' + n + '"]').prev();

      var stepFormAddClass = function(parm1, parm2) {
        $(parm1).on('click', function(event) {
         for (n = 0;n < 5;n++) {
          event.preventDefault();
          formStep.removeClass('selector');
          parm2.addClass('selector');
        }
      });
      stepFormAddClass('#next' + n + '', stepNext);
      stepFormAddClass('#prev' + n + '', stepPrev);
    };
  }

ошибка пропадает. но зато перестают перелистыватся туда сюда дивы

и моя форма ломается

[j0hnik]

function jqFormSteps() {
      var n = 0;
      var formStep = $('#form_step_one> fieldset');
      var stepNext = $('fieldset[data-step="' + n + '"]').next();
      var stepPrev = $('fieldset[data-step="' + n + '"]').prev();

      var stepFormAddClass = function(parm1, parm2) {
        $(parm1).on('click', function(event) {
          event.preventDefault();
          formStep.removeClass('selector');
          parm2.addClass('selector');
        });
      };
       for (n = 0;n < 5;n++) {
      stepFormAddClass('#next' + n + '', stepNext);
      stepFormAddClass('#prev' + n + '', stepPrev);
    }
  }

попробуйте вот так, если не получится, сделайте тестовый пример здесь.

[mrG0bliN]

function jqFormSteps() {
    var moveStep = {
      nextStep: function() {  $('[data-step]:visible').removeClass('selector').next().addClass('selector');  },
      prevStep: function() {  $('[data-step]:visible').removeClass('selector').prev().addClass('selector');  }
    };
    var stepFormAddClass = function(param1) {
      $(param1).on('click', function(event) {
        event.preventDefault();
        if (param1 == '.js__nextButton') { moveStep.nextStep(); } else { moveStep.prevStep();}
      });
    };
    stepFormAddClass('.js__nextButton');
    stepFormAddClass('.js__prevButton');
  }

переписал по новой. теперь работает переключение и нет ошибок