помогите переписать 2 функции в одну [закрыт]

105

31 марта 2021, 10:30

Закрыт. Этот вопрос не по теме. Ответы на него в данный момент не принимаются.

Хотите улучшить этот вопрос? Обновите вопрос так, чтобы он вписывался в тематику Stack Overflow на русском.

Закрыт 1 год назад.

Улучшить вопрос

подскажите как переписать 2 эти функции в одну, они отвечают за 2 стрелки слайдера и почти идентичны, мне сказали переписать это в одну функцию то не выходит из-за барьера знаний, надеюсь на помощь.

function next() {
  $wrapper.animate({
    'margin-left': '-=' + w
  }, set.anim, function () {
    var $this = $(this);
    $this.find('>li:first-child').appendTo($this);
    $this.css('margin-left', 0);
  });
}
function prev() {
  $wrapper.animate({
    'margin-left': '+=' + w
  }, set.anim, function () {
    var $this = $(this);
    $this.find('>li:last-child').prependTo($this);
    $this.css('margin-left', 0);
  });
}

Answer 1

видимо, как-то так?

function move(next = true){
    let sign = next ? '-' : '+';
    let anim = {'margin-left' : `${sign}=${w}` };
    $wrapper.animate(anim,  set.anim, function(){
        if(next){
             $(this).children("li:first-child").appendTo($(this));
        }
        else $(this).children("li:last-child").prependTo($(this));

        $(this).css('margin-left', 0);
    });
}

Answer 2

Про читаемость в вопросе ничего нет, по этому вот, держите:

function next(f) { 
  let m = 'margin-left'
  $wrapper.animate({
    [m] : `${f?"-":"+"}+${w}`
  }, set.anim, function(){ 
    $(this).css(m, 0).find(`>li:${f?"fir":"la"}st-child`)[`{f?"ap":"pre"}pendTo`]($(this))
  });
}