Как можно сократить этот JQuery код?

Question

[Andrew Kolomiets]

$('.pict' + i).fadeIn(300);
        $('.buy' + i).fadeIn(300);

Есть такая запись, можно как-то объединить под один fadeIn?

Comments

[Геннадий]

Как вариант, можно попробовать так:
$('.pict' + i, '.buy' + i).fadeIn(300);

[Andrew Kolomiets]

Геннадий, Не, так не работает, но благодарю за помощь

Answer 1

[Роман Сарваров]

$(`.pict${i}, .buy${i}`).fadeIn(300);

Answer 2

[Сергей delphinpro]

const classes = [
  '.pict',
  '.buy',
];
$(classes.map(c => c + i).join(',')).fadeIn(300);
// или
classes.forEach(cls => $(cls + i).fadeIn(300));

Вопрос: нафига?

Answer 3

[Михаил Ушенин]

Чтобы не писать одно и то же число в разных местах и не использовать глобальные константы, можно описать свою функцию и вызывать её:

$.fn.myFadeIn = function () {
    $(this).fadeIn(300);
}; 

$('...').myFadeIn();

Если потребуется изменить скорость анимации, это можно будет сделать в 1 месте.

Comments

[Andrew Kolomiets]

Очень даже полезно, благодарю.