Почему после работы скипта, срабатывает функция keyup?

Question

[Cheizer]

Друзья, помогите пожалуйста, всю голову сломал!
Есть простой скрипт калькулятора расчет стоимости услуг, на jquery.
Перерасчет делает сразу при смене значений в input по change.

К этому калькулятору я добавил jquery библиотеку CountTo , это просто для анимации счетчика, думаю знаете.

Все сделал, все работает, но вот засада, не могу понять, почему то после смены числа в input, счетчик отработает пересчитает и ЕСЛИ ПОСЛЕ ЭТОГО сделать клик мыши в сторону, число сбросится до нуля и опять пересчитает сумму второй раз.

Как будто срабатывает keyup или еще что то :( не могу понять как этот баг исправить, подскажите гуру js куда копать????

Вот поднял это все на codepin https://codepen.io/Cheizer/pen/WyErgB

Answer 1

[Никита]

У вас не keyup срабатывает а change, а change срабатывает когда фокус с поля теряется, ну и в итоге так и получается )

Comments

[Cheizer]

значит change, но при смене значения в select такой глюк не происходит, как это пофиксить? Подскажите пожалуйста.

[Никита]

убрать change на инпутах, а на селектах оставить

[Никита]

Никита, либо, на ченьж сравнивать вычисляемое значение с тем, что вы водится, и пересчитывать только когда они различны

[Cheizer]

Никита, а зачем так сравнивать? Просто убрал для input и вроде порядок! :)

jQuery('#calculating select').change(function() {
            calculate();
 });

[Ihor Bratukh]

Роман Кузнецов, а зачем целый плагин подключать для анимации чисел, если этот функционал уже есть в jQuery, замените содержимое функции counter(total)

function counter(total) {
	var $total = $('i.total');
	$({ Counter: parseFloat($total.text()) }).animate({ Counter: total }, {
		duration: 500,
		easing: 'swing',
		step: function() {
			var value = this.Counter.toFixed(1);
			value = value.replace(/(\d)(?=(\d{3})+(\D|$))/g, '$1 ');
			$total.text(value);
		}
	});
}

[Cheizer]

BRAGA96, спасибо, не знал, но что то с ним не так, точку добавляет в конце если число большое, и скорость лютая, неуправляемая, вернее шаг он каждое число мотает, а надо пропускать что ли, не знаю но что то не то с ним.

[Ihor Bratukh]

Роман Кузнецов, вот Вам еще функции для чисел, форматирование числа в пользовательский формат и фиксирование без округления.

Использовать так:

setNumberToUserFormat(setNumberToFixed(2553267.46382, 4), [' ', ',']);
//> "2 553 267,4638"

/**
 * Установить к-во чисел после запятой (без округления)
 * @param {number} number - число
 * @param {number} fixed - к-во чисел после запятой
 * @returns {number} - обрезанное число
**/
function setNumberToFixed(number, fixed) {
	if ((typeof number === 'number' || typeof number === 'string') && !isNaN(number - parseFloat(number))) {
		number = String(number);
		if (number.split('.').length > 1) {
			var left = number.split('.')[0];
			var right = number.split('.')[1].substr(0, (!fixed ? 4 : fixed));
			return Number(left + (fixed !== 0 ? '.' + right : ''));
		} else {
			return Number(number);
		}
	}
}

/**
 * Форматирования числа в пользовательский формат
 * @param {number} number - число
 * @param {array} format - формат числа, [0] - тысячи, [1] - дробные
 * @returns {string} - число в новом формате
**/
function setNumberToUserFormat(number, format) {
	if ((typeof number === 'number' || typeof number === 'string') && !isNaN(number - parseFloat(number))) {
		number = String(number);
		var format = $.extend({}, [' ', '.'], format);
		var reg = { dot: new RegExp(/\./g), digit: new RegExp(/(\d)(?=(\d\d\d)+([^\d]|$))/g) };
		if (number[0] === '0') {
			return number.replace(reg.dot, format[1]);
		} else if (number.split('.').length == 1) {
			return number.split('.')[0].replace(reg.digit, '$1' + format[0]);
		} else {
			var start = number.split('.')[0].replace(reg.digit, '$1' + format[0]);
			var end = number.split('.')[1].replace(reg.dot, format[1]);
			return start + format[1] + end;
		}
	}
}

[Cheizer]

BRAGA96, :)))) получается тот же CounTo по размеру, но спасибо, буду иметь ввиду.