Возможно ли сократить код jQuery?

Question

[Роман Огарков]

// Сложение и вычитание суммы по клику
	$('.p-btn').click(function(e) {
		e.preventDefault();
		var one = 1;
		var presentValue = $( this ).siblings('input:text').val();
		$( this ).siblings('input:text').val(Number(presentValue) + one);
		var priceValue = $( this ).closest('.basket-row').find('.sum-container span').attr('data-sum');
		priceValue = parseFloat(priceValue);
		var presentPriceValue = $( this ).closest('.basket-row').find('.sum-container span').text();
		presentPriceValue = parseFloat(presentPriceValue);
		$( this ).closest('.basket-row').find('.sum-container span').text(priceValue + presentPriceValue + ' руб.')
	});

	$('.m-btn').click(function(e) {
		e.preventDefault();
		var one = 1;
		var presentValue = $( this ).siblings('input:text').val();
		if (presentValue > 1) {
			$( this ).siblings('input:text').val(Number(presentValue) - one);
			var priceValue = $( this ).closest('.basket-row').find('.sum-container span').attr('data-sum');
			priceValue = parseFloat(priceValue);
			var presentPriceValue = $( this ).closest('.basket-row').find('.sum-container span').text();
			presentPriceValue = parseFloat(presentPriceValue);
			$( this ).closest('.basket-row').find('.sum-container span').text(presentPriceValue - priceValue + ' руб.');
		};
	});

Много повторяющихся строк в коде. Возможно ли вынести некоторые строки в функцию для сокращения записи?

Answer 1

[Роман Огарков]

var calculateQuantityPrice = function($action, $this) {
		var $siblings = $this.closest('.input-row').find('input:text');
		var $basketSum = $this.closest('.basket-row').find('.sum-container span');
		var presentValue = parseInt($siblings.val());
		var priceValue = parseInt($basketSum.attr('data-sum'));
		var presentPriceValue = parseInt($basketSum.text());

	    if ($action == 'plus') {
	        $siblings.val(presentValue + 1);
	        $basketSum.text(priceValue + presentPriceValue + ' руб.');
	    } else if ($action == 'minus') {
	        $siblings.val(presentValue - 1);
	        $basketSum.text(presentPriceValue - priceValue + ' руб.');
	    }
	};

	$('.p-btn').click(function(e) {
	    e.preventDefault();

	    calculateQuantityPrice('plus', $( this ))
	});

	$('.m-btn').click(function(e) {
	    e.preventDefault();

	    calculateQuantityPrice('minus', $( this ));
	});

Answer 2

[v- smerti]

Один уровень абстракции на функцию
Функция должна выполнять только 1 операцию. Она должна выполнять её хорошо. И не чего другого она делать не должна.

Answer 3

[Inwork277]

Ну дак засуньте это всё в функцию и передавайте туда параметр
isPlus == true -> сложение
isPlus == false -> вычитание

Comments

[v- smerti]

Говнокод. Функция должна выполнять только 1 операцию. Она должна выполнять её хорошо. И не чего другого она делать не должна.

[Inwork277]

Владимир Грабко: как Вашей душе угодно