-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);
  });
}
1
  • вынеси селектор в параметр и у тебя будет одна функция
    – Grundy
    12 июл 2019 в 7:20

2 ответа 2

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);
    });
}
7
  • похоже, но я предлагаю еще вызов объединить :) 12 июл 2019 в 8:28
  • @StrangerintheQ у тебя в коде вообще не понятно .Почему не дружишь с пробелами или хотя бы с комментариями ?
    – Randall
    12 июл 2019 в 8:32
  • @ИзбытокСусликов у телефона экран узкий :) 12 июл 2019 в 8:38
  • @ИзбытокСусликов imgur.com/a/DWLns98 12 июл 2019 в 8:41
  • 1
    @StrangerintheQ я сначала объединил, потом разъединил. не наглядно.
    – teran
    12 июл 2019 в 8:56
1

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

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))
  });
}
4
  • Издеваетесь.. Хммм... А могли бы вы еще больше запутать?)) 12 июл 2019 в 10:57
  • @StepanKasyanenko я думал над загадочной деструктуризацией но не стал 12 июл 2019 в 11:14
  • @StepanKasyanenko сделал еще немного ужаснее 12 июл 2019 в 11:43
  • Оооо! Вы мой герой - держите плюсик! 12 июл 2019 в 11:46

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