<!doctype html> <html> <script> function isNumber(value) { return typeof (value) != "boolean" && !isNaN(value) && value.length > 0; } function minMaxDefrost(value, min, max) { if (value.length == 0 || value == "-") return value; if (!isNumber(value)) return value.substring(0, value.length - 1); console.log("minMaxDefrost called ") if (parseFloat(value) < min){ return min; } else if (parseFloat(value) > max){ return max; } else { return Math.floor(value); } } minMaxDefrost(4, 0, 12); </script> </html>isNumber(4) se evalúa como false , por lo que está intentando llamar a un método de substring en 4 pero no existe. Use typeof value === 'number' en su lugar para probar si algo es un número. O mejor aún, use typeof value === 'string' antes de tratarlo como si fuera definitivamente una cadena.
isNumber no hace exactamente su trabajo como sugiere Mark Hanna... un mejor isNumber sería algo como
function isNumber(num){ let num1=num-0, num2=num-1 return num2!=num1 && !!(num2||num1) && typeof(num)!="object" //Infinity would fail the test num2!=num1 //Normal non-numbers would fail !!(num2||num1) //null would fail typeof(null)!="object" //However, 0, and even "0" will pass this test }Aquí está devuelto en el código que nos diste
function isNumber(num){ let num1=num-0, num2=num-1 return num2!=num1 && !!(num2||num1) && typeof(num)!="object" //Infinity would fail the test num2!=num1 //Normal non-numbers would fail !!(num2||num1) //null would fail typeof(null)!="object" //However, 0, and even "0" will pass this test } function minMaxDefrost(value, min, max) { if (value.length == 0 || value == "-") return value; if (!isNumber(value)) return value.substring(0, value.length - 1); console.log("minMaxDefrost called ") if (parseFloat(value) < min){ return min; } else if (parseFloat(value) > max){ return max; } else { return Math.floor(value); } } minMaxDefrost(4, 0, 12);