Оптимизировать функцию чисел javascript - PullRequest
Новая
       12

Оптимизировать функцию чисел javascript

0 голосов
/ 19 января 2019

Моя цель - получить пол номера с помощью JavaScript.Например: 

// 23 -> 20;
// 234 -> 230;
// 2345 -> 2300;
...

Я использую следующую функцию: 

var usersCount = 23;
var rounded;
if (usersCount < 100) {
    rounded = Math.floor(usersCount / 10) * 10;
} else if (usersCount < 1000) {
    rounded = Math.floor(usersCount / 100) * 100;
} else if (usersCount < 10000) {
    rounded = Math.floor(usersCount / 1000) * 1000;
} else if (usersCount < 100000) {
    rounded = Math.floor(usersCount / 10000) * 10000;
} else if (usersCount < 1000000) {
    rounded = Math.floor(usersCount / 100000) * 100000;
} else {
    rounded = usersCount;
}

Мне нужно улучшить эту функцию, чтобы она была динамичной, чтобы избежать установки else if или каких-либо переключателей.Как я могу это сделать?Есть ли какой-нибудь лучший подход для достижения желаемого результата?

Ответы [ 4 ]

0 голосов
/ 19 января 2019

Следующая функция воспроизводит поведение, приведенное в ваших примерах: 

function f(x) {
    var m, u;
    m = Math.floor(Math.log10(x));
    u = Math.pow(10, Math.max(1, m - 1));
    return u * Math.floor(x / u);
}

Если вы были бы довольны округлением до 2 значащих цифр, вы можете использовать 

function f(x) {
    return Number.parseFloat(x.toPrecision(2));
}
0 голосов
/ 19 января 2019

Вы можете использовать длину строки для вычисления множителя, используя Math.pow : 

const usersCount = 23;    
const round = s => {
    let c = Math.pow(10, s.toString().length - 1);
    return Math.floor(s / c) * c;
};
console.log(round(usersCount)); // 20


const round = s => {
  let c = Math.pow(10, s.toString().length - 1);
  return Math.floor(s / c) * c;
};
[
  23,
  199,
  888,
  99999,
  100001
].forEach(x => {
  console.log(round(x));
});
0 голосов
/ 19 января 2019

Самый компактный и оптимизированный способ переписать ваш код, который я сейчас могу себе представить, заключается в следующем: 

let usersCount = 23;
let rounded;
let x = 10 ** (usersCount.toString().length - 1);
rounded = Math.floor(usersCount / x) * x;

Он должен работать рекурсивно без ограничений, как вы и просили.

0 голосов
/ 19 января 2019

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

const userCount = 234567;

const roundIt = n => {
	const numberLength = Math.ceil(Math.log10(n + 1));
  const decrease = n > 100 ? 2 : 1;
  return Math.floor(n / Math.pow(10, numberLength - decrease)) * Math.pow(10, numberLength - decrease);
};

// Display user count
document.querySelector(".user-count").innerHTML = userCount;

// Display rounded count
document.querySelector(".user-count-rounded").innerHTML = roundIt(userCount);
<h1>
User count: <span class="user-count">0</span>
</h1>
<h1>
Rounded user count: <span class="user-count-rounded">0</span>
</h1>

Мы используем длину числа и округляем его, используя тот же код, который вы написали.

Вот скрипка: https://jsfiddle.net/anandchowdhary/fj0zk1x8/7/

...