Какие советы можете дать по улучшению данного кода?

Question

[DeniSidorenko]

Добрый день есть такой код фильтра на JS
Буду рад если потратите минуту своего времени и критичным взглядом напишите что следует поменять и что улучшить
заранее спасибо

$('#filter-aply').on('click',function(){


    var etage = $('.filter-item__buttons').children('.button.active').data('etg')
    etage = parseFloat(etage)


    // Square
    var squareMin = $("#js-range__metr").data('from')
    squareMin = parseFloat(squareMin)

    var squareMax = $("#js-range__metr").data('to')
    squareMax = parseFloat(squareMax)


    // Price
    var priceMin = $("#js-range__price").data('from')
    priceMin = parseFloat(priceMin)

    var priceMax = $("#js-range__price").data('to')
    priceMax = parseFloat(priceMax)

    var dataCount = 0;

    var saleItems = $('.sale-item')



    for(var i = 0; i<saleItems.length; i++){

      var item = saleItems[i]

      var etageLocal = $(item).data('etage')
      etageLocal = parseFloat(etageLocal)

      var price = $(item).data('price')
      price = parseFloat(price)

      var square = $(item).data('square')
      square = parseFloat(square)


      if(price <= priceMax && price >= priceMin){
        eqPrice = true;
      }
      else{
        eqPrice = false;
      }
      if(etageLocal == etage){
        eqEtage = true;
      }
      else{
        eqEtage = false;
      }
      if(square <= squareMax && square >= squareMin){
        eqSquare = true
      }
      else{
        eqSquare = false;
      }


      if(eqEtage == false || eqPrice == false || eqSquare == false){
        $(item).removeClass('showed')
        $(item).removeAttr('data-count')
      }
      if(eqPrice == true && eqEtage == true && eqSquare == true){
        $(item).addClass('showed')
        dataCount++;
        $(item).attr('data-count', dataCount)
        $('.sale-item__eye').attr('data-position', dataCount)
        $('.sale-item[data-count="1"]').click()
      }



    }


  })

Comments

[Михаил]

смешивание нативного js с jquery,
какой толк от отдельного parseFloat? etage = parseFloat(etage)

[Шохрух Шаймардонов]

Избавьтесь от jquery. Когда сделаете это, скорее всего уже на процентов 20 улучшится код, вот тогда и можно покритиковать

Answer 1

[Антон Неверов]

$('#filter-aply').on('click', () => {
	const jsMetr = $("#js-range__metr");
	const jsRange = $("#js-range__price");

	const etage = parseFloat($('.filter-item__buttons').children('.button.active').data('etg'));

	// Square
	const squareMin = parseFloat(jsMetr.data('from'));
	const squareMax = parseFloat(jsMetr.data('to'));

	// Price
	const priceMin = parseFloat(jsRange.data('from'));
	const priceMax = parseFloat(jsRange.data('to'));

	let dataCount = 0;

	$('.sale-item').each(function() {
		const etageLocal = parseFloat($(this).data('etage'));
		const price = parseFloat($(this).data('price'));
		const square = parseFloat($(this).data('square'));

		let eqPrice = false;
		let eqEtage = false;
		let eqSquare = false;

		if (price <= priceMax && price >= priceMin) eqPrice = true;
		if (etageLocal == etage) eqEtage = true;
		if (square <= squareMax && square >= squareMin) eqSquare = true


		if (!eqEtage || !eqPrice || !eqSquare) {
			$(this).removeClass('showed')
			$(this).removeAttr('data-count')
		}

		if(eqPrice && eqEtage && eqSquare){
			$(this).addClass('showed')
			dataCount++;
			$(this).attr('data-count', dataCount)
			$('.sale-item__eye').attr('data-position', dataCount)
			$('.sale-item[data-count="1"]').click()
		}
	});
});

Comments

[Антон Неверов]

Pardon Me! Where Do I Find 4giveness?, я не использую this в стрелочной функции.

$('.sale-item').each(function() {
...
$(this)...