Можно ли это как тот сократить?

Question

[godsplan]

j

$('.js-plus').on('click', function () {
//do
	})
	$('.js-minus').on('click', function () {
		//do
	})

То есть есть кнопка плюс знаения и кнопка минус, внутрик обработчика выполнятеся немного разные действия. Есть ли возможность сокращения этого кода?Мне кажется я неправильно все построил

Comments

[Israfil22]

Без содержимого Ваших обработчиков и контекста не понятно)

[godsplan]

Israfil22,

$('.js-plus').on('click', function () {
		calcSize++;
		$('#calc-size').val(calcSize);
		totalPrice()
	})
	$('.js-minus').on('click', function () {
		if (calcSize <= 1) {
			console.log(calcSize)
			return
		} else {
			calcSize--;
			$('#calc-size').val(calcSize);
			totalPrice()
		}
	})

[Ярослав Иванов]

godsplane, и что тут сокращать? 10 строчек жалко?

Answer 1

[Alex]

Примерно так можно:

$('.js-button').on('click', function () {
  ...

  if (this.hasClass('js-plus')) { ... }
  else { ... }   

  ...
})

Comments

[godsplan]

А стоит ли так делать?

[Kovalsky]

godsplane, вот это поворот. Ну ты же сам спросил как сократить, вот ответ

[godsplan]

Kovalsky, ну и что, может я хочу услышать что сокращать тут нечего

Answer 2

[Алексей Скляров]

Пишите один обработчик, в котором указываете код, который всегда выполняется для обоих случаев, для отдельного случая используйте if

Answer 3

[Israfil22]

Да, код можно улучшить. Но в этом абсолютно нет никакой необходимости.
Данный фрагмент настолько мал, что задаваться вопросом об его эффективности, скорее всего, бесполезно и даже вредно.
Если интересно услышать о том, какие есть варианты, то выбирайте из ответов)
Я бы на Вашем месте сделал:

const createCounter = () => {
    const counter = 1
    return {
        increment: () => counter++,
        decrement: () => {
            if (counter <= 1) return
            counter--                
        }
    }
}
сounterСontrollers = createCounter()
$('.js-plus').on('click', сounterСontrollers.increment)
$('.js-minus').on('click', сounterСontrollers.decrement)

Comments

[Ярослав Иванов]

А теперь объясняйте что такое замыкание.

[Israfil22]

Ярослав Иванов, а для каких целей?