Tengo la siguiente const "maxRange"... que compara dos valores y luego establece uno de esos valores si es menor que. ¿Hay una forma mejor/más limpia de lograr esta misma lógica?
const maxRange = Math.round(averageTotalPrice / 50) * 50 * 5.5 < maxTotalPrice ? Math.round(averageTotalPrice / 50) * 50 * 5.5 : maxTotalPriceParece que necesitas Math.min :
const maxRange = Math.min( Math.round(averageTotalPrice / 50) * 50 * 5.5, maxTotalPrice );creo que esto es mejor
let round = Math.round(averageTotalPrice / 50) * 50 * 5.5; const maxRange = Math.min(round, maxTotalPrice);Comenzaría deshaciéndome de los números mágicos 50 y 5.5 y usando Math.min en lugar del operador ternario.
Si Math.round(averageTotalPrice / MAGICAL_50) * MAGICAL_50 * MAGICAL_5_5 no es absolutamente obvio en el dominio en el que está trabajando, también lo pondría en una constante solo para aclarar de qué se trata.
Aquí hay un hipotético:
const BUCKET_SIZE = 5.5; const WHOLESALE_AMOUNT = 50; const maxRange = Math.min(Math.round(averageTotalPrice / WHOLESALE_AMOUNT ) * WHOLESALE_AMOUNT * BUCKET_SIZE, maxTotalPrice);