Как сделать данный код более простым и универсальным на JS?

Question

[DeniSidorenko]

Получаем такую строку ( string)
1.400.000 $
Задача, умножить данную строку на переменную rate и вернуть новое значения в таком состояние. ( Смена валюты) Я перевел из string в number( хотя мне кажется что не совсем корректно и можно рефакторинг сделать в данном случае )

Проблема еще в том как новое значения вернуть в состояние которая была, потому что новое значения получается слитно. Заранее спасибо ))

$('#cash-items li').on('click',function(){

    var name = $(this).text()
    var rate = $(this).data('rate')
    rate = parseFloat(rate)



    var itemsChanged = $('.rate_item__change')
    for(var i = 0; i<itemsChanged.length; i++){
      var price = $(itemsChanged[i]).text();
      price = price.replace(' $', '')
      price = price.replace('.', '')
      price = price.replace('.', '')
      price = parseFloat(price)
      
      var newPrice = price * rate;
      newPrice = parseInt(newPrice)
      $(itemsChanged[i]).text(newPrice + ' ' + name);
    }

  })

Comments

[Одиночка Айс]

Например, погуглить? https://coderwall.com/p/5ftjsw/js-convert-price-by... Или засунуть ваш код в функцию, или парсить цену на сервере, например через PHP - https://www.php.net/manual/ru/function.money-format.php

[i1yas]

Проблема еще в том как новое значения вернуть в состояние которая была, потому что новое значения получается слитно

Вынесети форматтер валюты в отдельную функцию и вызывайте когда нужно, простой пример:

function formatCurrency(value, type) {
  var dots = value.toLocaleString().split(',').join('.');
  if (type == "dollar") return dots + " $"
  if (type == "euro") return dots + " €";
  return dots;
}

Answer 1

[Александр Таратин]

function strReverse(s){
return s.split('').reverse().join('');
}
var f = strReverse(strReverse(('1.400.000 $'.replace(/\D/g, '') * 1000).toString()).split(/(\d{3})/).filter(Boolean).join('.'))
console.log(f)

Comments

[DeniSidorenko]

Спасибо большое